Coverity #1352291 fixed - Xcos swig generated code 05/18005/3
Clément DAVID [Wed, 30 Mar 2016 08:48:24 +0000 (10:48 +0200)]
Change-Id: I0f8d4ae2b3e820f23ac8c78bb69b29f54343a25f

scilab/modules/xcos/src/jni/JavaController.i
scilab/modules/xcos/src/jni/JavaController_wrap.cxx

index 41e6731..6d24c3f 100644 (file)
@@ -208,14 +208,10 @@ namespace std {
     return $null;
   }
   $1 = &temp;
-  *$1 = "";
 }
 
 %typemap(argout) std::string &OUTPUT {
-  jstring jnewstring = NULL;
-  if ($1) {
-     jnewstring = JCALL1(NewStringUTF, jenv, $1->c_str());
-  }
+  jstring jnewstring = JCALL1(NewStringUTF, jenv, $1->c_str());
   JCALL3(SetObjectArrayElement, jenv, $input, 0, jnewstring);
 }
 
index 0325e94..0552529 100644 (file)
@@ -1584,15 +1584,11 @@ SWIGEXPORT jboolean JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_Contr
       return 0;
     }
     arg5 = &temp5;
-    *arg5 = "";
   }
   result = (bool)((org_scilab_modules_scicos::Controller const *)arg1)->getObjectProperty(arg2,arg3,arg4,*arg5);
   jresult = (jboolean)result; 
   {
-    jstring jnewstring = NULL;
-    if (arg5) {
-      jnewstring = jenv->NewStringUTF(arg5->c_str());
-    }
+    jstring jnewstring = jenv->NewStringUTF(arg5->c_str());
     jenv->SetObjectArrayElement(jarg5, 0, jnewstring);
   }
   return jresult;