Fix Coverity issues introduced by https://codereview.scilab.org/#/c/13430/ ... 33/13433/1
Vincent COUVERT [Tue, 14 Jan 2014 09:10:30 +0000 (10:10 +0100)]
** CID 1154247:  Resource leak  (RESOURCE_LEAK)
/modules/string/sci_gateway/c/sci_convstr.c: 117 in sci_convstr()

** CID 1154246:  Unchecked return value  (CHECKED_RETURN)
/modules/xml/src/c/XMLMlistsManagement.c: 92 in createXMLObjectAtPos()

Change-Id: I049b5e2d17f1a4b6ed04208a82e182f46e1ea18e

scilab/modules/string/sci_gateway/c/sci_convstr.c
scilab/modules/xml/src/c/XMLMlistsManagement.c

index e7a85de..c966361 100644 (file)
@@ -113,6 +113,7 @@ int sci_convstr(char *fname, unsigned long fname_len)
     /* To input the string matrix */
     if (getAllocatedMatrixOfWideString(pvApiCtx, piAddr1, &iRows1, &iCols1, &pstInput))
     {
+        freeAllocatedMatrixOfWideString(iRows1, iCols1, pstInput);
         Scierror(999, _("%s: Wrong type for input argument #%d: Matrix of strings expected.\n"), fname, 1);
         return 0;
     }
index f0e80df..27d2b7e 100644 (file)
@@ -89,7 +89,6 @@ int createXMLObjectAtPos(int type, int pos, int id, void *pvApiCtx)
         printError(&err, 0);
         return 0;
     }
-    getMListType(mlistaddr, pvApiCtx);
     return 1;
 }