On 1/26/20 11:20 PM, Leon Romanovsky wrote: > net/ethtool/ioctl.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/net/ethtool/ioctl.c b/net/ethtool/ioctl.c > index 182bffbffa78..0501b615e920 100644 > --- a/net/ethtool/ioctl.c > +++ b/net/ethtool/ioctl.c > @@ -17,6 +17,7 @@ > #include <linux/phy.h> > #include <linux/bitops.h> > #include <linux/uaccess.h> > +#include <linux/vermagic.h> > #include <linux/vmalloc.h> > #include <linux/sfp.h> > #include <linux/slab.h> > @@ -655,6 +656,7 @@ static noinline_for_stack int ethtool_get_drvinfo(struct net_device *dev, > > memset(&info, 0, sizeof(info)); > info.cmd = ETHTOOL_GDRVINFO; > + strlcpy(info.version, UTS_RELEASE, sizeof(info.version)); This is much more reasonable, thanks for addressing everyone's concerns Leon. -- Florian
