diff src/protocols/gg/gg.c @ 12277:739e3aca067a

[gaim-migrate @ 14581] Readding the error checking that I took out in the last commit. I thought about this more today and just confirmed.... It ensures correct behavior if the UI doesn't implement this part of the notify API (and it's not required to). Bartosz Oler agreed that I was right when presented with this case. I'm sorry for the indecision on this. committer: Tailor Script <tailor@pidgin.im>
author Richard Laager <rlaager@wiktel.com>
date Thu, 01 Dec 2005 00:36:19 +0000
parents 2b61f681ff99
children 508512caa22e
line wrap: on
line diff
--- a/src/protocols/gg/gg.c	Wed Nov 30 22:52:44 2005 +0000
+++ b/src/protocols/gg/gg.c	Thu Dec 01 00:36:19 2005 +0000
@@ -1038,6 +1038,16 @@
 				(GaimNotifyCloseCallback)ggp_sr_close_cb,
 				gaim_connection_get_account(gc));
 
+		if (h == NULL) {
+			gaim_debug_error("gg", "ggp_pubdir_reply_handler: "
+					 "Unable to display the search results.\n");
+			gaim_notify_error(gc, NULL,
+					  _("Unable to display the search results."),
+					  NULL);
+			ggp_sr_close_cb(gaim_connection_get_account(gc));
+			return;
+		}
+
 		info->searchresults_window = h;
 	} else {
 		gaim_notify_searchresults_new_rows(gc, results,