zynaddsubfx

commit eee5dd1f8758edbd61383a1bfb881b051e29b526
parent 2fc9fb5345d679f02b4fc4d2f62be6cf3ab7c9b2
Author: Friedolino <[email protected]>
Date:   Sat, 20 Jun 2020 23:54:23 +0200

fix strncpy and remove debug printf code

Diffstat:
M
src/Misc/Part.cpp
 | 
8
+++++---

1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/Misc/Part.cpp b/src/Misc/Part.cpp
@@ -174,10 +174,8 @@ static const Ports partPorts = {
                 time (&rawtime);
                 const struct tm* timeinfo = localtime (&rawtime);
                 strftime (p->loaded_file,23,"%F_%R.xiz",timeinfo); 
-                printf("ceated new filename %s\n", p->loaded_file);
             }
             p->saveXML(p->loaded_file);
-            printf("saving to file %s\n", p->loaded_file);
         }},
     //{"kit#16::T:F", "::Enables or disables kit item", 0,
     //    [](const char *m, RtData &d) {
@@ -1053,6 +1051,7 @@ int Part::saveXML(const char *filename)
 
     int result = xml.saveXMLfile(filename, gzip_compression);
     strncpy(loaded_file,filename, sizeof(loaded_file));
+    loaded_file[sizeof(loaded_file)-1] = '\0';
     return result;
 }
 
@@ -1067,7 +1066,10 @@ int Part::loadXMLinstrument(const char *filename)
         return -10;
 
     // store filename in member variable
-    strncpy(loaded_file,filename, sizeof(loaded_file));
+    int length = sizeof(loaded_file)-1;
+    strncpy(loaded_file, filename, length);
+    // set last element to \0 in case filname is too long or not terminated
+    loaded_file[length]='\0';
 
     getfromXMLinstrument(xml);
     xml.exitbranch();