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

scsi: lpfc: Replace all non-returning strlcpy() with strscpy()

strlcpy() reads the entire source buffer first. This read may exceed the
destination size limit. This is both inefficient and can lead to linear
read overflows if a source string is not NUL-terminated [1]. In an effort
to remove strlcpy() completely [2], replace strlcpy() here with strscpy().
No return values were used, so direct replacement is safe.

[1] https://www.kernel.org/doc/html/latest/process/deprecated.html#strlcpy
[2] https://github.com/KSPP/linux/issues/89

Signed-off-by: Azeem Shaikh <azeemshaikh38@gmail.com>
Link: https://lore.kernel.org/r/20230530155745.343032-1-azeemshaikh38@gmail.com
Reviewed-by: Justin Tee <justin.tee@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

authored by

Azeem Shaikh and committed by
Martin K. Petersen
73be26b1 8f0c17bf

+1 -1
+1 -1
drivers/scsi/lpfc/lpfc_hbadisc.c
··· 4836 4836 }; 4837 4837 4838 4838 if (state < NLP_STE_MAX_STATE && states[state]) 4839 - strlcpy(buffer, states[state], size); 4839 + strscpy(buffer, states[state], size); 4840 4840 else 4841 4841 snprintf(buffer, size, "unknown (%d)", state); 4842 4842 return buffer;