[ovs-dev] [PATCH] ofproto-dpif: Clone packet-ins in handle_flow_miss().

Ethan Jackson ethan at nicira.com
Tue Jan 3 22:30:17 UTC 2012


> I think that this changes correct code into a memory leak.  The
> list_remove() in the LIST_FOR_EACH_SAFE loop inside handle_flow_miss()
> ensures that the caller will not free the packet with its
> ofpbuf_list_delete() call (since the packet is no longer in the list).

Ah yes you're right.  I misread the code.  I'll drop this.

Ethan



More information about the dev mailing list