(return_files) use proper API for freeing memory

1 files changed, 6 insertions, 4 deletions

diff --git a/URPM.xs b/URPM.xs
index d792539..f23664e 100644
--- a/URPM.xs
+++ b/URPM.xs
@@ -699,9 +699,10 @@ return_files(const Header header, int filter_mode) {
     char **baseNames = td_baseNames.data;
     char **dirNames = td_dirNames.data;
     int32_t *dirIndexes = td_dirIndexes.data;
+    int is_oldfilenames = !baseNames || !dirNames || !dirIndexes;
 
     char **list = NULL;
-    if (!baseNames || !dirNames || !dirIndexes) {
+    if (is_oldfilenames) {
       if (!headerGet(header, RPMTAG_OLDFILENAMES, &td_list, HEADERGET_DEFAULT))
         return;
       list = td_list.data;
@@ -729,9 +730,10 @@ return_files(const Header header, int filter_mode) {
       mXPUSHs(newSVpv(s, len));
     }
 
-    free(baseNames);
-    free(dirNames);
-    free(list);
+    rpmtdFreeData(&td_baseNames);
+    rpmtdFreeData(&td_dirNames);
+    if (is_oldfilenames)
+      rpmtdFreeData(&td_list);
   }
   PUTBACK;
 }