diff options
author | Amitoj Kaur Chawla <amitoj1606@gmail.com> | 2016-06-23 00:52:01 -0400 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2016-06-23 12:55:58 -0400 |
commit | 5c93f56f770e69c4cf2dbaebecd6bcca205949f9 (patch) | |
tree | 895b867b6fd3d05c9a72fd28c052423921f4521c /fs/dlm | |
parent | 505ee5283c3d23220dfbf7630bfc4efdc391e743 (diff) |
dlm: 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>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm')
-rw-r--r-- | fs/dlm/lowcomms.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 1ab012a27d9f..963016c8f3d1 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c | |||
@@ -1279,10 +1279,9 @@ static void init_local(void) | |||
1279 | if (dlm_our_addr(&sas, i)) | 1279 | if (dlm_our_addr(&sas, i)) |
1280 | break; | 1280 | break; |
1281 | 1281 | ||
1282 | addr = kmalloc(sizeof(*addr), GFP_NOFS); | 1282 | addr = kmemdup(&sas, sizeof(*addr), GFP_NOFS); |
1283 | if (!addr) | 1283 | if (!addr) |
1284 | break; | 1284 | break; |
1285 | memcpy(addr, &sas, sizeof(*addr)); | ||
1286 | dlm_local_addr[dlm_local_count++] = addr; | 1285 | dlm_local_addr[dlm_local_count++] = addr; |
1287 | } | 1286 | } |
1288 | } | 1287 | } |