On Thu, May 15, 2025 at 12:22 AM Alex Elder <elder@xxxxxxxx> wrote: > > On 5/14/25 1:20 PM, Siddarth Gundu wrote: > > strcpy() is deprecated; use strscpy() instead. > > > > Both the destination and source buffer are of fixed length > > so strscpy with 2-arguments is used. > > > > Link: https://github.com/KSPP/linux/issues/88 > > Signed-off-by: Siddarth Gundu <siddarthsgml@xxxxxxxxx> > > --- > > drivers/block/rbd.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c > > index faafd7ff43d6..92b38972db1c 100644 > > --- a/drivers/block/rbd.c > > +++ b/drivers/block/rbd.c > > @@ -39,6 +39,7 @@ > > > > #include <linux/kernel.h> > > #include <linux/device.h> > > +#include <linux/string.h> > > #include <linux/module.h> > > #include <linux/blk-mq.h> > > #include <linux/fs.h> > > @@ -3654,7 +3655,7 @@ static void __rbd_lock(struct rbd_device *rbd_dev, const char *cookie) > > Could the cookie argument possibly be defined with > its size? I.e.: > __rbd_lock(struct rbd_device *rbd_dev, const char cookie[32]) > > I see all the callers pass an array that's 32 characters, > but the function argument doesn't guarantee that. > > You could also abstract the cookie with a typedef and > operations on it. I see, I will send a v2 patch soon creating a typedef local to rbd.c Thanks a lot for the review. -- With Gratitude Siddarth
