Modify ↓
Opened at 2008-04-01T21:05:31Z
Closed at 2008-04-05T12:36:53Z
#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: |
Description
#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/libevent-1.1a.so.1 #13 0xb7ea6f65 in event_base_loop () from /usr/lib/libevent-1.1a.so.1 #14 0xb7ea6ce5 in event_base_dispatch () from /usr/lib/libevent-1.1a.so.1 #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 yes yes
Attachments (0)
Change History (2)
comment:1 Changed at 2008-04-04T23:28:44Z by
comment:2 Changed at 2008-04-05T12:36:53Z by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed by changeset:devel,377.
Note: See
TracTickets for help on using
tickets.
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).