Re: QPOPPER problem....

From: Marco S Hyman (marcat_private)
Date: Sat Jun 27 1998 - 15:40:11 PDT

Next message: Bryan (Zeli) Kartzman: "Re: QPOPPER problem.... ONE crude patch..."

Previous message: Miquel van Smoorenburg: "patch: qpopper (plugs another hole too)"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

"Bruno Lopes F. Cabral" writes:
 >  #ifdef HAVE_VPRINTF
 > -        vsprintf(mp,format,ap);
 > +        vsnprintf(mp,sizeof(message) - strlen(mp) -1,format,ap);

I don't think so.  Check out the code... mp is pointing at the terminating
null from the previous sprintf.  In effect you've coded
sizeof( message ) - -1.  Not what you wanted at all.

// marc

Next message: Bryan (Zeli) Kartzman: "Re: QPOPPER problem.... ONE crude patch..."
Previous message: Miquel van Smoorenburg: "patch: qpopper (plugs another hole too)"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b30 : Fri Apr 13 2001 - 14:00:04 PDT