[Pidgin] #1989: Segmentation fault in jabber protocol plugin

[Pidgin]

#1989: Segmentation fault in jabber protocol plugin
------------------------+---------------------------------------------------
  Reporter:  snakeru    |       Owner:  seanegan
      Type:  defect     |      Status:  assigned
  Priority:  minor      |   Milestone:  2.1.1   
 Component:  libpurple  |     Version:  2.0.2   
Resolution:             |    Keywords:          
   Pending:  0          |  
------------------------+---------------------------------------------------
Comment (by snakeru):

 Here is yahoo crash:
 {{{
 #0  0xb517f95b in yahoo_string_encode (gc=0xdb77da8,
     str=0xe3893f8 "toluojosh, this is my IM Page on SNIMMER. You can chat
 with me there: http://kmc_guttikonda2002.snimmer.com/", utf8=0xbff5e1f4)
 at util.c:53
 53              if (yd->jp && utf8 && *utf8)
 (gdb) #0  0xb517f95b in yahoo_string_encode (gc=0xdb77da8,
     str=0xe3893f8 "toluojosh, this is my IM Page on SNIMMER. You can chat
 with me there: http://kmc_guttikonda2002.snimmer.com/", utf8=0xbff5e1f4)
 at util.c:53
 #1  0xb5182425 in yahoo_send_im (gc=0xdb77da8, who=0xb4f7d69c "toluojosh",
     what=0xb502e694 "toluojosh, this is my IM Page on SNIMMER. You can
 chat with me there: http://kmc_guttikonda2002.snimmer.com/", flags=0) at
 yahoo.c:3245
 #2  0xb549d57b in serv_send_im (gc=0xdb77da8, name=0xb4f7d69c "toluojosh",
     message=0xb502e694 "toluojosh, this is my IM Page on SNIMMER. You can
 chat with me there: http://kmc_guttikonda2002.snimmer.com/", flags=0) at
 server.c:135
 #3  0xb54f6f9b in pygaim_server_serv_send_im (self=0x0, args=0xb57cc554)
     at generated.c:5632
 #4  0xb58bf129 in ?? ()
 #5  0x00000000 in ?? ()
 }}}
 ICQ crash core file already deleted, sorry.
 Here is the patch that I use to fight it:
 {{{
 --- gaim-2.0.0beta6/libgaim/server.c    2007-01-18 20:28:24.000000000
 -0800
 +++ gaim-2.0.0beta6-my/libgaim/server.c 2007-07-02 07:28:30.000000000
 -0700
 @@ -123,6 +123,7 @@

         g_return_val_if_fail(gc != NULL, val);
         g_return_val_if_fail(gc->prpl != NULL, val);
 +       g_return_val_if_fail(gc->proto_data != NULL, val);

         prpl_info = GAIM_PLUGIN_PROTOCOL_INFO(gc->prpl);
 }}}
 This patch is wrong because it makes it returns -EINVAL while it should
 use some other value. But otherwise it saves me from several crashes a
 day.
 These messages are sent not on signoff but immidiatedly after receiving
 signed-on. They are sent in big amounts, may be sometimes gaim sends
 signed-off and I am missing it and continue sending messages (they are
 sent with slight delays b/w each other).

-- 
Ticket URL: <http://developer.pidgin.im/ticket/1989#comment:7>
Pidgin <http://pidgin.im>
Pidgin