> 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
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to