From: Greg KH <gre...@linuxfoundation.org>
Date: Sun, 9 Dec 2018 17:32:45 +0100

> +     } else {
>               port_spec = hso_get_config_data(interface);
> +             if (IS_ERR_VALUE((long)port_spec))
> +                     goto exit;

'port_spec' is an 'int', it makes no sense to cast it 3 times all the
way back to 'int' to figure out if it is a negative error value or
not. (--> long --> void * --> int)

Reply via email to