aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/drm_edid.c
diff options
context:
space:
mode:
authorBenoit Taine <benoit.taine@lip6.fr>2014-05-26 11:21:22 -0400
committerDave Airlie <airlied@redhat.com>2014-05-27 01:48:11 -0400
commit89086bca67f04a538765faf39ea7c1b718beaf34 (patch)
treef66eab324ae7d583da2c8cf7d568c7ca303d4e8a /drivers/gpu/drm/drm_edid.c
parentc66de8cc2ea629412338e4976ef33ed5b72a2339 (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.c3
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 }