From a470aacdb2e01b52d59ebe9575dda9e46c00d593 Mon Sep 17 00:00:00 2001 From: John-Mark Bell Date: Sat, 18 Jul 2015 23:29:51 +0100 Subject: RISCOS: fix obvious NULL dereferences in URL complete. --- riscos/gui/url_bar.c | 6 +++++- riscos/url_complete.c | 9 +++++---- 2 files changed, 10 insertions(+), 5 deletions(-) (limited to 'riscos') diff --git a/riscos/gui/url_bar.c b/riscos/gui/url_bar.c index a25ea8748..d910b7592 100644 --- a/riscos/gui/url_bar.c +++ b/riscos/gui/url_bar.c @@ -178,7 +178,11 @@ struct url_bar *ro_gui_url_bar_create(struct theme_descriptor *theme) url_bar->text_size = RO_GUI_MAX_URL_SIZE; url_bar->text_buffer = malloc(url_bar->text_size); - strncpy(url_bar->text_buffer, "", url_bar->text_size); + if (url_bar->text_buffer == NULL) { + free(url_bar); + return NULL; + } + url_bar->text_buffer[0] = 0; url_bar->hidden = false; diff --git a/riscos/url_complete.c b/riscos/url_complete.c index 3d189a01d..ff91b1c18 100644 --- a/riscos/url_complete.c +++ b/riscos/url_complete.c @@ -85,10 +85,11 @@ void ro_gui_url_complete_start(struct toolbar *toolbar) ro_gui_url_complete_close(); url = ro_toolbar_get_url(toolbar); - - url_complete_matched_string = strdup(url); - if (url_complete_matched_string) - url_complete_parent = parent; + if (url != NULL) { + url_complete_matched_string = strdup(url); + if (url_complete_matched_string) + url_complete_parent = parent; + } } -- cgit v1.2.3