Changeset 24047 for gs2-extensions/tdb-edit/trunk/src/src/tdbget-src/tdbget.cpp

Timestamp:

2011-05-19T12:10:52+12:00 (13 years ago)

Author:

jmt12

Message:

I'd misunderstood where I needed to free the memory allocated to storing char*s in the TDB_DATA. Apparently I only have to free them when they are returned by tdb_fetch() or similar. Removed segfault causing calls to free()

File:

• gs2-extensions/tdb-edit/trunk/src/src/tdbget-src/tdbget.cpp (modified) (3 diffs)

gs2-extensions/tdb-edit/trunk/src/src/tdbget-src/tdbget.cpp

@@ -50 +50 @@
 printUsage (char *program_name)
 {
-  cerr << "usage: " << program_name << " <database_path> <key>" << endl;
-  cerr << endl;
+  cerr << "usage: " << program_name << " <database_path> <key>" << endl << endl;
 }
 /** printUsage() **/
@@ -92 +91 @@
     }
     cout << endl; // used to be printf("\n");
+    // caller responsible for freeing this memory
     free(value.dptr);
   }
@@ -98 +98 @@
     cout << endl; // used to be printf("\n");
   }
-  free(key.dptr);
 
   tdb_close (tdb);