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).