diff options
author | Tobin C. Harding <me@tobin.cc> | 2017-03-14 21:20:12 +1100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-16 11:41:37 +0900 |
commit | aa6ca807baace49992ad6990c9aeda22289be5cd (patch) | |
tree | 24572bde92368de307504a95c8ad6565d851fd07 /drivers/staging/ks7010/ks_hostif.c | |
parent | 7bb6313d011f16405b338a228f540d3ab5aee58a (diff) | |
download | op-kernel-dev-aa6ca807baace49992ad6990c9aeda22289be5cd.zip op-kernel-dev-aa6ca807baace49992ad6990c9aeda22289be5cd.tar.gz |
staging: ks7010: remove custom return values
Driver code uses custom return values (often positive) to signal error
condition instead of using standard kernel error codes.
Replace custom return values with standard kernel error codes.
Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ks7010/ks_hostif.c')
-rw-r--r-- | drivers/staging/ks7010/ks_hostif.c | 26 |
1 files changed, 16 insertions, 10 deletions
diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index 1e49717..db10e16 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -110,14 +110,13 @@ int get_current_ap(struct ks_wlan_private *priv, struct link_ap_info_t *ap_info) struct local_ap_t *ap; union iwreq_data wrqu; struct net_device *netdev = priv->net_dev; - int rc = 0; DPRINTK(3, "\n"); ap = &priv->current_ap; if ((priv->connect_status & CONNECT_STATUS_MASK) == DISCONNECT_STATUS) { memset(ap, 0, sizeof(struct local_ap_t)); - return 1; + return -EPERM; } /* bssid */ @@ -198,7 +197,7 @@ int get_current_ap(struct ks_wlan_private *priv, struct link_ap_info_t *ap_info) DPRINTK(4, "\n ext_rate_set_size=%d\n rate_set_size=%d\n", ap_info->ext_rate_set.size, ap_info->rate_set.size); - return rc; + return 0; } static @@ -1125,12 +1124,13 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet) struct ieee802_1x_hdr *aa1x_hdr; struct wpa_eapol_key *eap_key; struct ethhdr *eth; + int rc; packet_len = packet->len; if (packet_len > ETH_FRAME_LEN) { DPRINTK(1, "bad length packet_len=%d\n", packet_len); - dev_kfree_skb(packet); - return -1; + rc = -EOVERFLOW; + goto err_kfree_skb; } if (((priv->connect_status & CONNECT_STATUS_MASK) == DISCONNECT_STATUS) @@ -1157,8 +1157,8 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet) if (!pp) { DPRINTK(3, "allocate memory failed..\n"); - dev_kfree_skb(packet); - return -2; + rc = -ENOMEM; + goto err_kfree_skb; } p = (unsigned char *)pp->data; @@ -1171,9 +1171,8 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet) if (memcmp(&priv->eth_addr[0], eth->h_source, ETH_ALEN)) { DPRINTK(1, "invalid mac address !!\n"); DPRINTK(1, "ethernet->h_source=%pM\n", eth->h_source); - dev_kfree_skb(packet); - kfree(pp); - return -3; + rc = -ENXIO; + goto err_kfree; } /* MAC address copy */ @@ -1276,6 +1275,13 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet) } return result; + +err_kfree: + kfree(pp); +err_kfree_skb: + dev_kfree_skb(packet); + + return rc; } #define ps_confirm_wait_inc(priv) do { \ |