On Thu, 09 Apr 2015 15:39:37 -0700
Joe Perches <joe@xxxxxxxxxxx> wrote:
> And Steven,
>
> commit e71456ae9871
> ("netfilter: Remove checks of seq_printf() return values")
> mistakenly converted this to use seq_has_overflowed()
>
> Ideally all seq_show functions would be converted from
>
> int foo_seq_show(...)
> {
> seq_printf(s, ...);
> return seq_has_overflowed(s);
> }
>
> to
>
> int foo_seq_show(...)
> {
> seq_printf(s, ...);
> return 0;
> }
>
> I made that mistake in a patch and corrected it later.
>
> Here's a suggested -next patch:
>
> Perhaps it shouldn't be submitted now as it may conflict
> with what Richard might submit if he propses an equivalent
> patch to above for -next.
>
> There is an appropriate use of return seq_has_overflowed in:
>
> net/netfilter/xt_hashlimit.c:825: return seq_has_overflowed(s);
>
> That's not a seq_show function, it's a helper for one.
I have other things to work on, so feel free to fix it up among
yourselves.
-- Steve
--
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
