Mercurial > pidgin.yaz
view AUTHORS @ 5531:10920646988f
[gaim-migrate @ 5931]
Ambrose LI (acli) writes:
" This is w.r.t. the 0.64cvs as of around 20:25 EDT.
When gaim starts up, it displays the signon window, and
then immeidately segfaults. The problem is that
tobase64() in util.c uses a size_t (which could very
likely be unsigned) as a second parameter, but proxy.c
calls it with a -1. "
at my questioning the specific change he made:
"Actually the initialization is not the matter (just
something I did while debugging and did not remove, since it
makes sense in any case), it's the other change that matters.
In util.h, the javadoc-like comments states that passing len
= -1 means that tobase64() should calculated len =
strlen(buf). However, since size_t is (or can be) unsigned,
the "if (len < 0)" check never succeeds, so passing
-1
actually results in a very large integer being passed.
So the "if (len < 0)" check actually need to be
changed so
that the passed "-1" will actually get recognized."
so since most of us aren't segfaulting on start, i'm guessing this is a
platform/os specific problem, but that the change shouldn't affect the
rest of us anyway.
committer: Tailor Script <tailor@pidgin.im>
| author | Luke Schierer <lschiere@pidgin.im> |
|---|---|
| date | Mon, 26 May 2003 12:45:05 +0000 |
| parents | f514aa3c01db |
| children | 32ae378082b0 |
line wrap: on
line source
gaim - the Pimpin' Penguin IM Clone that's Good for the Soul ============================================================ We've got an IRC room now too, #gaim on irc.freenode.net. Come check us out. Active Developers: ------------------ Rob Flynn <rob@marko.net> - Maintainer AIM: RobFlynn ICQ: 1358450 MSN: rmf410@hotmail.com Yahoo: IBRRob Jabber: robflynn@jabber.org Sean Egan <bj91704@binghamton.edu> - Lead Developer AIM: SeanEgn ICQ: 96642211 MSN: SeanEgn@hotmail.com Yahoo: SeanEgn Jabber: SeanEgn@jabber.org Christian 'ChipX86' Hammond - Developer & Webmaster Herman 'Hermanator' Bloggs - Win32 Port Nathan 'faceprint' Walp - Developer Mark 'KingAnt' Doliner - Developer Luke 'LSchiere' Schierer - Support Crazy Patch Writers: -------------------- Ben Miller Decklin Foster Etan 'deryni' Reisner Ethan 'Paco-Paco' Blanton Robert 'Robot101' McQueen Retired: -------- Adam Fritzler - Former libfaim Maintainer Eric Warmenhoven <eric@warmenhoven.org> - Former Lead Developer Jim Duchek <jimduchek@ou.edu> - Former Maintainer Jim Seymour - Former Jabber Developer Mark Spencer <markster@marko.net> - Original Author Syd Logan - Hacker and Designated Driver (lazy bum)