Prevent division by zero (#335)

* Prevent division by zero

* Throw exception is size = 0

Author: alxbilger

Plugin/src/SofaPython3/DataHelper.cpp

@@ -362,6 +362,8 @@ py::buffer_info toBufferInfo(BaseData& m)
 
 py::object convertToPython(BaseData* d)
 {
+    assert(d != nullptr);
+
     const AbstractTypeInfo& nfo{ *(d->getValueTypeInfo()) };
     if(hasArrayFor(d)){
         return getPythonArrayFor(d);
@@ -377,21 +379,28 @@ py::object convertToPython(BaseData* d)
         }
         else
         {
-            size_t dim0 = nfo.size(d->getValueVoidPtr())/nfo.size();
-            size_t dim1 = nfo.size();
-            for(size_t i=0;i<dim0;i++)
+            if (nfo.size() != 0)
             {
-                py::list list1;
-                for(size_t j=0;j<dim1;j++)
+                size_t dim0 = nfo.size(d->getValueVoidPtr())/nfo.size();
+                size_t dim1 = nfo.size();
+                for(size_t i=0;i<dim0;i++)
                 {
-                    if(nfo.Integer())
-                        list1.append(nfo.getIntegerValue(d->getValueVoidPtr(),i*dim1+j));
-                    else if(nfo.Scalar())
-                        list1.append(nfo.getScalarValue(d->getValueVoidPtr(),i*dim1+j));
-                    else
-                        throw py::type_error("Invalid type");
+                    py::list list1;
+                    for(size_t j=0;j<dim1;j++)
+                    {
+                        if(nfo.Integer())
+                            list1.append(nfo.getIntegerValue(d->getValueVoidPtr(),i*dim1+j));
+                        else if(nfo.Scalar())
+                            list1.append(nfo.getScalarValue(d->getValueVoidPtr(),i*dim1+j));
+                        else
+                            throw py::type_error("Invalid type");
+                    }
+                    list.append(list1);
                 }
-                list.append(list1);
+            }
+            else
+            {
+                throw std::runtime_error("Abstract type info corresponding to Data " + d->getName() + " is empty\n");
             }
         }
         return std::move(list);