staging: ks7010: don't print skb->dev->name if skb is null
authorColin Ian King <colin.king@canonical.com>
Mon, 15 Aug 2016 14:45:04 +0000 (15:45 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 21 Aug 2016 16:12:57 +0000 (18:12 +0200)
A null pointer dereference will occur when skb is null and
skb->dev->name is printed.  Replace the skb->dev->name with
plain text "ks_wlan" to fix this.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks_hostif.c

index f4cee81..c5fc31c 100644 (file)
@@ -485,8 +485,7 @@ void hostif_data_indication(struct ks_wlan_private *priv)
                        netif_rx(skb);
                } else {
                        printk(KERN_WARNING
-                              "%s: Memory squeeze, dropping packet.\n",
-                              skb->dev->name);
+                              "ks_wlan: Memory squeeze, dropping packet.\n");
                        priv->nstats.rx_dropped++;
                }
                break;
@@ -521,8 +520,7 @@ void hostif_data_indication(struct ks_wlan_private *priv)
                        netif_rx(skb);
                } else {
                        printk(KERN_WARNING
-                              "%s: Memory squeeze, dropping packet.\n",
-                              skb->dev->name);
+                              "ks_wlan: Memory squeeze, dropping packet.\n");
                        priv->nstats.rx_dropped++;
                }
                break;