On Fri, Jul 31, 2015 at 3:06 AM, Daniel Borkmann <dan...@iogearbox.net> wrote: > Did you test all variants on this? > > I.e. what happens when you replace an existing one, I think the > refcnt should also be dropped here. It looks like we only drop > it, in case we tried to add an action to an already existing index ... >
Yeah, clearly the replace case is anti-pattern too, like you showed. But that is a different bug, right? Since 'bind' is independent of 'replace'. As in $subject, my patch only fixes the binding case. I will send a following patch to fix replace case. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
