From: Hangbin Liu <liuhang...@gmail.com> Date: Wed, 4 Jul 2018 14:29:46 +0800
> @@ -7062,8 +7073,7 @@ int dev_change_flags(struct net_device *dev, unsigned > int flags) > return ret; > > changes = (old_flags ^ dev->flags) | (old_gflags ^ dev->gflags); > - __dev_notify_flags(dev, old_flags, changes); > - return ret; > + return __dev_notify_flags(dev, old_flags, changes); By doing this, you will lose any error code returned from dev_open(). That is why we return 'ret' instead of plain '0' here.
