diff options
| author | Andrzej Hajda <a.hajda@samsung.com> | 2014-04-15 09:33:01 -0400 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2014-04-29 19:48:28 -0400 |
| commit | 25c8b5c3048cb6c98d402ca8d4735ccf910f727c (patch) | |
| tree | 31a7fe1abcded36ca269adb54e5229962951c1d7 | |
| parent | d8af20bcae25e8792ccd8c94404e6e57e7db75f2 (diff) | |
drm/exynos: balance framebuffer refcount
exynos_drm_crtc_mode_set assigns primary framebuffer to plane without
taking reference. Then during framebuffer removal it is dereferenced twice,
causing oops. The patch fixes it.
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | drivers/gpu/drm/exynos/exynos_drm_crtc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.c b/drivers/gpu/drm/exynos/exynos_drm_crtc.c index e930d4fe29c7..1ef5ab9c9d51 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_crtc.c +++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.c | |||
| @@ -145,6 +145,7 @@ exynos_drm_crtc_mode_set(struct drm_crtc *crtc, struct drm_display_mode *mode, | |||
| 145 | 145 | ||
| 146 | plane->crtc = crtc; | 146 | plane->crtc = crtc; |
| 147 | plane->fb = crtc->primary->fb; | 147 | plane->fb = crtc->primary->fb; |
| 148 | drm_framebuffer_reference(plane->fb); | ||
| 148 | 149 | ||
| 149 | return 0; | 150 | return 0; |
| 150 | } | 151 | } |
