video: gc2145: fix set_fmt()

facchinm · mmahadevan108 · commit 054d60fb1c39 · 2024-11-22T13:04:22.000-06:00
Trivial fix for bound check

Signed-off-by: Martino Facchin &lt;m.facchin@arduino.cc&gt;
diff --git a/drivers/video/gc2145.c b/drivers/video/gc2145.c
@@ -1039,7 +1039,7 @@ static int gc2145_set_fmt(const struct device *dev, enum video_endpoint_id ep,
 	}
 
 	/* Check if camera is capable of handling given format */
-	for (int i = 0; i == ARRAY_SIZE(fmts); i++) {
+	for (int i = 0; i < RESOLUTIONS_MAX; i++) {
 		if (fmts[i].width_min == fmt->width && fmts[i].height_min == fmt->height &&
 		    fmts[i].pixelformat == fmt->pixelformat) {
 			res = (enum resolutions)i;

Original file line number	Diff line number	Diff line change
`@@ -1039,7 +1039,7 @@ static int gc2145_set_fmt(const struct device *dev, enum video_endpoint_id ep,`
`1039`	`1039`	`}`
`1040`	`1040`
`1041`	`1041`	`/* Check if camera is capable of handling given format */`
`1042`		`- for (int i = 0; i == ARRAY_SIZE(fmts); i++) {`
	`1042`	`+ for (int i = 0; i < RESOLUTIONS_MAX; i++) {`
`1043`	`1043`	`if (fmts[i].width_min == fmt->width && fmts[i].height_min == fmt->height &&`
`1044`	`1044`	`fmts[i].pixelformat == fmt->pixelformat) {`
`1045`	`1045`	`res = (enum resolutions)i;`