Re: [PATCH] staging/rtl8712: Remove all strcpy() uses in favor of strscpy()

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

 



On Mon, Jul 26, 2021 at 11:11:48AM +0300, Dan Carpenter wrote:
> On Fri, Jul 23, 2021 at 05:15:10PM +0200, Len Baker wrote:
> > Hi,
> >
> > On Wed, Jul 21, 2021 at 11:06:24AM +0300, Dan Carpenter wrote:
> > > On Mon, Jul 19, 2021 at 03:24:38PM +0000, David Laight wrote:
> > > >
> > > > I know only root can set module parameters, but having one
> > > > that contains a string used as a printf format seems
> > > > dangerous at best.
> > > >
> > > > Isn't it best to let userspace rename the interfaces later on?
> > >
> > > Yeah.  I think you're right.
> >
> > Sorry, but I don't understand if the code needs to be improved or not.
> > Is the code shown by Dan not correct?
>
> We should remove the ifname[] array and the module option completely.

Thanks for the clarification.

Regards,
Len





[Index of Archives]     [Linux Driver Development]     [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