Changeset 911d97a


Ignore:
Timestamp:
2011-08-04T15:19:54Z (9 years ago)
Author:
Wilmer van der Gaast <wilmer@…>
Branches:
master
Children:
7732193
Parents:
f988ad3
Message:

Error handling fixes.

Found one double free() bug causing troubles when a buddy_msg() handler takes
down the IM connection immediately.

Files:
3 edited

Legend:

Unmodified
Added
Removed
  • irc_im.c

    rf988ad3 r911d97a  
    451451{
    452452        irc_user_t *iu = data;
    453         char *msg = g_string_free( iu->pastebuf, FALSE );
     453        char *msg;
    454454        GSList *l;
     455       
     456        msg = g_string_free( iu->pastebuf, FALSE );
     457        iu->pastebuf = NULL;
     458        iu->pastebuf_timer = 0;
    455459       
    456460        for( l = irc_plugins; l; l = l->next )
     
    479483       
    480484        g_free( msg );
    481         iu->pastebuf = NULL;
    482         iu->pastebuf_timer = 0;
    483485       
    484486        return FALSE;
  • protocols/jabber/jabber.c

    rf988ad3 r911d97a  
    319319                        imcb_error( ic, "OAuth failure" );
    320320                        imc_logout( ic, TRUE );
     321                        return 0;
    321322                }
    322323        }
  • protocols/jabber/sasl.c

    rf988ad3 r911d97a  
    448448{
    449449        struct im_connection *ic = data;
    450         imcb_remove_buddy( ic, "jabber_oauth", NULL );
     450        if( g_slist_find( jabber_connections, ic ) )
     451                imcb_remove_buddy( ic, "jabber_oauth", NULL );
    451452        return FALSE;
    452453}
Note: See TracChangeset for help on using the changeset viewer.