[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230418195001.GB21058@breakpoint.cc>
Date: Tue, 18 Apr 2023 21:50:01 +0200
From: Florian Westphal <fw@...len.de>
To: Quentin Monnet <quentin@...valent.com>
Cc: Florian Westphal <fw@...len.de>, bpf@...r.kernel.org,
netdev@...r.kernel.org, netfilter-devel@...r.kernel.org,
dxu@...uu.xyz, qde@...cy.de
Subject: Re: [PATCH bpf-next v3 5/6] tools: bpftool: print netfilter link info
Quentin Monnet <quentin@...valent.com> wrote:
> > + if (nf_link_count > (INT_MAX / sizeof(info))) {
> > + fprintf(stderr, "link count %d\n", nf_link_count);
>
> The only nit I have is that we could use p_err() here, and have a more
> descriptive message (letting user know that we've reached a limit).
Drats, I'll replace this and will retain your RvB tag in v4, thanks.
Powered by blists - more mailing lists