You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@age.apache.org by jg...@apache.org on 2022/10/17 23:23:33 UTC

[age] branch master updated: Fix delete_global_graphs

This is an automated email from the ASF dual-hosted git repository.

jgemignani pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/age.git


The following commit(s) were added to refs/heads/master by this push:
     new 0aba332  Fix delete_global_graphs
0aba332 is described below

commit 0aba332cd783ee0bcd580186af77b19ab6dc2acf
Author: John Gemignani <jr...@gmail.com>
AuthorDate: Mon Oct 17 15:51:34 2022 -0700

    Fix delete_global_graphs
    
    Fixed the delete_global_graphs function. It was not keeping track of
    the previous graph global context pointer.
    
    This was causing a memory leak for multiple graph contexts if
    individual graphs were deleted.
---
 src/backend/utils/adt/age_global_graph.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/backend/utils/adt/age_global_graph.c b/src/backend/utils/adt/age_global_graph.c
index 3da11b4..c18f9c9 100644
--- a/src/backend/utils/adt/age_global_graph.c
+++ b/src/backend/utils/adt/age_global_graph.c
@@ -858,7 +858,8 @@ static bool delete_specific_GRAPH_global_contexts(char *graph_name)
             return true;
         }
 
-        /* advance to the next one */
+        /* save the current as previous and advance to the next one */
+        prev_ggctx = curr_ggctx;
         curr_ggctx = next_ggctx;
     }