Revert "Coverity #1098451,#1098452,#1098453 Resource Leak Fixed" 01/18001/1
Clément DAVID [Tue, 29 Mar 2016 14:46:14 +0000 (16:46 +0200)]
The fix is invalid, the change should be made on the
getAllocatedSingleString() function.

This reverts commit 4dca6265af8afb634412f1bfbfa06bd9d4e09b84.

Change-Id: I10c829d7c745bb4168245752fd75c696668b0cc0

scilab/modules/arnoldi/sci_gateway/c/sci_dseupd.c

index 0db9a67..86274db 100644 (file)
@@ -439,7 +439,6 @@ int sci_dseupd(char *fname, void *pvApiCtx)
     // Retrieve a matrix of double at position 2.
     if (getAllocatedSingleString(pvApiCtx, piAddrpHOWMANY, &pHOWMANY))
     {
-        freeAllocatedSingleString(pHOWMANY);
         Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), fname, 2);
         return 1;
     }
@@ -455,7 +454,6 @@ int sci_dseupd(char *fname, void *pvApiCtx)
     // Retrieve a matrix of double at position 7.
     if (getAllocatedSingleString(pvApiCtx, piAddrpBMAT, &pBMAT))
     {
-        freeAllocatedSingleString(pBMAT);
         freeAllocatedSingleString(pHOWMANY);
         Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), fname, 7);
         return 1;
@@ -474,7 +472,6 @@ int sci_dseupd(char *fname, void *pvApiCtx)
     // Retrieve a matrix of double at position 9.
     if (getAllocatedSingleString(pvApiCtx, piAddrpWHICH, &pWHICH))
     {
-        freeAllocatedSingleString(pWHICH);
         freeAllocatedSingleString(pBMAT);
         freeAllocatedSingleString(pHOWMANY);
         Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), fname, 9);