Coverity #132120, #1097640, #1099299 fixed 60/18460/4
Dhruv Khattar [Sun, 14 Aug 2016 10:00:31 +0000 (15:00 +0530)]
Change-Id: I4e51ee7d5825c869af950c7a5f6a809bf393a08d

scilab/modules/hdf5/src/cpp/H5Dataspace.cpp
scilab/modules/hdf5/src/cpp/H5Dataspace.hxx
scilab/modules/hdf5/src/cpp/H5File.cpp
scilab/modules/hdf5/src/cpp/handle_properties.cpp

index 3d5aa4b..1bc91fb 100644 (file)
@@ -31,7 +31,7 @@ H5Dataspace::~H5Dataspace()
     }
 }
 
-hid_t H5Dataspace::getH5Id()
+hid_t H5Dataspace::getH5Id() const
 {
     return space;
 }
index 1b652c2..4c2e728 100644 (file)
@@ -31,7 +31,7 @@ public:
     H5Dataspace(H5Object & _parent, hid_t _space);
     ~H5Dataspace();
 
-    virtual hid_t getH5Id();
+    virtual hid_t getH5Id() const;
     virtual std::vector<unsigned int> getDims(const bool b) const;
     virtual std::string getTypeName() const;
 
index 8c0aab6..9db1827 100644 (file)
@@ -27,7 +27,7 @@ namespace org_modules_hdf5
 
 void H5File::init(const hid_t fapl)
 {
-    bool opened;
+    bool opened = false;
 
 #if !defined(__HDF5ERROR_PRINT__)
     H5Eset_auto(H5E_DEFAULT, 0, 0);
index 2e0476c..61da7a2 100644 (file)
@@ -2428,13 +2428,10 @@ static bool export_handle_uicontrol(int parent, int uid)
     getHandleIntProperty(uid, __GO_UI_STRING_SIZE__, &size);
     int col = 0;
     getHandleIntProperty(uid, __GO_UI_STRING_COLNB__, &col);
-    int row = size / col;
 
     int dims[2];
-    dims[0] = row;
-    dims[1] = col;
 
-    if (col == 0 || row == 0)
+    if (col == 0)
     {
         dims[0] = 1;
         dims[1] = 1;
@@ -2444,6 +2441,9 @@ static bool export_handle_uicontrol(int parent, int uid)
     }
     else
     {
+        int row = size / col;
+        dims[0] = row;
+        dims[1] = col;
         char** string = nullptr;
         getHandleStringVectorProperty(uid, __GO_UI_STRING__, &string);
         writeStringMatrix6(parent, "string", 2, dims, string);