vici: Prevent uninitialized memory access when finding VICI_END in message
VICI_END (7) shouldn't be encoded in a message. However, if we encounter it, we should at least set `out` accordingly so callers can abort the enumeration. By not doing so previously and returning TRUE, callers might access the possibly uninitialized name/value arguments passed to the enumerator.
This commit is contained in:
@@ -189,7 +189,7 @@ METHOD(enumerator_t, parse_enumerate, bool,
|
|||||||
this->list = FALSE;
|
this->list = FALSE;
|
||||||
break;
|
break;
|
||||||
case VICI_END:
|
case VICI_END:
|
||||||
return TRUE;
|
break;
|
||||||
default:
|
default:
|
||||||
DBG1(DBG_ENC, "unknown encoding type: %u", type);
|
DBG1(DBG_ENC, "unknown encoding type: %u", type);
|
||||||
return FALSE;
|
return FALSE;
|
||||||
|
|||||||
Reference in New Issue
Block a user