netdev
[Top] [All Lists]

[PATCH 7/9][ATM]: [iphase] use after free, found by Coverity tool

To: netdev@xxxxxxxxxxx
Subject: [PATCH 7/9][ATM]: [iphase] use after free, found by Coverity tool
From: "chas williams - CONTRACTOR" <chas@xxxxxxxxxxxxxxxx>
Date: Mon, 07 Mar 2005 17:07:32 -0500
Cc: davem@xxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
please apply to 2.6 -- thanks!

# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
#   2005/03/07 15:16:16-05:00 chas@xxxxxxxxxxxxxxxxxxxxxx 
#   [ATM]: [iphase] use after free, found by Coverity tool
#   
#   Signed-off-by: Alexander Nyberg <alexn@xxxxxxxxx>
#   Signed-off-by: Chas Williams <chas@xxxxxxxxxxxxxxxx>
# 
# drivers/atm/iphase.c
#   2005/03/07 15:15:58-05:00 chas@xxxxxxxxxxxxxxxxxxxxxx +1 -1
#   [ATM]: [iphase] use after free, found by Coverity tool
#   
#   Signed-off-by: Alexander Nyberg <alexn@xxxxxxxxx>
#   Signed-off-by: Chas Williams <chas@xxxxxxxxxxxxxxxx>
# 
diff -Nru a/drivers/atm/iphase.c b/drivers/atm/iphase.c
--- a/drivers/atm/iphase.c      2005-03-07 15:36:19 -05:00
+++ b/drivers/atm/iphase.c      2005-03-07 15:36:19 -05:00
@@ -3244,8 +3244,8 @@
        iadev_count--;
        ia_dev[iadev_count] = NULL;
        _ia_dev[iadev_count] = NULL;
+       IF_EVENT(printk("deregistering iav at (itf:%d)\n", dev->number);)
        atm_dev_deregister(dev);
-       IF_EVENT(printk("iav deregistered at (itf:%d)\n", dev->number);)
 
        iounmap(iadev->base);  
        pci_disable_device(pdev);

<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH 7/9][ATM]: [iphase] use after free, found by Coverity tool, chas williams - CONTRACTOR <=