diff options
author | Figo.zhang <figo1802@gmail.com> | 2009-06-10 00:18:38 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-11 05:47:14 -0400 |
commit | c0feed87fe452c20681ae29909adaecedd1ec9f3 (patch) | |
tree | 29e99cd46d95e9a123c144fdb0b56c7fd168966f | |
parent | 50d36a93dc2c773c9625c62275b88ac936d9c4ba (diff) |
tehuti: No need check vfree() pointer.
vfree() does its own 'NULL' check, so no need for check before
calling it.
Signed-off-by: Figo.zhang <figo1802@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/tehuti.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/net/tehuti.c b/drivers/net/tehuti.c index 093807a182f2..3c2679cd196b 100644 --- a/drivers/net/tehuti.c +++ b/drivers/net/tehuti.c | |||
@@ -948,8 +948,7 @@ static void print_rxfd(struct rxf_desc *rxfd); | |||
948 | 948 | ||
949 | static void bdx_rxdb_destroy(struct rxdb *db) | 949 | static void bdx_rxdb_destroy(struct rxdb *db) |
950 | { | 950 | { |
951 | if (db) | 951 | vfree(db); |
952 | vfree(db); | ||
953 | } | 952 | } |
954 | 953 | ||
955 | static struct rxdb *bdx_rxdb_create(int nelem) | 954 | static struct rxdb *bdx_rxdb_create(int nelem) |
@@ -1482,10 +1481,8 @@ static void bdx_tx_db_close(struct txdb *d) | |||
1482 | { | 1481 | { |
1483 | BDX_ASSERT(d == NULL); | 1482 | BDX_ASSERT(d == NULL); |
1484 | 1483 | ||
1485 | if (d->start) { | 1484 | vfree(d->start); |
1486 | vfree(d->start); | 1485 | d->start = NULL; |
1487 | d->start = NULL; | ||
1488 | } | ||
1489 | } | 1486 | } |
1490 | 1487 | ||
1491 | /************************************************************************* | 1488 | /************************************************************************* |