Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

hamradio: baycom: replace strcpy() with strscpy()

The strcpy() function has been deprecated and replaced with strscpy().
There is an effort to make this change treewide:
https://github.com/KSPP/linux/issues/88.

Signed-off-by: Ethan Carter Edwards <ethan@ethancedwards.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://patch.msgid.link/3qo3fbrak7undfgocsi2s74v4uyjbylpdqhie4dohfoh4welfn@joq7up65ug6v
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

authored by

Ethan Carter Edwards and committed by
Jakub Kicinski
3b147be9 b341f6fd

+5 -5
+2 -2
drivers/net/hamradio/baycom_par.c
··· 427 427 break; 428 428 429 429 case HDLCDRVCTL_GETMODE: 430 - strcpy(hi->data.modename, bc->options ? "par96" : "picpar"); 430 + strscpy(hi->data.modename, bc->options ? "par96" : "picpar"); 431 431 if (copy_to_user(data, hi, sizeof(struct hdlcdrv_ioctl))) 432 432 return -EFAULT; 433 433 return 0; ··· 439 439 return baycom_setmode(bc, hi->data.modename); 440 440 441 441 case HDLCDRVCTL_MODELIST: 442 - strcpy(hi->data.modename, "par96,picpar"); 442 + strscpy(hi->data.modename, "par96,picpar"); 443 443 if (copy_to_user(data, hi, sizeof(struct hdlcdrv_ioctl))) 444 444 return -EFAULT; 445 445 return 0;
+1 -1
drivers/net/hamradio/baycom_ser_fdx.c
··· 531 531 return baycom_setmode(bc, hi->data.modename); 532 532 533 533 case HDLCDRVCTL_MODELIST: 534 - strcpy(hi->data.modename, "ser12,ser3,ser24"); 534 + strscpy(hi->data.modename, "ser12,ser3,ser24"); 535 535 if (copy_to_user(data, hi, sizeof(struct hdlcdrv_ioctl))) 536 536 return -EFAULT; 537 537 return 0;
+2 -2
drivers/net/hamradio/baycom_ser_hdx.c
··· 570 570 break; 571 571 572 572 case HDLCDRVCTL_GETMODE: 573 - strcpy(hi->data.modename, "ser12"); 573 + strscpy(hi->data.modename, "ser12"); 574 574 if (bc->opt_dcd <= 0) 575 575 strcat(hi->data.modename, (!bc->opt_dcd) ? "*" : (bc->opt_dcd == -2) ? "@" : "+"); 576 576 if (copy_to_user(data, hi, sizeof(struct hdlcdrv_ioctl))) ··· 584 584 return baycom_setmode(bc, hi->data.modename); 585 585 586 586 case HDLCDRVCTL_MODELIST: 587 - strcpy(hi->data.modename, "ser12"); 587 + strscpy(hi->data.modename, "ser12"); 588 588 if (copy_to_user(data, hi, sizeof(struct hdlcdrv_ioctl))) 589 589 return -EFAULT; 590 590 return 0;