Re: [PATCH 0/3] staging: rtl8723bs: Replace sprintf with scnprintf

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

 



On Mon, Mar 01, 2021 at 02:45:00PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 3/1/21 2:12 PM, Candy Febriyanto wrote:
> > This patchset replaces most calls to sprintf with scnprintf, thereby
> > preventing potential buffer overflows. The rest I left alone because
> > they write to a buffer passed by a caller that doesn't pass its size
> > alongside it.
> > 
> > Candy Febriyanto (3):
> >   staging: rtl8723bs: core: Replace sprintf with scnprintf
> >   staging: rtl8723bs: hal: Replace sprintf with scnprintf
> >   staging: rtl8723bs: os_dep: Replace sprintf with scnprintf
> > 
> >  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  3 +-
> >  drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |  4 +-
> >  drivers/staging/rtl8723bs/hal/hal_com.c       | 45 ++++++++++---------
> >  .../staging/rtl8723bs/os_dep/ioctl_linux.c    | 20 +++++----
> >  drivers/staging/rtl8723bs/os_dep/mlme_linux.c |  6 +--
> >  5 files changed, 41 insertions(+), 37 deletions(-)
> 
> Thanks, the entire series looks good to me, for the series:
> 
> Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> 
> Regards,
> 
> Hans
> 

Thank you Hans.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-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