minor bug in user ppp?
Brian Somers
brian at Awfulhak.org
Wed Jan 21 13:49:19 PST 2004
Yep, that's a bug... feel free to apply the patch.
Cheers.
On Wed, 21 Jan 2004 13:01:52 -0800, Brooks Davis <brooks at one-eyed-alien.net> wrote:
> I'm reviewing uses of sdl->sdl_data so see if I can make a minor change
> to the way that data is stored and I noticed what I think is a bug in
> the userland ppp implementation. It looks like it's trying to print the
> interface name from the sdl, but the precision specifier is wrong. A
> simpiler printf a little ways above this one appears to be correct. I'm
> I correct that this is a bug? See the patch below.
>
> -- Brooks
>
> Index: usr.sbin/ppp/arp.c
> ===================================================================
> RCS file: /usr/cvs/src/usr.sbin/ppp/arp.c,v
> retrieving revision 1.44
> diff -u -p -r1.44 arp.c
> --- usr.sbin/ppp/arp.c 16 Jan 2002 14:03:51 -0000 1.44
> +++ usr.sbin/ppp/arp.c 21 Jan 2004 20:49:42 -0000
> @@ -302,7 +302,7 @@ arp_EtherAddr(int s, struct in_addr ipad
> if ((ifa->sin_addr.s_addr & netmask->sin_addr.s_addr) ==
> (ipaddr.s_addr & netmask->sin_addr.s_addr)) {
> log_Printf(verbose ? LogPHASE : LogDEBUG,
> - "Found interface %.*s for %s\n", dl->sdl_alen,
> + "Found interface %.*s for %s\n", dl->sdl_nlen,
> dl->sdl_data, inet_ntoa(ipaddr));
> memcpy(hwaddr, dl, dl->sdl_len);
> free(buf);
>
> --
> Any statement of the form "X is the one, true Y" is FALSE.
> PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4
>
--
Brian <brian at Awfulhak.org> <brian@[uk.]FreeBSD.org>
<http://www.Awfulhak.org> <brian@[uk.]OpenBSD.org>
Don't _EVER_ lose your sense of humour !
More information about the freebsd-net
mailing list