nmlorg / naim

naim is a console client for AOL Instant Messenger (AIM), AOL I Seek You (ICQ), Internet Relay Chat (IRC), and The lily CMC.
http://naim.n.ml.org/
1 stars 0 forks source link

Invisible AOL users crash naim #4

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Sign into AIM with NAIM
2. Sign into AIM with another username on a client that supports invisible 
mode.
3. Message the user on NAIM.  The notification that a user is typing a 
message will trigger an assert.

What is the expected output? What do you see instead?

naim: firetalk.c:838: firetalk_callback_typing: Assertion `buddyiter-
>online != 0' failed.
           Aborted

What version of the product are you using? On what operating system?
naim 0.11.8.2.1
FC6

Please provide any additional information below.
I had someone try this out earlier today and it crashed.  I am attempting 
to reproduce it, but for some reason the GAIM versions I have or the AIM 
servers are not sending messages that the user is typing.

Original issue reported on code.google.com by rrt...@gmail.com on 16 May 2007 at 10:16

GoogleCodeExporter commented 9 years ago
Confirmed, messages must be sent both ways for GAIM to send a 'user is typing' 
message.  Other IM clients may work without initiating a conversation first 
(someone 
did it to me twice this morning)  This is reproducable 100% of the time.

Original comment by rrt...@gmail.com on 16 May 2007 at 10:32

GoogleCodeExporter commented 9 years ago
Try this patch.

Incidentally, if you come up with other bugs, now might be a good time to speak,
because this may warrant 0.11.8.2.2.

Original comment by joshua.w...@gmail.com on 16 May 2007 at 10:42

Attachments:

GoogleCodeExporter commented 9 years ago
gah, "Fixed" makes it go away. Changed to "started" so 
users can
verify. 

Original comment by joshua.w...@gmail.com on 16 May 2007 at 10:43

GoogleCodeExporter commented 9 years ago
line 742 as well, maybe surrounding lines.  I don't know if AIM supports 
warning 
anymore, so that line might be invalid anyway, otherwise it could cause the 
same 
issue.  This invisibility stuff could cause even more problems besides what I 
found.

Original comment by rrt...@gmail.com on 19 May 2007 at 12:06

GoogleCodeExporter commented 9 years ago
This was "A debugging assertion that caused random crashes has been removed." 
in the
0.11.8.3.2 release notes.

Original comment by nml...@gmail.com on 21 Jul 2009 at 5:59