LGTM, thanks.
Reviewed-by: Yifeng Sun <pkusunyif...@gmail.com>
On Tue, May 12, 2020 at 8:23 AM William Tu <u9012...@gmail.com> wrote:
> Clang reports:
> tests/oss-fuzz/miniflow_target.c:209:26: error: suggest braces around \
> initialization of subobject
> [-Werror,-Wmissing-braces]
> struct flow flow2 = {0};
>
> Fix it by using memset.
>
> Cc: Bhargava Shastry <bshas...@sect.tu-berlin.de>
> Cc: Yifeng Sun <pkusunyif...@gmail.com>
> Signed-off-by: William Tu <u9012...@gmail.com>
> ---
> tests/oss-fuzz/miniflow_target.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/oss-fuzz/miniflow_target.c
> b/tests/oss-fuzz/miniflow_target.c
> index 393443061e85..50b8b0e64237 100644
> --- a/tests/oss-fuzz/miniflow_target.c
> +++ b/tests/oss-fuzz/miniflow_target.c
> @@ -206,8 +206,9 @@ test_minimask_combine(struct flow *flow)
> struct minimask minicombined;
> uint64_t storage[FLOW_U64S];
> } m;
> - struct flow flow2 = {0};
> + struct flow flow2;
>
> + memset(&flow2, 0, sizeof flow2);
> mask.masks = *flow;
> minimask = minimask_create(&mask);
>
> --
> 2.7.4
>
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
