Re: [PATCH] netlink: do not proceed if dump's start() errs

From: Jason A. Donenfeld
Date: Wed Sep 27 2017 - 09:06:50 EST


On Wed, Sep 27, 2017 at 3:05 PM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
> I guess you could change it to
>
> if (cb->start)
> ret = cb->start(cb);
> if (!ret)
> ret = netlink_dump(sk);

Very clean. I'll do it like that. I'll wait a bit longer before
submitting v2, but beyond that, seems sane to you?

Jason