src: clang build warning fix

fix UB with string concatenations. += operator makes things clearer for compiler's perspective. PR-URL: #28480 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Jul 12, 2019 · fe1b96d · fe1b96d
1 parent 1cc5c54
commit fe1b96d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/node_metadata.cc b/src/node_metadata.cc
@@ -34,8 +34,10 @@ std::string GetOpenSSLVersion() {
   // sample openssl version string format
   // for reference: "OpenSSL 1.1.0i 14 Aug 2018"
   char buf[128];
-  const int start = search(OPENSSL_VERSION_TEXT, 0, ' ') + 1;
-  const int end = search(OPENSSL_VERSION_TEXT + start, start, ' ');
+  const char* etext = OPENSSL_VERSION_TEXT;
+  const int start = search(etext, 0, ' ') + 1;
+  etext += start;
+  const int end = search(etext, start, ' ');
   const int len = end - start;
   snprintf(buf, sizeof(buf), "%.*s", len, &OPENSSL_VERSION_TEXT[start]);
   return std::string(buf);