On 10/7/18 4:40 AM, Christian Brauner wrote: >> @@ -2718,6 +2743,13 @@ static int ipmr_rtm_dumplink(struct sk_buff *skb, >> struct netlink_callback *cb) >> unsigned int e = 0, s_e; >> struct mr_table *mrt; >> >> + if (cb->strict_check) { >> + int err = ipmr_valid_dumplink(cb->nlh, cb->extack); >> + >> + if (err) >> + return err; > > Nit: can we remove the unnecessary \n, please.
Coding standards dictate a newline between declarations and code. And that is my preference too.