On Tue, Apr 05, 2016 at 07:18:14PM +0200, walter harms wrote: > > > Am 05.04.2016 02:56, schrieb Guillaume Nault: > > @@ -1043,12 +1048,39 @@ static int ppp_dev_configure(struct net *src_net, struct net_device *dev, > > const struct ppp_config *conf) > > { > > struct ppp *ppp = netdev_priv(dev); > > + struct file *file; > > int indx; > > + int err; > > + > > + if (conf->fd < 0) { > > + file = conf->file; > > + if (!file) { > > + err = -EBADF; > > + goto out; > > why not just return -EBADF; > > > + } > > + } else { > > + file = fget(conf->fd); > > + if (!file) { > > + err = -EBADF; > > + goto out; > > why not just return -EBADF; > Just because the 'out' label is declared anyway and because this centralises the return point. But I agree returning -EBADF directly could be more readable. I don't have strong opinion. -- To unsubscribe from this list: send the line "unsubscribe linux-ppp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html