Changeset 32252 for trunk/magic/remove/src/streaksremove.c

Timestamp:

Aug 31, 2011, 1:30:49 PM (15 years ago)

Author:

bills

Message:

use psFitsTable structure when copying extra extensions.
Fix previously existing memory leak of extra extension header in sources file.

File:

• trunk/magic/remove/src/streaksremove.c (modified) (5 diffs)

trunk/magic/remove/src/streaksremove.c

@@ -9 +9 @@
 
 #include "streaksremove.h"
-
-#include "psFitsTableNew.h"
 
 static pmConfig *parseArguments(int argc, char **argv);
@@ -280 +278 @@
 
     if (statsFileName) {
-    // Write out
         psString resolved = pmConfigConvertFilename(statsFileName, config, true, true); // Resolved filename
         if (!resolved) {
@@ -294 +291 @@
     }
 
-    // all done. Clean up to look for memory leaks.
+    // all done. Clean up and look for memory leaks.
 
     psFree(sfiles);
@@ -1060 +1057 @@
         }
 
-        psArray *inTable = psFitsReadTable(in->fits);
-        if (!inTable) {
+        psFitsTable *table = psFitsReadTableNew(in->fits);
+        if (!table) {
             psErrorStackPrint(stderr, "failed to read table in %s", in->resolved_name);
             streaksExit("", PS_EXIT_DATA_ERROR);
         }
-        if (! psFitsWriteTable(out->fits, header, inTable, extname)) {
+        if (! psFitsWriteTableNew(out->fits, header, table, extname)) {
             psErrorStackPrint(stderr, "failed to write table to %s", out->resolved_name);
             streaksExit("", PS_EXIT_DATA_ERROR);
         }
+        psFree(header);
+        psFree(table);
     }
 
@@ -1075 +1074 @@
         streaksExit("", PS_EXIT_DATA_ERROR);
     }
-    out->fits = NULL;
-}
+    out->fits = NULL;   // freed by psFitsclose()
+}