Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1237568 > unrolled thread

[PATCH] pcmcia: use kstrdup() in pcmcia_device_query()

Started byGeliang Tang <geliangtang@163.com>
First post2015-10-01 18:50 +0200
Last post2015-10-01 18:50 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [PATCH] pcmcia: use kstrdup() in pcmcia_device_query() Geliang Tang <geliangtang@163.com> - 2015-10-01 18:50 +0200

#1237568 — [PATCH] pcmcia: use kstrdup() in pcmcia_device_query()

FromGeliang Tang <geliangtang@163.com>
Date2015-10-01 18:50 +0200
Subject[PATCH] pcmcia: use kstrdup() in pcmcia_device_query()
Message-ID<qePdo-1O6-13@gated-at.bofh.it>
Use kstrdup instead of kmalloc and strncpy.

Signed-off-by: Geliang Tang <geliangtang@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 majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web