On Tue, Feb 18, 2014 at 01:27:08AM +0000, Patrick McHardy wrote: > On Mon, Feb 17, 2014 at 10:03:01PM +0100, Florian Westphal wrote: > > Patrick McHardy <kaber@xxxxxxxxx> wrote: > > > On Mon, Feb 17, 2014 at 04:48:05PM +0100, Florian Westphal wrote: > > > > We segfault on 'list filter' when meta expr is used as _u8 > > > > returns invalid register 0. > > > > > > This seems to only affect the master branch. Not sure how many people > > > are using that and if its worth doing a new release. I'd rather merge > > > the next-3.14 branch and have people use that. > > > > I am now on 0545e0c13b as head but I still get segfault: > > > > $ nft add rule filter output meta mark set 42 > > $ nft list table -> segv > > > > .. and my patch should still apply afaics. > > Right, we've fixed that for other types. Please push your patch to master. > > We should probably add a big warning to libnftnl for these cases or > even an assert() since its misuse of the API that leads to hard to > debug problems. Indeed. I started a patch for that here but failed to find time so far to finish it, it should help to catch wrong API uses. -- 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
