g++ is more strict (no initialization of string to NULL -- since it cannot dereference)

Author: enebo

argparser.cpp

@@ -194,8 +194,10 @@ bool ArgParser::initPlatformDir() {
     if (!found) {               // try via PATH search
         logMsg("initPlatformDir: trying to find executable on PATH");
         std::string location = findOnPath(platformDir.c_str());
-        strncpy(path, location.c_str(), PATH_MAX);
-        found = true;
+        if (location.size() > 0) {
+            strncpy(path, location.c_str(), PATH_MAX);
+            found = true;
+        }
     }
 
     // Check if bin/jruby file exists; this logs a message if not found

utilsfuncs.cpp

@@ -144,7 +144,7 @@ string findOnPath(const char* name) {
 
 	start = sep + 1;
     }
-    return NULL;
+    return "";
 }
 
 const char* getSysError(char *str, int strSize) {