Changeset 42938

Timestamp:

Nov 12, 2025, 4:47:51 PM (6 months ago)

Author:

eugene

Message:

free memory on exit

Location:

trunk/Ohana/src/opihi/lib.shell

Files:

• ConfigInit.c (modified) (1 diff)
• command.c (modified) (3 diffs)
• gprint.c (modified) (1 diff)

trunk/Ohana/src/opihi/lib.shell/ConfigInit.c

@@ -22 +22 @@
 }
 
-// this function is only called at shutdown, so it is not thread protect
+// this function is only called at shutdown, so it is not thread safe
 void ConfigFree () {
   if (GlobalConfig) free (GlobalConfig);
+  FreeConfigFile (); // frees the config variable structures
   return;
 }

trunk/Ohana/src/opihi/lib.shell/command.c

@@ -36 +36 @@
   /* solve math expresions, assign variable, if needed.  any entry in line of the form {foo}
    * returns value or tmp vector / buffer */
-  line = parse (&status, line);        /* line is freed here, new one allocated */
+  line = parse (&status, line);
+  /* NOTE: line is freed here, new one allocated.  This is NOT freed by the 'quit' command */
   if (!status) goto escape;
 
@@ -42 +43 @@
   *outline = line;
   
+  // this value is not freed by 'quit' (called as command below)
   argv = parse_commands (line, &argc);
   if (argc == 0) {
@@ -50 +52 @@
 
   /* save the original values of argv since command may modify the array */
+  // this value is not freed by 'quit' (called as command below)
   ALLOCATE (targv, char *, argc);
   for (i = 0; i < argc; i++) targv[i] = argv[i];

trunk/Ohana/src/opihi/lib.shell/gprint.c

@@ -93 +93 @@
 
   pthread_mutex_unlock (&init_stream_mutex);
+}
+
+// call just before exiting opihi-based programs
+void gprintFree () {
+  for (int i = 0; i < Nstreams; i++) {
+    FreeIOBuffer (streams[i][0].buffer);
+    FREE (streams[i][0].buffer);
+    FREE (streams[i][0].name);
+    FREE (streams[i]);
+  }
+  FREE (streams);
+  return;
 }