gnuastro-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[gnuastro-commits] master 823162f1 2/3: Library (wcs.c): fix memory leak


From: Mohammad Akhlaghi
Subject: [gnuastro-commits] master 823162f1 2/3: Library (wcs.c): fix memory leak
Date: Thu, 29 Sep 2022 12:28:00 -0400 (EDT)

branch: master
commit 823162f127b6064c85ece3fc9aceb0d373d43500
Author: Pedram Ashofteh Ardakani <pedramardakani@pm.me>
Commit: Mohammad Akhlaghi <mohammad@akhlaghi.org>

    Library (wcs.c): fix memory leak
    
    Until now, the 'wcs' structure was freed but the pointer itself still
    remained. Also, the 'coords' and its links were not freed.
    
    With this commit, both of these structures and their pointers are
    freed. Checking with valgrind, now using 'gal_wcs_coverage' is leak-free.
---
 lib/wcs.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/wcs.c b/lib/wcs.c
index 47be5227..ba83d6b5 100644
--- a/lib/wcs.c
+++ b/lib/wcs.c
@@ -2155,7 +2155,8 @@ gal_wcs_coverage(char *filename, char *hdu, size_t *ondim,
 
   /* Clean up and return success. */
   free(dsize);
-  wcsfree(wcs);
+  wcsfree(wcs); free(wcs);
+  gal_list_data_free(coords);
   return 1;
 }
 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]