Coverity #1321256, #1321259 fixed

Change-Id: Ic63eecf18467f93a27acef5d6936fccc0bef714f

diff --git a/scilab/modules/preferences/sci_gateway/cpp/sci_addModulePreferences.cpp b/scilab/modules/preferences/sci_gateway/cpp/sci_addModulePreferences.cpp
index 3e06d3a..7542c2b 100644 (file)
--- a/scilab/modules/preferences/sci_gateway/cpp/sci_addModulePreferences.cpp
+++ b/scilab/modules/preferences/sci_gateway/cpp/sci_addModulePreferences.cpp
@@ -67,7 +67,10 @@ int sci_addModulePreferences(char * fname, void *pvApiCtx)
             return 0;
         }
 
-        getAllocatedSingleString(pvApiCtx, addr, array[i]);
+        if (getAllocatedSingleString(pvApiCtx, addr, array[i]))
+        {
+            return 0;
+        }
     }
 
     expTbxPath = expandPathVariable(const_cast<char *>(tbxPath));

diff --git a/scilab/modules/preferences/sci_gateway/cpp/sci_removeModulePreferences.cpp b/scilab/modules/preferences/sci_gateway/cpp/sci_removeModulePreferences.cpp
index 805718e..218b90a 100644 (file)
--- a/scilab/modules/preferences/sci_gateway/cpp/sci_removeModulePreferences.cpp
+++ b/scilab/modules/preferences/sci_gateway/cpp/sci_removeModulePreferences.cpp
@@ -51,7 +51,10 @@ int sci_removeModulePreferences(char * fname, void *pvApiCtx)
         {
             Scierror(999, gettext("%s: Wrong type for input argument #%i: string expected.\n"), fname, 1);
         }
-        getAllocatedSingleString(pvApiCtx, addr, &tbxName);
+        if (getAllocatedSingleString(pvApiCtx, addr, &tbxName))
+        {
+            return 0;
+        }
     }
 
     try