Re: af_packet.c bug?

["David S. Miller" <davem@xxxxxxxxxxxxx>]

On Mon, 28 Mar 2005 21:55:57 +0200
Thomas Graf <tgraf@xxxxxxx> wrote:

> * Ben Greear <424858D4.8060604@xxxxxxxxxxxxxxx> 2005-03-28 11:19
> > 
> > What is the '13' doing here?  Maybe it should be IFNAMSIZ?
> > 
> >     /*
> >      *      Find the device first to size check it
> >      */
> > 
> >     saddr->spkt_device[13] = 0;
> >     dev = dev_get_by_name(saddr->spkt_device);
> >     err = -ENODEV;
> >     if (dev == NULL)
> >             goto out_unlock;
> 
> Seems so, please adopt the size of spkt_device in struct sockaddr_pkt
> as well if you change it, it's currently hardcoded as 14.

No can do, that structure is exported to userspace.
Thus, changing it from 14 to IFNAMESZ (which is 16) will
break stuff.