drm/i915: check for return value
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>
Thu, 8 Oct 2015 13:58:00 +0000 (19:28 +0530)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 9 Dec 2015 13:32:22 +0000 (14:32 +0100)
We were not checking the return value of drm_encoder_init() which can
fail. And if it fails then we will be working with an uninitialized
encoder.

Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1444312681-10149-2-git-send-email-sudipm.mukherjee@gmail.com
drivers/gpu/drm/i915/intel_dp.c

index 7e3ffc30a00e5032b2eb6417f44ccd6013204a5a..78fe8e083b4820efa4648bf0ecafdd9db976c740 100644 (file)
@@ -5987,8 +5987,9 @@ intel_dp_init(struct drm_device *dev,
        intel_encoder = &intel_dig_port->base;
        encoder = &intel_encoder->base;
 
-       drm_encoder_init(dev, &intel_encoder->base, &intel_dp_enc_funcs,
-                        DRM_MODE_ENCODER_TMDS);
+       if (drm_encoder_init(dev, &intel_encoder->base, &intel_dp_enc_funcs,
+                            DRM_MODE_ENCODER_TMDS))
+               goto err_encoder_init;
 
        intel_encoder->compute_config = intel_dp_compute_config;
        intel_encoder->disable = intel_disable_dp;
@@ -6037,6 +6038,7 @@ intel_dp_init(struct drm_device *dev,
 
 err_init_connector:
        drm_encoder_cleanup(encoder);
+err_encoder_init:
        kfree(intel_connector);
 err_connector_alloc:
        kfree(intel_dig_port);