diff options
author | Julia Lawall <julia@diku.dk> | 2010-05-13 15:58:56 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2010-05-18 01:57:05 -0400 |
commit | 6ebc22e6d06760466859b79d7b3b3edad264a230 (patch) | |
tree | c55bcec8359e666a625b9108a7375f18c7f5c4b7 /drivers/gpu/drm/drm_fops.c | |
parent | 96525a2f7448c427bb8d99240907a6bd2d9e818c (diff) |
drivers/gpu/drm: Use kzalloc
Use kzalloc rather than the combination of kmalloc and memset.
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression x,size,flags;
statement S;
@@
-x = kmalloc(size,flags);
+x = kzalloc(size,flags);
if (x == NULL) S
-memset(x, 0, size);
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Reviewed-by: Corbin Simpson <MostAwesomeDude@gmail.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/drm_fops.c')
-rw-r--r-- | drivers/gpu/drm/drm_fops.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c index 9d532d7fdf59..e7aace20981f 100644 --- a/drivers/gpu/drm/drm_fops.c +++ b/drivers/gpu/drm/drm_fops.c | |||
@@ -243,11 +243,10 @@ static int drm_open_helper(struct inode *inode, struct file *filp, | |||
243 | 243 | ||
244 | DRM_DEBUG("pid = %d, minor = %d\n", task_pid_nr(current), minor_id); | 244 | DRM_DEBUG("pid = %d, minor = %d\n", task_pid_nr(current), minor_id); |
245 | 245 | ||
246 | priv = kmalloc(sizeof(*priv), GFP_KERNEL); | 246 | priv = kzalloc(sizeof(*priv), GFP_KERNEL); |
247 | if (!priv) | 247 | if (!priv) |
248 | return -ENOMEM; | 248 | return -ENOMEM; |
249 | 249 | ||
250 | memset(priv, 0, sizeof(*priv)); | ||
251 | filp->private_data = priv; | 250 | filp->private_data = priv; |
252 | priv->filp = filp; | 251 | priv->filp = filp; |
253 | priv->uid = current_euid(); | 252 | priv->uid = current_euid(); |