Re: [patch] Staging: otus: fix strcpy() overflow

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 07, 2010 at 02:18:50PM -0700, Dan Carpenter wrote:
> wrq->name is only 16 characters long but "IEEE 802.11-MIMO" is 16
> characters + a NULL character, so it's too long.  I changed the name to
> just "802.11-MIMO".
> 
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
> 
> diff --git a/drivers/staging/otus/ioctl.c b/drivers/staging/otus/ioctl.c
> index a48c8e4..3887932 100644
> --- a/drivers/staging/otus/ioctl.c
> +++ b/drivers/staging/otus/ioctl.c
> @@ -507,7 +507,7 @@ int usbdrvwext_giwname(struct net_device *dev,
>  {
>  	/* struct usbdrv_private *macp = dev->ml_priv; */
>  
> -	strcpy(wrq->name, "IEEE 802.11-MIMO");
> +	strcpy(wrq->name, "802.11-MIMO");

Nack, use instead:

IEEE 802.11abgn

  Luis
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux