stats: fixed implicit integer sign extension in stats_get_off()

nvlsianpu · nashif · commit 87c1f9a6e4f9 · 2021-10-28T15:15:56.000-04:00
Introduced explicit casting for fixing static analyze issue. fixes #39812 Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
diff --git a/subsys/stats/stats.c b/subsys/stats/stats.c
@@ -43,7 +43,7 @@ stats_get_name(const struct stats_hdr *hdr, int idx)
 static uint16_t
 stats_get_off(const struct stats_hdr *hdr, int idx)
 {
-	return sizeof(*hdr) + idx * hdr->s_size;
+	return (uint16_t) (sizeof(*hdr) + idx * (int) hdr->s_size);
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ stats_get_name(const struct stats_hdr *hdr, int idx)`
`43`	`43`	`static uint16_t`
`44`	`44`	`stats_get_off(const struct stats_hdr *hdr, int idx)`
`45`	`45`	`{`
`46`		`- return sizeof(hdr) + idx hdr->s_size;`
	`46`	`+ return (uint16_t) (sizeof(hdr) + idx (int) hdr->s_size);`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`/**`