fixed res may not be initialized in read_size

Bernhard Boser · Bernhard Boser · commit c95a94c562bc · 2020-11-12T17:43:40.000-08:00
diff --git a/shared-module/msgpack/__init__.c b/shared-module/msgpack/__init__.c
@@ -90,11 +90,12 @@ STATIC uint32_t read4(msgpack_stream_t *s) {
 }
 
 STATIC size_t read_size(msgpack_stream_t *s, uint8_t len_index) {
-    size_t res;
+    size_t res = 0;
     switch (len_index) {
         case 0:  res = (size_t)read1(s);  break;
         case 1:  res = (size_t)read2(s);  break;
         case 2:  res = (size_t)read4(s);  break;
+        default: mp_raise_ValueError(translate("too big"));
     }
     return res;
 }

Original file line number	Diff line number	Diff line change
`@@ -90,11 +90,12 @@ STATIC uint32_t read4(msgpack_stream_t *s) {`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`STATIC size_t read_size(msgpack_stream_t *s, uint8_t len_index) {`
`93`		`- size_t res;`
	`93`	`+ size_t res = 0;`
`94`	`94`	`switch (len_index) {`
`95`	`95`	`case 0: res = (size_t)read1(s); break;`
`96`	`96`	`case 1: res = (size_t)read2(s); break;`
`97`	`97`	`case 2: res = (size_t)read4(s); break;`
	`98`	`+ default: mp_raise_ValueError(translate("too big"));`
`98`	`99`	`}`
`99`	`100`	`return res;`
`100`	`101`	`}`