brcmfmac: finalize transmit upon any rollback failure

All rollback failures should result in freeing of the sk_buff
by calling brcmf_txfinalize().

Reviewed-by: Hante Meuleman <meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c b/drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c
index 1cfec5601c68..13518ecce047 100644
--- a/drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c
+++ b/drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c
@@ -1616,7 +1616,6 @@ brcmf_fws_rollback_toq(struct brcmf_fws_info *fws, struct sk_buff *skb)
                                 /* free the hanger slot */
                                 brcmf_fws_hanger_poppkt(&fws->hanger, hslot,
                                                         &pktout, true);
-                                brcmf_txfinalize(fws->drvr, skb, false);
                                 rc = -EINVAL;
                                 goto fail;
                         }
@@ -1650,9 +1649,10 @@ brcmf_fws_rollback_toq(struct brcmf_fws_info *fws, struct sk_buff *skb)
 
 
 fail:
-        if (rc)
+        if (rc) {
+                brcmf_txfinalize(fws->drvr, skb, false);
                 fws->stats.rollback_failed++;
-        else
+        } else
                 fws->stats.rollback_success++;
         return rc;
 }