On Thu, Mar 16, 2000 at 05:22:32PM -0500, Sean Porterfield wrote:
> > /* Convert a signed decimal field to a C `int' */
> > static int
> > signed2int (buf, len)
> > Byte *buf;
> > int len;
> > {
> > int v = 0;
> > while (len)
> > {
> > v *= 10;
> > v += buf[0] & 0x0f;
> > len--;
> > if (--len == 0 && (buf[0] & 0xf0) == 0xd0)
> > v = -v;
> > buf++;
> > }
> > return v;
> > }
> >
> > It seems we were checking the zone of a random garbage character (the first
> > character after the end of the field). I've just fixed this in CVS.
> >
> > -Jay 'Eraserhead' Felice
> >
>
> CVS code doesn't have the len--; line in it. When I keyed exactly as you have it
> here, I got a seg fault/core dump when it was "RETR QTEMP/FIELDS"
Eek. The "len--;" is incorrect, apologies.
>
> Using the code from CVS does not cause the crash. I haven't checked for any other
> problems yet.
>
> Thanks for your quick response!
>
> Sean
>
+---
| This is the LINUX5250 Mailing List!
| To submit a new message, send your mail to [EMAIL PROTECTED]
| To subscribe to this list send email to [EMAIL PROTECTED]
| To unsubscribe from this list send email to [EMAIL PROTECTED]
| Questions should be directed to the list owner/operator: [EMAIL PROTECTED]
+---
