
15 Sep
2021
15 Sep
'21
1:47 a.m.
On Tue, 14 Sep 2021 20:31:29 +0300 Julian Wiedmann wrote:
- if (!dev->ethtool_ops->set_coalesce) + if (!dev->ethtool_ops->set_coalesce && !dev->ethtool_ops->get_coalesce) return -EOPNOTSUPP;
This needs to be
if (!set_coalesce || !get_coalesce) return -EOPNOTSUPP;
Otherwise you end up calling a NULL pointer below if just _one_ of the callbacks is available.
Good catch, care to send a fix?