Memory leak in PJ_ob_tran.c

[dusanjovic]

There's a small memory leak in PJ_ob_tran.c (in freeup_new):

diff --git a/src/PJ_ob_tran.c b/src/PJ_ob_tran.c
index b0c22f66..eae3ab1d 100644
--- a/src/PJ_ob_tran.c
+++ b/src/PJ_ob_tran.c
@@ -86,7 +86,7 @@ static void *freeup_new (PJ *P) {                       /* Destructor */
         return pj_dealloc (P);
 
     if (P->opaque->link)
-        return pj_dealloc (P->opaque->link);
+        pj_dealloc (P->opaque->link);
 
     pj_dealloc (P->opaque);
     return pj_dealloc(P);

freeup_new must not return before both P->opaque and P are deallocated.

[rouault]

Can you turn your proposed fix into a pull request so it is tested by continuous integration and can be merged easily ?

[rouault]

Fixed per #498
Thanks