diff options
| author | Tillmann Heidsieck <theidsieck@leenox.de> | 2015-10-10 21:47:19 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-10-13 04:56:27 +0200 |
| commit | cbb41b91e68a302087762823136c9067138cff7c (patch) | |
| tree | 8d7c4b34a369a781b57246a7fa9b3ae3e9a5160e /drivers/atm | |
| parent | atm: iphase: return -ENOMEM instead of -1 in case of failed kmalloc() (diff) | |
| download | linux-cbb41b91e68a302087762823136c9067138cff7c.tar.xz linux-cbb41b91e68a302087762823136c9067138cff7c.zip | |
atm: iphase: fix misleading indention
Fix a smatch warning:
drivers/atm/iphase.c:1178 rx_pkt() warn: curly braces intended?
The code is correct, the indention is misleading. In case the allocation
of skb fails, we want to skip to the end.
Signed-off-by: Tillmann Heidsieck <theidsieck@leenox.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/atm')
| -rw-r--r-- | drivers/atm/iphase.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/atm/iphase.c b/drivers/atm/iphase.c index d5010d7a99c3..7d00f2994738 100644 --- a/drivers/atm/iphase.c +++ b/drivers/atm/iphase.c @@ -1176,7 +1176,7 @@ static int rx_pkt(struct atm_dev *dev) if (!(skb = atm_alloc_charge(vcc, len, GFP_ATOMIC))) { if (vcc->vci < 32) printk("Drop control packets\n"); - goto out_free_desc; + goto out_free_desc; } skb_put(skb,len); // pwang_test |