summaryrefslogtreecommitdiffstats
path: root/net/tipc/server.c
diff options
context:
space:
mode:
authorAmitoj Kaur Chawla <amitoj1606@gmail.com>2016-06-23 10:19:37 +0530
committerDavid S. Miller <davem@davemloft.net>2016-06-27 09:56:58 -0400
commit810bf11033637a2069952afb9c37f3afd3bbfe41 (patch)
treed6167bc53bd00cbeb63643e552fb85960d2344a4 /net/tipc/server.c
parentac5fd4f4b97f5ac9a01303c024d26f34a99633c8 (diff)
downloadop-kernel-dev-810bf11033637a2069952afb9c37f3afd3bbfe41.zip
op-kernel-dev-810bf11033637a2069952afb9c37f3afd3bbfe41.tar.gz
tipc: Use kmemdup instead of kmalloc and memcpy
Replace calls to kmalloc followed by a memcpy with a direct call to kmemdup. The Coccinelle semantic patch used to make this change is as follows: @@ expression from,to,size,flag; statement S; @@ - to = \(kmalloc\|kzalloc\)(size,flag); + to = kmemdup(from,size,flag); if (to==NULL || ...) S - memcpy(to, from, size); Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com> Acked-by: Ying Xue <ying.xue@windriver.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/server.c')
-rw-r--r--net/tipc/server.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/net/tipc/server.c b/net/tipc/server.c
index 272d20a..215849c 100644
--- a/net/tipc/server.c
+++ b/net/tipc/server.c
@@ -418,13 +418,12 @@ static struct outqueue_entry *tipc_alloc_entry(void *data, int len)
if (!entry)
return NULL;
- buf = kmalloc(len, GFP_ATOMIC);
+ buf = kmemdup(data, len, GFP_ATOMIC);
if (!buf) {
kfree(entry);
return NULL;
}
- memcpy(buf, data, len);
entry->iov.iov_base = buf;
entry->iov.iov_len = len;
OpenPOWER on IntegriCloud