diff options
author | Sabrina Dubroca <sd@queasysnail.net> | 2016-04-22 05:28:03 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-04-24 14:31:58 -0400 |
commit | 497f358aa4c0d99b75ec204407389920d5e33ec5 (patch) | |
tree | 735124d9ec42abdf4776cc7d638bbe839ac9ac6e | |
parent | c10c63ea739bce3b8a6ab85c7bb472d900c9b070 (diff) |
macsec: don't put a NULL rxsa
The "deliver:" path of macsec_handle_frame can be called with
rx_sa == NULL. Check rx_sa != NULL before calling macsec_rxsa_put().
Fixes: c09440f7dcb3 ("macsec: introduce IEEE 802.1AE driver")
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
Acked-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/macsec.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/macsec.c b/drivers/net/macsec.c index 5f3ea8026074..2a2136b7d324 100644 --- a/drivers/net/macsec.c +++ b/drivers/net/macsec.c | |||
@@ -1161,7 +1161,8 @@ deliver: | |||
1161 | macsec_extra_len(macsec_skb_cb(skb)->has_sci)); | 1161 | macsec_extra_len(macsec_skb_cb(skb)->has_sci)); |
1162 | macsec_reset_skb(skb, secy->netdev); | 1162 | macsec_reset_skb(skb, secy->netdev); |
1163 | 1163 | ||
1164 | macsec_rxsa_put(rx_sa); | 1164 | if (rx_sa) |
1165 | macsec_rxsa_put(rx_sa); | ||
1165 | count_rx(dev, skb->len); | 1166 | count_rx(dev, skb->len); |
1166 | 1167 | ||
1167 | rcu_read_unlock(); | 1168 | rcu_read_unlock(); |