staging: wilc1000: handle freeing of 'key' & 'seq' data in wilc_add_rx_gtk()
authorAjay Singh <ajay.kathat@microchip.com>
Tue, 26 Jun 2018 06:07:11 +0000 (11:37 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 28 Jun 2018 12:55:51 +0000 (21:55 +0900)
Handle freeing of memory allocated to keep 'key' & 'seq' in wilc_add_rx_gtk().
Once completion event is received, free the memory allocated for
to avoid missing of free in work function.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/host_interface.c

index 2062f4e6f3f362b0ab8375a7121847910d0892c1..2251c39f659aed00ba75606d7cbe65cd4b74e1bf 100644 (file)
@@ -1667,8 +1667,6 @@ out_wep:
                }
 out_wpa_rx_gtk:
                complete(&msg->work_comp);
-               kfree(hif_key->attr.wpa.key);
-               kfree(hif_key->attr.wpa.seq);
                break;
 
        case WPA_PTK:
@@ -2812,8 +2810,6 @@ int wilc_add_rx_gtk(struct wilc_vif *vif, const u8 *rx_gtk, u8 gtk_key_len,
        }
 
        wait_for_completion(&msg->work_comp);
-       kfree(msg);
-       return 0;
 
 free_key:
        kfree(msg->body.key_info.attr.wpa.key);