code clean-up

be more prudent before freeing memory
lyonel · Oct 12, 2021 · 353866c · 353866c
1 parent 0b8149a
commit 353866c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/core/usb.cc b/src/core/usb.cc
@@ -311,6 +311,7 @@ static bool load_usbids(const string & name)
         buffer[linelen-1] = '\0';                 // chop \n
       string line = string(buffer);
       free(buffer);
+      buffer = NULL;
 
       description = NULL;
       t = 0;
@@ -338,6 +339,7 @@ static bool load_usbids(const string & name)
         }
       }
     }
+    if(buffer != NULL) free(buffer);
   }
 
   fclose(usbids);
@@ -393,6 +395,7 @@ bool scan_usb(hwNode & n)
     {
       string line = hw::strip(string(buffer));
       free(buffer);
+      buffer = NULL;
 
       if(line.length()<=0)
       {
@@ -497,6 +500,7 @@ bool scan_usb(hwNode & n)
         }
       }
     }
+    if(buffer != NULL) free(buffer);
   }
   if(defined)
     addUSBChild(n, device, bus, lev, prnt);