#386 closed defect (fixed)

Crash in "account del" confirmation code

Reported by: wilmer Owned by:
Priority: normal Milestone: 1.2.1
Component: BitlBee Version: 1.2
Keywords: Cc:
IRC client+version: Client-independent Operating System: Public server
OS version/distro:


#0  0x08053a6a in account_del (irc=0x0, acc=0x80cd298) at account.c:184
184		for( a = irc->accounts; a; a = (l=a)->next )
(gdb) bt
#0  0x08053a6a in account_del (irc=0x0, acc=0x80cd298) at account.c:184
#1  0x0805d570 in cmd_account_del_yes (w=0x0, data=0x80cd298) at root_commands.c:217
#2  0x0805cdc1 in query_answer (irc=0x80cc170, q=0x80de920, ans=1) at query.c:146
#3  0x0805ea8c in cmd_yesno (irc=0x80cc170, cmd=0xbfe32d74) at root_commands.c:770
#4  0x0805d0f5 in root_command (irc=0x80cc170, cmd=0xbfe32d74) at root_commands.c:95
#5  0x0805d05a in root_command_string (irc=0x80cc170, u=0x80dea18, command=0x80de89a "yes", flags=0)
    at root_commands.c:77
#6  0x080598d5 in irc_send (irc=0x80cc170, nick=0x80c53f0 "root", s=0x80de89a "yes", flags=0) at irc.c:1090
#7  0x0805ad3e in irc_cmd_privmsg (irc=0x80cc170, cmd=0x80de950) at irc_commands.c:288
#8  0x0805bdfe in irc_exec (irc=0x80cc170, cmd=0x80de950) at irc_commands.c:673
#9  0x08057ab7 in irc_process (irc=0x80cc170) at irc.c:395
#10 0x08054121 in bitlbee_io_current_client_read (data=0x80cc170, fd=0, cond=GAIM_INPUT_READ) at bitlbee.c:185
#11 0x0806331f in b_event_passthrough (fd=0, event=2, data=0x80cc200) at events_libevent.c:143
#12 0xb7ea6c79 in event_base_priority_init () from /usr/lib/
#13 0xb7ea6f65 in event_base_loop () from /usr/lib/
#14 0xb7ea6ce5 in event_base_dispatch () from /usr/lib/
#15 0x0806324d in b_main_run () at events_libevent.c:83
#16 0x08061d4f in main (argc=1, argv=0xbfe33414, envp=0xbfe3341c) at unix.c:129

My guess at steps to reproduce:

account del 0
account del 0

Attachments (0)

Change History (2)

comment:1 Changed at 2008-04-04T23:28:44Z by wilmer

It may make sense to just change the query callback function prototype. Right now it still passes a pointer "w" (comes from Gaim, it's a window handle or something like that) and a data pointer, while it could also get a pointer to the query structure, which will contain more useful information (a reliable irc_t pointer, which is where things go wrong right now).

comment:2 Changed at 2008-04-05T12:36:53Z by wilmer

Resolution: fixed
Status: newclosed

Modify Ticket

as closed The ticket will remain with no owner.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.