On Tue, Oct 31, 2017 at 09:27:02AM -0500, Mark Michelson wrote:
> The ct_lb action previously assumed that any address arguments were
> IPv4. This patch expands the parsing, formatting, and encoding of ct_lb
> to be amenable to IPv6 addresses as well.
> 
> Signed-off-by: Mark Michelson <[email protected]>

Thanks for working on this.

With this change, I get the following warning from Clang 3.8:

    ../ovn/lib/actions.c:939:32: error: variable 'dst' is used uninitialized 
whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
    ../ovn/lib/actions.c:953:30: note: uninitialized use occurs here
    ../ovn/lib/actions.c:939:28: note: remove the 'if' if its condition is 
always true
    ../ovn/lib/actions.c:932:21: error: variable 'dst' is used uninitialized 
whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
    ../ovn/lib/actions.c:953:30: note: uninitialized use occurs here
    ../ovn/lib/actions.c:932:17: note: remove the 'if' if its condition is 
always true
    ../ovn/lib/actions.c:886:13: note: variable 'dst' is declared here

I can't figure out what it's talking about, and I can't figure out a
simple way to suppress it.  Do you have any idea?
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to