On Tue, Jul 21, 2015 at 12:59:12PM -0400, Russell Bryant wrote:
> While running ovn-controller under valgrind, I noticed several
> variations of the same warning. Switching this allocation from malloc
> to zalloc resolves it.
>
> For reference, some examples from valgrind are:
>
> ==8487== Conditional jump or move depends on uninitialised value(s)
> ==8487== at 0x408935: hmap_next_with_hash__ (hmap.h:275)
> ==8487== by 0x4088BF: hmap_first_with_hash (hmap.h:286)
> ==8487== by 0x408357: ovn_flow_lookup (ofctrl.c:307)
> ==8487== by 0x408259: ofctrl_add_flow (ofctrl.c:274)
> ==8487== by 0x40B00F: pipeline_run (pipeline.c:330)
> ==8487== by 0x408F65: main (ovn-controller.c:397)
> ==8487==
> ==8487== Use of uninitialised value of size 8
> ==8487== at 0x4088B3: hmap_first_with_hash (hmap.h:286)
> ==8487== by 0x408357: ovn_flow_lookup (ofctrl.c:307)
> ==8487== by 0x408259: ofctrl_add_flow (ofctrl.c:274)
> ==8487== by 0x40B00F: pipeline_run (pipeline.c:330)
> ==8487== by 0x408F65: main (ovn-controller.c:397)
>
> Signed-off-by: Russell Bryant <rbry...@redhat.com>
I fixed this in my tunnel key series but I forgot to properly note it or
to break it out as a separate patch.
I've now posted it independently:
http://openvswitch.org/pipermail/dev/2015-July/058172.html
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
