Coverity #1099209, #1099221, #1099225 fixed 73/18373/2
Dhruv Khattar [Sun, 17 Jul 2016 19:11:00 +0000 (00:11 +0530)]
Change-Id: I5a8f358c58809d7ae33ab17cf930f3fb084e194f

scilab/modules/xml/sci_gateway/cpp/sci_percent_foo_i_XMLList.cpp
scilab/modules/xml/src/cpp/XMLDocument.cpp

index 24731df..835406e 100644 (file)
@@ -98,7 +98,7 @@ int sci_percent_foo_i_XMLList(char *fname, void* pvApiCtx)
     prefix = os_strdup(fname);
     underscore = strchr(prefix, '_');
     *underscore = '\0';
-    format = (char *)MALLOC(sizeof(char *) * (strlen(prefix) + strlen("_xmlFormat") + 1));
+    format = (char *)MALLOC(sizeof(char) * (strlen(prefix) + strlen("_xmlFormat") + 1));
     sprintf(format, "%s_xmlFormat", prefix);
     free(prefix);
 
index 3b9e6c8..fd8dbaf 100644 (file)
@@ -109,7 +109,7 @@ XMLDocument::XMLDocument(char *uri, char *version): XMLObject()
 
     if (expandedPath)
     {
-        newUri = (char *)xmlMalloc(sizeof(char *) * (strlen(expandedPath) + 1));
+        newUri = (char *)xmlMalloc(sizeof(char) * (strlen(expandedPath) + 1));
         memcpy(newUri, expandedPath, sizeof(char) * (strlen(expandedPath) + 1));
         document->URL = (xmlChar *) newUri;
         FREE(expandedPath);
@@ -307,7 +307,7 @@ void XMLDocument::setDocumentURL(const std::string & url) const
     if (expandedPath)
     {
         xmlFree((void *)document->URL);
-        newURL = (char *)xmlMalloc(sizeof(char *) * (strlen(expandedPath) + 1));
+        newURL = (char *)xmlMalloc(sizeof(char) * (strlen(expandedPath) + 1));
         memcpy(newURL, expandedPath, sizeof(char) * (strlen(expandedPath) + 1));
         document->URL = (xmlChar *) newURL;
         FREE(expandedPath);