changing to kzalloc lets us get rid of some lines. The other concern
here is that some members of binding->ptype are still uninitialized at
the end of the function.

Signed-off-by: Salym Senyonga <salyms...@gmail.com>
---
 drivers/staging/ozwpan/ozproto.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/ozwpan/ozproto.c b/drivers/staging/ozwpan/ozproto.c
index c303ae1..e7138ed 100644
--- a/drivers/staging/ozwpan/ozproto.c
+++ b/drivers/staging/ozwpan/ozproto.c
@@ -664,7 +664,7 @@ void oz_binding_add(const char *net_dev)
 {
        struct oz_binding *binding;
 
-       binding = kmalloc(sizeof(struct oz_binding), GFP_KERNEL);
+       binding = kzalloc(sizeof(struct oz_binding), GFP_KERNEL);
        if (!binding)
                return;
 
@@ -679,10 +679,6 @@ void oz_binding_add(const char *net_dev)
                        kfree(binding);
                        return;
                }
-       } else {
-               oz_dbg(ON, "Binding to all netcards\n");
-               memset(binding->name, 0, OZ_MAX_BINDING_LEN);
-               binding->ptype.dev = NULL;
        }
        dev_add_pack(&binding->ptype);
        spin_lock_bh(&g_binding_lock);
-- 
1.7.5.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to