Coverity #1321243, #1321246, #1321242, #1321236, #1321239, #1321238, #1097871, #13212... 25/18025/8
Dhruv Khattar [Sun, 3 Apr 2016 19:44:09 +0000 (00:44 +0530)]
Change-Id: I65f241e512d50c7546bc1386e2833ea76de373da

scilab/modules/elementary_functions/sci_gateway/cpp/sci_sum.cpp
scilab/modules/hdf5/sci_gateway/cpp/sci_hdf5_listvar_v3.cpp
scilab/modules/matio/src/cpp/GetCharVariable.cpp
scilab/modules/parameters/src/c/parameters.c
scilab/modules/polynomials/sci_gateway/cpp/sci_pppdiv.cpp

index 462ba3a..0d87ffc 100644 (file)
@@ -120,12 +120,6 @@ types::Function::ReturnValue sci_sum(types::typed_list &in, int _iRetCount, type
         {
             std::wstring wstFuncName = L"%" + in[0]->getShortTypeStr() + L"_sum";
             types::Function::ReturnValue ret = Overload::call(wstFuncName, in, _iRetCount, out);
-
-            if (isCopy && pDblIn)
-            {
-                pDblIn->killMe();
-            }
-
             return ret;
         }
     }
index 80edcc4..65f07e2 100644 (file)
@@ -78,7 +78,7 @@ types::Function::ReturnValue sci_hdf5_listvar_v3(types::typed_list &in, int _iRe
         return types::Function::Error;
     }
 
-    if (_iRetCount < 1 && _iRetCount > 4)
+    if (_iRetCount < 1 || _iRetCount > 4)
     {
         Scierror(999, _("%s: Wrong number of output argument(s): %d to %d expected.\n"), fname.data(), 1, 4);
         return types::Function::Error;
index 31875b2..8176974 100644 (file)
@@ -115,7 +115,7 @@ matvar_t* GetCharMatVar(types::String* pStr, const char* name)
             Scierror(999, _("%s: No more memory.\n"), "GetCharMatVar");
             return NULL;
         }
-        
+
         for (int i = 0; i < pDims[0]; ++i)
         {
             ppcName[i] = wide_string_to_UTF8(pStr->get(i));
index 62abbfb..98b5c44 100644 (file)
@@ -715,7 +715,6 @@ SciErr getColVectorOfDoubleInPList(void* _pvCtx, int * _piAddress, const char *
     int pos_label = 0, i = 0;
     int m_label = 0, n_label = 0;
     int m_tmp = 0, n_tmp = 0;
-    int * len_label = NULL;
     double * tmp_values = NULL;
     char ** label_list = NULL;
     SciErr _SciErr;
@@ -728,11 +727,6 @@ SciErr getColVectorOfDoubleInPList(void* _pvCtx, int * _piAddress, const char *
     pos_label = commonFindLabel(_pvCtx, _piAddress, _pstLabel);
     *_piFound = pos_label;
 
-    if (len_label)
-    {
-        FREE(len_label);
-        len_label = NULL;
-    }
     freeArrayOfString(label_list, m_label * n_label);
 
     if (pos_label != -1)
index 47671a8..580357d 100644 (file)
@@ -37,7 +37,6 @@ types::Function::ReturnValue sci_pppdiv(types::typed_list &in, int _iRetCount, t
 {
     double* pdblInR[2]  = {NULL, NULL};// real part of denominator and numerator
     double* pdblInI[2]  = {NULL, NULL};// rimaginary part
-    bool bDouble        = false;
     bool pbComplex[2]   = {false, false};
     int piSize[2]       = {0, 0}; // rank+1 of denominator and numerator
     int iErr            = 0;
@@ -62,12 +61,6 @@ types::Function::ReturnValue sci_pppdiv(types::typed_list &in, int _iRetCount, t
     {
         if (in[i]->isDouble())
         {
-            if (bDouble)
-            {
-                Scierror(999, _("%s: Wrong type for input argument #%d: A polynomial expected.\n"), "pppdiv", i + 1);
-                return types::Function::Error;
-            }
-
             types::Double* pDblIn = in[i]->getAs<types::Double>();
             if (pDblIn->isScalar() == false)
             {