aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/intel_bios.c
diff options
context:
space:
mode:
authorThomas Meyer <thomas@m3y3r.de>2011-11-29 16:08:00 -0500
committerDaniel Vetter <daniel.vetter@ffwll.ch>2012-01-17 11:33:24 -0500
commit493dea2876df144ec57a6a9efbe55db43c7a729e (patch)
tree1105571017f1c380df8ac54f9be28ee378e7cca0 /drivers/gpu/drm/i915/intel_bios.c
parentd7e96feab83d29e27d14c60f1fa7c716ef7880cd (diff)
drm/i915: Use kcalloc instead of kzalloc to allocate array
The advantage of kcalloc is, that will prevent integer overflows which could result from the multiplication of number of elements and size and it is also a bit nicer to read. The semantic patch that makes this change is available in https://lkml.org/lkml/2011/11/25/107 Signed-off-by: Thomas Meyer <thomas@m3y3r.de> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_bios.c')
-rw-r--r--drivers/gpu/drm/i915/intel_bios.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c
index 63880e2e5cfd..50656339d922 100644
--- a/drivers/gpu/drm/i915/intel_bios.c
+++ b/drivers/gpu/drm/i915/intel_bios.c
@@ -572,7 +572,7 @@ parse_device_mapping(struct drm_i915_private *dev_priv,
572 DRM_DEBUG_KMS("no child dev is parsed from VBT\n"); 572 DRM_DEBUG_KMS("no child dev is parsed from VBT\n");
573 return; 573 return;
574 } 574 }
575 dev_priv->child_dev = kzalloc(sizeof(*p_child) * count, GFP_KERNEL); 575 dev_priv->child_dev = kcalloc(count, sizeof(*p_child), GFP_KERNEL);
576 if (!dev_priv->child_dev) { 576 if (!dev_priv->child_dev) {
577 DRM_DEBUG_KMS("No memory space for child device\n"); 577 DRM_DEBUG_KMS("No memory space for child device\n");
578 return; 578 return;