On Thu, Jan 17, 2019 at 10:22:12AM -0800, Yifeng Sun wrote:
> In this piece of code, 'struct ofpbuf b' should always point to
> metadata so that metadata can be filled with values through ofpbuf
> operations, like ofpbuf_put_hex and ofpbuf_push_zeros. However,
> ofpbuf_push_zeros may change the data pointer of 'struct ofpbuf b'
> and therefore, metadata will not contain the expected values.
> 
> This patch fixes it by changing ofpbuf_push_zeros to
> ofpbuf_put_zeros.
> 
> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10863
> Signed-off-by: Yifeng Sun <[email protected]>
> ---
> v1->v2: Fix a bug in previous version, thanks Ben!
> v2->v3: Use ofpbuf_put_zeros by previous discussion.

Thanks!  Applied and backported.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to