ethtool: fec: sanitize ethtool_fecparam->fec
authorJakub Kicinski <kuba@kernel.org>
Thu, 25 Mar 2021 01:11:59 +0000 (18:11 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 25 Mar 2021 23:46:53 +0000 (16:46 -0700)
commit42ce127d98641f2cb19cd499b5b3beb472c7eff1
treef2188b4114169376bd5e2883732b370aae68f56a
parentd3b37fc805d9ef697451730ebdfc7e35e6c2ace8
ethtool: fec: sanitize ethtool_fecparam->fec

Reject NONE on set, this mode means device does not support
FEC so it's a little out of place in the set interface.

This should be safe to do - user space ethtool does not allow
the use of NONE on set. A few drivers treat it the same as OFF,
but none use it instead of OFF.

Similarly reject an empty FEC mask. The common user space tool
will not send such requests and most drivers correctly reject
it already.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ethtool/ioctl.c