On 08/01/14 at 02:21pm, Nikolay Aleksandrov wrote: > Sorry for the late response but I just got around to check the 3rd patch now, > so my question about this fragment is: before nft_hash_remove() would free the > element after removal, but after this change I don't see where and when the > removed element would get freed ?
You are right. Looking at this closer we were leaking references even before the change because it would not call nft_data_uninit() to release the data objects. So it should really just call nft_hash_elem_destroy(). _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev