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