Changeset 33658 for trunk/Ohana/src/getstar

Timestamp:

Apr 1, 2012, 3:11:00 PM (14 years ago)

Author:

eugene

Message:

replace strdup with strcreate (ohana memory safety)

Location:

trunk/Ohana/src/getstar/src

Files:

• args_coords.c (modified) (2 diffs)
• dvoImagesAtCoords.c (modified) (2 diffs)

trunk/Ohana/src/getstar/src/args_coords.c

@@ -26 +26 @@
       exit (1);
     }
-    coordsFile = strdup(argv[N]);
+    coordsFile = strcreate(argv[N]);
     remove_argument (N, &argc, argv);
   }
@@ -36 +36 @@
       exit (1);
     }
-    astromFile = strdup(argv[N]);
+    astromFile = strcreate(argv[N]);
     remove_argument (N, &argc, argv);
   }

trunk/Ohana/src/getstar/src/dvoImagesAtCoords.c

@@ -142 +142 @@
         char *left_bracket = rindex(dbImages[N].name, '[');
         if (!fullNames && left_bracket) {
-            copy = strdup(left_bracket + 1);
+            copy = strcreate(left_bracket + 1);
             name = copy;
             // zap the .hdr]
@@ -158 +158 @@
         }
         if  (copy) {
-            // XXX: Temporarily leak this copy. free gets redirected to
-            // ohana_free but strdup didn't allocate the memory
-            // free(copy);
+          free(copy);
         }
     }