From 91dd1d75d76c31aaab6088482a6a9e5a19777afe Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Fri, 25 Jan 2002 19:08:00 +0000 Subject: Do some NULL checking on the url before using it. 2002-01-25 Jeffrey Stedfast * component-factory.c (destination_folder_handle_motion): Do some NULL checking on the url before using it. (destination_folder_handle_drop): Make sure the uri is non-NULL before freeing. svn path=/trunk/; revision=15473 --- mail/component-factory.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'mail/component-factory.c') diff --git a/mail/component-factory.c b/mail/component-factory.c index 801e6f2971..cfd4478875 100644 --- a/mail/component-factory.c +++ b/mail/component-factory.c @@ -470,7 +470,7 @@ destination_folder_handle_motion (EvolutionShellComponentDndDestinationFolder *f CamelURL *url; url = camel_url_new (physical_uri, NULL); - noselect = camel_url_get_param (url, "noselect"); + noselect = url ? camel_url_get_param (url, "noselect") : NULL; if (noselect && !g_strcasecmp (noselect, "yes")) /* uh, no way to say "illegal" */ @@ -478,7 +478,8 @@ destination_folder_handle_motion (EvolutionShellComponentDndDestinationFolder *f else *suggested_action_return = GNOME_Evolution_ShellComponentDnd_ACTION_MOVE; - camel_url_free (url); + if (url) + camel_url_free (url); return TRUE; } @@ -549,7 +550,9 @@ destination_folder_handle_drop (EvolutionShellComponentDndDestinationFolder *des camel_url_free (uri); return FALSE; } - camel_url_free (uri); + + if (uri) + camel_url_free (uri); g_print ("in destination_folder_handle_drop (%s)\n", physical_uri); -- cgit