[PATCH] Fix bug in "manual" GNOME proxy handling
Michael Ellerman
michael at ellerman.id.au
Tue Mar 4 23:48:55 EST 2008
On Tue, 2008-03-04 at 20:22 -0800, Jeff Connelly wrote:
> 2008/3/4 Michael Ellerman <michael at ellerman.id.au>:
> > In libpurple/proxy.c, purple_gnome_proxy_get_info() uses gconftool to
> > get the GNOME proxy settings. It explicitly looks for proxy type =
> > "none", in which case no proxy is used, and also proxy type = "manual"
> > in which case the global pidgin proxy is used.
> >
> > However the check for "manual" is missing a ! on the strcmp(), so it
> > incorrectly falls through.
> For what it is worth, I've used g_str_equal() instead of strcmp()
> throughout the MySpaceIM protocol plugin to avoid this particular
> mistake.
>
> g_str_equal() was intended to be used with glib to compare strings
> within hash tables, but it works pretty well as a general-purpose
> function, and has saner return values.
Yeah, strcmp() sucks. People get it wrong all the time. I didn't know
about g_str_equal() - if the Pidgin cabal prefer that it would be a good
change IMHO.
cheers
ps. apologies if the patch is word-wrapped, I think I forgot to turn off
encryption.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://pidgin.im/pipermail/devel/attachments/20080305/3580fdf2/attachment.sig>
More information about the Devel
mailing list