[ovs-dev] [PATCH] classifier: Fix use of uninitialized value.

Gregory Rose gvrose8192 at gmail.com
Tue Mar 17 21:50:13 UTC 2020


On 3/17/2020 2:39 PM, William Tu wrote:
> Coverity reports use of uninitialized value of cursor.
> This happens in cls_cursor_start(), when rule is false,
> cursor.subtable is uninitialized. CID 279324.
>
> Signed-off-by: William Tu <u9012063 at gmail.com>
> ---
>   lib/classifier.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/lib/classifier.c b/lib/classifier.c
> index 0fad953213e4..a22f9fb957ea 100644
> --- a/lib/classifier.c
> +++ b/lib/classifier.c
> @@ -1370,6 +1370,7 @@ cls_cursor_start(const struct classifier *cls, const struct cls_rule *target,
>       struct cls_cursor cursor;
>       struct cls_subtable *subtable;
>   
> +    memset(&cursor, 0x0, sizeof cursor);
>       cursor.cls = cls;
>       cursor.target = target && !cls_rule_is_catchall(target) ? target : NULL;
>       cursor.version = version;

Reviewed-by: Greg Rose <gvrose8192 at gmail.com>


More information about the dev mailing list