[Gaim-patches] [ gaim-Patches-1223021 ] fixes bug #1222552

[SourceForge.net <no...@so...>]

Patches item #1223021, was opened at 2005-06-18 02:35
Message generated for change (Comment added) made by jroth2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1223021&group_id=235

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Jason Roth (jroth2)
Assigned to: Nobody/Anonymous (nobody)
Summary: fixes bug #1222552

Initial Comment:
As described in bug report #1222552, gaim will
disconnect the current msn session and corrupt
blist.xml when you try to add an msn screenname with a
space in it (e.g. "foo ba...@ho...").  This patch
adds a check to userlist.c in the form of a three line
if statement

if(strrchr(who, " ") == NULL)
{
   gaim_debug("msn", "MSN screennames cannot contain
spaces");
   return;
}

----------------------------------------------------------------------

>Comment By: Jason Roth (jroth2)
Date: 2005-06-21 02:39

Message:
Logged In: YES 
user_id=1298163

yeah, sorry, i misread my man page.  strchr(who, " ") !=
NULL is better.

----------------------------------------------------------------------

Comment By: Richard Laager (rlaager)
Date: 2005-06-19 04:25

Message:
Logged In: YES 
user_id=156487

My man page says: 
       The strchr() and strrchr() functions return a pointer to  the  
matched 
       character or NULL if the character is not found. 
 
I also don't see the point of using strrchr. Wouldn't strchr do? 
 
Therefore, shouldn't this be something like: 
if (strchr(who, " ") != NULL) 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1223021&group_id=235