diff options
author | John Mark Bell <jmb@netsurf-browser.org> | 2003-12-28 16:17:31 +0000 |
---|---|---|
committer | John Mark Bell <jmb@netsurf-browser.org> | 2003-12-28 16:17:31 +0000 |
commit | 3a8b8485adc6a0e5e1d8182b64951d077b842093 (patch) | |
tree | 885717e731d48046b5a54583a81fda17fdf7b94e | |
parent | b0b2ec850f74d3dd8bd2fffa76fbd2af20b42705 (diff) | |
download | netsurf-3a8b8485adc6a0e5e1d8182b64951d077b842093.tar.gz netsurf-3a8b8485adc6a0e5e1d8182b64951d077b842093.tar.bz2 |
[project @ 2003-12-28 16:17:31 by jmb]
Fix bug in HTTP auth where trying to fetch a page into a non-existent
browser window caused NetSurf to fall over.
We now check that the window still exists. If not, we ignore the request.
svn path=/import/netsurf/; revision=464
-rw-r--r-- | desktop/browser.c | 3 | ||||
-rw-r--r-- | desktop/gui.h | 1 | ||||
-rw-r--r-- | riscos/window.c | 16 |
3 files changed, 20 insertions, 0 deletions
diff --git a/desktop/browser.c b/desktop/browser.c index 67f43cba2..e6011e2c3 100644 --- a/desktop/browser.c +++ b/desktop/browser.c @@ -209,6 +209,9 @@ void browser_window_open_location_historical(struct browser_window* bw, assert(bw != 0 && url != 0); + /* Check window still exists, if not, don't bother going any further */ + if (!gui_window_in_list(bw->window)) return; + if (bw->url != NULL) browser_window_destroy(bw, false); diff --git a/desktop/gui.h b/desktop/gui.h index 72ead54b4..61b3088b5 100644 --- a/desktop/gui.h +++ b/desktop/gui.h @@ -18,6 +18,7 @@ typedef struct gui_window gui_window; #include <stdbool.h> #include "netsurf/desktop/browser.h" +bool gui_window_in_list(gui_window *g); gui_window *gui_create_browser_window(struct browser_window *bw); gui_window *gui_create_download_window(struct content *content); void gui_window_destroy(gui_window* g); diff --git a/riscos/window.c b/riscos/window.c index 01a8a5b96..ed7dddc5b 100644 --- a/riscos/window.c +++ b/riscos/window.c @@ -12,6 +12,7 @@ */ #include <assert.h> +#include <stdbool.h> #include <string.h> #include "oslib/wimp.h" #include "oslib/wimpspriteop.h" @@ -24,6 +25,21 @@ gui_window *window_list = 0; +/** + * Checks if a window still exists. + */ +bool gui_window_in_list(gui_window *g) { + + gui_window *temp; + + if (g == window_list) return true; + + for(temp=window_list; temp->next != g && temp->next!=0; temp=temp->next) ; + + if (temp->next == NULL) return false; + + return true; +} /** * Create and open a new browser window. |