# HG changeset patch # User Richard Laager # Date 1159292289 0 # Node ID 776615acf309d32ba02178eb8f93fef07c55f521 # Parent 86e1ebd8ee7fd0db1a2192982f3cc5bf6e0e8222 [gaim-migrate @ 17356] After the comments on gaim-devel about 'bad' code, I looked into the function referenced. It appears that yahoo_buddy_icon_upload_connected() is only called as a callback. It's registered from yahoo_buddy_icon_upload(). yahoo_buddy_icon_upload() is only called twice. In neither case can a NULL make it down to yahoo_buddy_icon_upload_connected(). As this is an exceptional case rather than a normal one, a g_return_if_fail() is more appropriate. That adds logging, makes the intent clearer, and will help the Coverity software detect violations of this assumption. committer: Tailor Script diff -r 86e1ebd8ee7f -r 776615acf309 libgaim/protocols/yahoo/yahoo_picture.c --- a/libgaim/protocols/yahoo/yahoo_picture.c Tue Sep 26 00:28:02 2006 +0000 +++ b/libgaim/protocols/yahoo/yahoo_picture.c Tue Sep 26 17:38:09 2006 +0000 @@ -438,8 +438,7 @@ GaimAccount *account; struct yahoo_data *yd; - if (!d) - return; + g_return_if_fail(d != NULL); gc = d->gc; account = gaim_connection_get_account(gc); @@ -500,6 +499,8 @@ GaimAccount *account = gaim_connection_get_account(gc); struct yahoo_data *yd = gc->proto_data; + g_return_if_fail(d != NULL); + if (yd->jp) { if (gaim_proxy_connect(account, gaim_account_get_string(account, "xferjp_host", YAHOOJP_XFER_HOST), gaim_account_get_int(account, "xfer_port", YAHOO_XFER_PORT),