Use kstrdup instead of kmalloc and strncpy.

Signed-off-by: Geliang Tang <geliangt...@163.com>
---
 drivers/pcmcia/ds.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/pcmcia/ds.c b/drivers/pcmcia/ds.c
index 0decee6..489ea10 100644
--- a/drivers/pcmcia/ds.c
+++ b/drivers/pcmcia/ds.c
@@ -468,12 +468,10 @@ static int pcmcia_device_query(struct pcmcia_device 
*p_dev)
                        if ((length < 2) || (length > 255))
                                continue;
 
-                       new = kmalloc(sizeof(char) * length, GFP_KERNEL);
+                       new = kstrdup(tmp, GFP_KERNEL);
                        if (!new)
                                continue;
 
-                       new = strncpy(new, tmp, length);
-
                        tmp = p_dev->prod_id[i];
                        p_dev->prod_id[i] = new;
                        kfree(tmp);
-- 
2.5.0


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to