Date: Mon, 12 Apr 2010 11:15:45 +0400 From: Hizel Ildar <hizel@vyborg.ru> To: freebsd-current@freebsd.org Subject: Re: ipfw bug on i386 Message-ID: <20100412111545.0a8a3308@hzwork.vyborg.ru> In-Reply-To: <4BC2C0D1.9090700@yandex.ru> References: <4BC1790F.1020205@lissyara.su> <20100412100756.1ef3af73@hzwork.vyborg.ru> <4BC2C0D1.9090700@yandex.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
=D0=92 Mon, 12 Apr 2010 10:42:25 +0400
"Andrey V. Elsukov" <bu7cher@yandex.ru> =D0=BF=D0=B8=D1=88=D0=B5=D1=82:
> On 12.04.2010 10:07, Hizel Ildar wrote:
> > Hey! I'm fix this bug :D
> >
> > patch:
> >
> > foo# diff -ruN main.c~ main.c
> > --- main.c~ 2010-03-04 19:54:56.000000000 +0300
> > +++ main.c 2010-04-12 09:37:21.000000000 +0400
> > @@ -553,7 +553,7 @@
> > }
> >
> > while (fgets(buf, BUFSIZ, f)) { /* read commands */
> > - char linename[10];
> > + char linename[11];
> > char *args[2];
> >
> > lineno++;
>=20
> Can you test your it with 100k lines? :)
> I think it can be fixed with something similar to:
>=20
> - sprintf(linename, "Line %d", lineno);
> + snprintf(linename, sizeof(linename), "Line %d",
> lineno);
>=20
and the variable linename will be incorrect information, it is necessary
to increase the linename to 16 at least
char linename[16]
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100412111545.0a8a3308>