diff options
author | Benoit Taine <benoit.taine@lip6.fr> | 2014-05-26 11:21:22 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2014-05-27 01:48:11 -0400 |
commit | 89086bca67f04a538765faf39ea7c1b718beaf34 (patch) | |
tree | f66eab324ae7d583da2c8cf7d568c7ca303d4e8a /drivers/gpu/drm/drm_edid.c | |
parent | c66de8cc2ea629412338e4976ef33ed5b72a2339 (diff) |
drm/edid: Use kmemdup instead of kmalloc + memcpy
This issue was reported by coccicheck using the semantic patch
at scripts/coccinelle/api/memdup.cocci
Signed-off-by: Benoit Taine <benoit.taine@lip6.fr>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/drm_edid.c')
-rw-r--r-- | drivers/gpu/drm/drm_edid.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 7a4fd2ed1280..d74239fec291 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c | |||
@@ -3228,10 +3228,9 @@ int drm_edid_to_speaker_allocation(struct edid *edid, u8 **sadb) | |||
3228 | 3228 | ||
3229 | /* Speaker Allocation Data Block */ | 3229 | /* Speaker Allocation Data Block */ |
3230 | if (dbl == 3) { | 3230 | if (dbl == 3) { |
3231 | *sadb = kmalloc(dbl, GFP_KERNEL); | 3231 | *sadb = kmemdup(&db[1], dbl, GFP_KERNEL); |
3232 | if (!*sadb) | 3232 | if (!*sadb) |
3233 | return -ENOMEM; | 3233 | return -ENOMEM; |
3234 | memcpy(*sadb, &db[1], dbl); | ||
3235 | count = dbl; | 3234 | count = dbl; |
3236 | break; | 3235 | break; |
3237 | } | 3236 | } |