scsi: st: Fix array overflow in st_setup()

kmakisara · gregkh · commit e4d1ca0a84a6 · 2025-04-25T10:45:16.000+02:00
[ Upstream commit a018d1c ] Change the array size to follow parms size instead of a fixed value. Reported-by: Chenyuan Yang <chenyuan0y@gmail.com> Closes: https://lore.kernel.org/linux-scsi/CALGdzuoubbra4xKOJcsyThdk5Y1BrAmZs==wbqjbkAgmKS39Aw@mail.gmail.com/ Signed-off-by: Kai Mäkisara <Kai.Makisara@kolumbus.fi> Link: https://lore.kernel.org/r/20250311112516.5548-2-Kai.Makisara@kolumbus.fi Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
diff --git a/drivers/scsi/st.c b/drivers/scsi/st.c
@@ -4120,7 +4120,7 @@ static void validate_options(void)
  */
 static int __init st_setup(char *str)
 {
-	int i, len, ints[5];
+	int i, len, ints[ARRAY_SIZE(parms) + 1];
 	char *stp;
 
 	stp = get_options(str, ARRAY_SIZE(ints), ints);

Original file line number	Diff line number	Diff line change
`@@ -4120,7 +4120,7 @@ static void validate_options(void)`
`4120`	`4120`	`*/`
`4121`	`4121`	`static int __init st_setup(char *str)`
`4122`	`4122`	`{`
`4123`		`- int i, len, ints[5];`
	`4123`	`+ int i, len, ints[ARRAY_SIZE(parms) + 1];`
`4124`	`4124`	`char *stp;`
`4125`	`4125`
`4126`	`4126`	`stp = get_options(str, ARRAY_SIZE(ints), ints);`