On Wed, Feb 07, 2018 at 09:49:02AM +0100, Felix Fietkau wrote: > flow_offload_del frees the flow, so all associated resource must be > freed before. > > Since the ct entry in struct flow_offload_entry was allocated by > flow_offload_alloc, it should be freed by flow_offload_free to take care > of the error handling path when flow_offload_add fails. > > While at it, make flow_offload_del static, since it should never be > called directly, only from the gc step Applied, thanks Felix! -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html