net: coap: fix uninitialized usage of hdr_len

jakubwitowski · fabiobaltieri · commit 828f5e8b2334 · 2025-02-11T11:45:05.000Z
The hdr_len was uninitialized when none of if-else
conditions were met in decode_delta() function.

Signed-off-by: Jakub Witowski &lt;jakub.witowski9302@gmail.com&gt;
diff --git a/subsys/net/lib/coap/coap.c b/subsys/net/lib/coap/coap.c
@@ -498,6 +498,7 @@ static int decode_delta(uint8_t *data, uint16_t offset, uint16_t *pos, uint16_t
 			uint16_t opt, uint16_t *opt_ext, uint16_t *hdr_len)
 {
 	int ret = 0;
+	*hdr_len = 0;
 
 	if (opt == COAP_OPTION_EXT_13) {
 		uint8_t val;

Original file line number	Diff line number	Diff line change
`@@ -498,6 +498,7 @@ static int decode_delta(uint8_t data, uint16_t offset, uint16_t pos, uint16_t`
`498`	`498`	`uint16_t opt, uint16_t opt_ext, uint16_t hdr_len)`
`499`	`499`	`{`
`500`	`500`	`int ret = 0;`
	`501`	`+ *hdr_len = 0;`
`501`	`502`
`502`	`503`	`if (opt == COAP_OPTION_EXT_13) {`
`503`	`504`	`uint8_t val;`