]> git.proxmox.com Git - mirror_ovs.git/commit
ovs-ofctl: Free leaked minimatch
authorYifeng Sun <pkusunyifeng@gmail.com>
Wed, 11 Sep 2019 21:18:30 +0000 (14:18 -0700)
committerBen Pfaff <blp@ovn.org>
Thu, 19 Sep 2019 16:23:45 +0000 (09:23 -0700)
commit3b6a5b737ae2f8f87320b52a6f5a3cbe1a0b1c0d
treee8951abd43e227305d77c882bb95368fbe9f4918
parentc98eedf9efd1c90799184147271b765475d38a6a
ovs-ofctl: Free leaked minimatch

Valgrind reported:

1056: ofproto - bundle with multiple flow mods (OpenFlow 1.4)

==19220== 160 bytes in 2 blocks are definitely lost in loss record 24 of 34
==19220==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19220==    by 0x4979A4: xmalloc (util.c:138)
==19220==    by 0x42407D: miniflow_alloc (flow.c:3340)
==19220==    by 0x4296CF: minimatch_init (match.c:1758)
==19220==    by 0x46273D: parse_ofp_str__ (ofp-flow.c:1759)
==19220==    by 0x465B9E: parse_ofp_str (ofp-flow.c:1790)
==19220==    by 0x465CE0: parse_ofp_flow_mod_str (ofp-flow.c:1817)
==19220==    by 0x465DF6: parse_ofp_flow_mod_file (ofp-flow.c:1876)
==19220==    by 0x410BA3: ofctl_flow_mod_file.isra.19 (ovs-ofctl.c:1773)
==19220==    by 0x417933: ovs_cmdl_run_command__ (command-line.c:223)
==19220==    by 0x406F68: main (ovs-ofctl.c:179)

This patch fixes it.

Acked-by: William Tu <u9012063@gmail.com>
Signed-off-by: Yifeng Sun <pkusunyifeng@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
utilities/ovs-ofctl.c