Răsfoiți Sursa

isdn: icn: use strlcpy() when parsing setup options

If you pass an invalid string here then you probably deserve the memory
corruption, but it annoys static analysis tools so lets fix it.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Dan Carpenter 10 ani în urmă
părinte
comite
10640d3455
1 a modificat fișierele cu 1 adăugiri și 1 ștergeri
  1. 1 1
      drivers/isdn/icn/icn.c

+ 1 - 1
drivers/isdn/icn/icn.c

@@ -1609,7 +1609,7 @@ icn_setup(char *line)
 	if (ints[0] > 1)
 	if (ints[0] > 1)
 		membase = (unsigned long)ints[2];
 		membase = (unsigned long)ints[2];
 	if (str && *str) {
 	if (str && *str) {
-		strcpy(sid, str);
+		strlcpy(sid, str, sizeof(sid));
 		icn_id = sid;
 		icn_id = sid;
 		if ((p = strchr(sid, ','))) {
 		if ((p = strchr(sid, ','))) {
 			*p++ = 0;
 			*p++ = 0;