From: Mandeep Singh Baines Date: Tue, 15 Jan 2013 21:15:34 +0000 (-0800) Subject: CHROMIUM: drm/exynos: drop fb reference in page_flip error path X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=851f1871787bd03f7722b04527e6a7ec328893ff;p=people%2Faperard%2Flinux-chromebook.git CHROMIUM: drm/exynos: drop fb reference in page_flip error path Commit "CHROMIUM: drm/exynos: Fix fb refcounting", moved the exynos_drm_fb_get before the kds failure check. To avoid a leak, we need to drop the reference if kds fails. BUG=chrome-os-partner:16069 TEST=compile,run Change-Id: I832d0f42884ba33953fa40bb156b38e01bf8a3c5 Signed-off-by: Mandeep Singh Baines Reviewed-on: https://gerrit.chromium.org/gerrit/41302 Reviewed-by: Stéphane Marchesin --- diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.c b/drivers/gpu/drm/exynos/exynos_drm_crtc.c index fae84d1bb10ec..97602d0bd2e6e 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_crtc.c +++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.c @@ -516,6 +516,7 @@ static int exynos_drm_crtc_page_flip(struct drm_crtc *crtc, return 0; fail_kds: + exynos_drm_fb_put(exynos_fb); *pkds = NULL; spin_lock_irqsave(&dev->event_lock, flags); exynos_crtc->flip_in_flight--;