Changeset 81bd7e3

Timestamp:

Apr 18, 2023, 2:09:52 PM (19 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

ADT, ast-experimental, master

Children:

a5ea261

Parents:

66b6f66

Message:

Adding explination to the memory leak in driver/cfa.cc.

File:

• driver/cfa.cc (modified) (1 diff)

driver/cfa.cc

@@ -44 +44 @@
         static int flags = 0;
 
+    // This allocation 'leaks' memory from the program to the execution
+    // environment, as putenv does not manage the storage of the string used
+    // as an environment variable. This leak is necessary to ensure the
+    // underlying C string is allocated long enough.
         if ( putenv( (char *)( *new string( string( __CFA_FLAGPREFIX__ + to_string( flags++ ) + "__=" ) + arg ) ).c_str() ) ) {
                 cerr << argv[0] << " error, cannot set environment variable." << endl;