diff options
author | Amitoj Kaur Chawla <amitoj1606@gmail.com> | 2016-06-23 10:19:37 +0530 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-06-27 09:56:58 -0400 |
commit | 810bf11033637a2069952afb9c37f3afd3bbfe41 (patch) | |
tree | d6167bc53bd00cbeb63643e552fb85960d2344a4 | |
parent | ac5fd4f4b97f5ac9a01303c024d26f34a99633c8 (diff) | |
download | linux-810bf11033637a2069952afb9c37f3afd3bbfe41.tar.bz2 |
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>
-rw-r--r-- | net/tipc/server.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/tipc/server.c b/net/tipc/server.c index 272d20a795d5..215849ce453d 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; |