diff options
author | Jeffrey Stedfast <fejj@ximian.com> | 2002-01-26 03:08:00 +0800 |
---|---|---|
committer | Jeffrey Stedfast <fejj@src.gnome.org> | 2002-01-26 03:08:00 +0800 |
commit | 91dd1d75d76c31aaab6088482a6a9e5a19777afe (patch) | |
tree | f738aa8ee7e50beae7940b398a6ec621eed1f644 /mail/component-factory.c | |
parent | f1a90dc6dba4b22b5823ecd11ffc486d8f308339 (diff) | |
download | gsoc2013-evolution-91dd1d75d76c31aaab6088482a6a9e5a19777afe.tar.gz gsoc2013-evolution-91dd1d75d76c31aaab6088482a6a9e5a19777afe.tar.zst gsoc2013-evolution-91dd1d75d76c31aaab6088482a6a9e5a19777afe.zip |
Do some NULL checking on the url before using it.
2002-01-25 Jeffrey Stedfast <fejj@ximian.com>
* 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
Diffstat (limited to 'mail/component-factory.c')
-rw-r--r-- | mail/component-factory.c | 9 |
1 files changed, 6 insertions, 3 deletions
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); |