> 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