Hi Josh,

On wo, 2016-06-15 at 09:03 -0500, Josh Poimboeuf wrote:

I agree it looks like a false positive, though the code is a bit
convoluted, so I'm not surprised that gcc might get confused.

Agree.

How about initializing opt to 0?

I'm unconvinced. Especially since this only shows up in 64 bits builds.

 static inline int ip_vs_proc_sync_conn(struct netns_ipvs *ipvs, __u8

-	struct ip_vs_sync_conn_options opt;
+	struct ip_vs_sync_conn_options opt = {0};

You forgot to append
                                                  /* I give up */

here. More common would be to use
                                                  /* silence GCC */
Thanks,


Paul Bolle