Gitweb links:

...log 
http://git.netsurf-browser.org/netsurf.git/shortlog/2a9e89de751b6a6600feee2bc3960affb73c6988
...commit 
http://git.netsurf-browser.org/netsurf.git/commit/2a9e89de751b6a6600feee2bc3960affb73c6988
...tree 
http://git.netsurf-browser.org/netsurf.git/tree/2a9e89de751b6a6600feee2bc3960affb73c6988

The branch, master has been updated
       via  2a9e89de751b6a6600feee2bc3960affb73c6988 (commit)
      from  54e31b65d14dd1e228106eae0c60ab42802a97ed (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commitdiff 
http://git.netsurf-browser.org/netsurf.git/commit/?id=2a9e89de751b6a6600feee2bc3960affb73c6988
commit 2a9e89de751b6a6600feee2bc3960affb73c6988
Author: Chris Young <[email protected]>
Commit: Chris Young <[email protected]>

    Fix potential memory leak

diff --git a/frontends/amiga/tree.c b/frontends/amiga/tree.c
index 9049709..79753a7 100644
--- a/frontends/amiga/tree.c
+++ b/frontends/amiga/tree.c
@@ -234,6 +234,7 @@ static void ami_tree_redraw_req_dr(void *p)
 
        if(ami_gui_get_space_box(twin->objects[GID_BROWSER], &bbox) != 
NSERROR_OK) {
                amiga_warn_user("NoMemory", "");
+               free(atrr_data);
                return;
        }
 
@@ -282,6 +283,8 @@ static void ami_tree_redraw_req(void *p)
                .plot = &amiplot
        };
 
+       free(atrr_data);
+
        if(!twin->win) return;
 
        ami_update_pointer(twin->win, GUI_POINTER_WAIT);
@@ -341,7 +344,6 @@ static void ami_tree_redraw_req(void *p)
                }
        }
 
-       free(atrr_data);
        ami_gui_free_space_box(bbox);
        ami_update_pointer(twin->win, GUI_POINTER_DEFAULT);
        ami_clearclipreg(glob);


-----------------------------------------------------------------------

Summary of changes:
 frontends/amiga/tree.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/frontends/amiga/tree.c b/frontends/amiga/tree.c
index 9049709..79753a7 100644
--- a/frontends/amiga/tree.c
+++ b/frontends/amiga/tree.c
@@ -234,6 +234,7 @@ static void ami_tree_redraw_req_dr(void *p)
 
        if(ami_gui_get_space_box(twin->objects[GID_BROWSER], &bbox) != 
NSERROR_OK) {
                amiga_warn_user("NoMemory", "");
+               free(atrr_data);
                return;
        }
 
@@ -282,6 +283,8 @@ static void ami_tree_redraw_req(void *p)
                .plot = &amiplot
        };
 
+       free(atrr_data);
+
        if(!twin->win) return;
 
        ami_update_pointer(twin->win, GUI_POINTER_WAIT);
@@ -341,7 +344,6 @@ static void ami_tree_redraw_req(void *p)
                }
        }
 
-       free(atrr_data);
        ami_gui_free_space_box(bbox);
        ami_update_pointer(twin->win, GUI_POINTER_DEFAULT);
        ami_clearclipreg(glob);


-- 
NetSurf Browser

_______________________________________________
netsurf-commits mailing list
[email protected]
http://listmaster.pepperfish.net/cgi-bin/mailman/listinfo/netsurf-commits-netsurf-browser.org

Reply via email to