Changeset 6737

Timestamp:

Mar 30, 2006, 11:51:49 AM (20 years ago)

Author:

Paul Price

Message:

Squashing memory leak.

Location:

branches/rel10_ifa/psModules/src/astrom

Files:

• pmConcepts.c (modified) (2 diffs)
• pmFPAConstruct.c (modified) (1 diff)

branches/rel10_ifa/psModules/src/astrom/pmConcepts.c

@@ -92 +92 @@
         // Trap the lists, which can't be copied in the ordinary way without a warning
         if (blank->type == PS_DATA_LIST) {
-            copy = psMetadataItemAlloc(blank->name, PS_DATA_LIST, blank->comment, blank->data.V);
+            copy = psMemIncrRefCounter(blank->data.V);
         } else {
             copy = psMetadataItemCopy(blank);
@@ -99 +99 @@
             psLogMsg(__func__, PS_LOG_WARN, "Unable to add blank version of concept %s\n", blank->name);
         }
+        psFree(copy);                   // Drop reference
     }
     psFree(specsIter);

branches/rel10_ifa/psModules/src/astrom/pmFPAConstruct.c

@@ -119 +119 @@
         psFree(chip);                   // Drop reference
         psFree(cellNamesIter);
+        psFree(cellNames);
     }
     psFree(componentsIter);