[ovs-dev] [PATCH] ovs-ofctl: fix memory leak reported by valgrind.

Ben Pfaff blp at ovn.org
Thu Jan 21 00:34:41 UTC 2016


On Wed, Jan 20, 2016 at 03:32:58PM -0800, William Tu wrote:
> Testcase 909: ofproto-dpif - patch ports
> The ofpbuf->base is set to OFPBUF_STUB at
>     ofputil_decode_switch_features()->ofpbuf_use_const()
> so ofpbuf_delete does not free it. Call trace is below:
>     vconn_stream_recv(vconn-stream.c:155)
>     do_recv(vconn.c:640)
>     vconn_recv(vconn.c:598)
>     vconn_recv_block(vconn.c:738)
>     vconn_recv_xid__(vconn.c:758)
>     port_iterator_fetch_features(ovs-ofctl.c:846)
>     port_iterator_init(ovs-ofctl.c:889)
>     fetch_ofputil_phy_port(ovs-ofctl.c:975)
> 
> Signed-off-by: William Tu <u9012063 at gmail.com>

Thanks for reporting the problem.  I'd prefer a different solution.  I
posted it:
        http://openvswitch.org/pipermail/dev/2016-January/064792.html



More information about the dev mailing list