commit 7a5c562d1aa7e893f0a0910a3f6860b450127acd
author comic fans <comicfans44@gmail.com> Thu Mar 17 14:29:27 2016 +0200
committer Pekka Paalanen <pekka.paalanen@collabora.co.uk> Thu Mar 17 14:29:27 2016 +0200
tree fa95c6a877016d2419aad914f89023f30dfb84f8
parent 720e0c9bfa2ce8febc813209babfc49e85e94fa5

compositor-drm: fix memcmp using a bad pointer in drm_outout_choose_initial_mode

current_mode is already the pointer, taking the address of it is wrong.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94562
Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
[Pekka: rewrote the patch]
Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>

src/compositor-drm.c

diff --git a/src/compositor-drm.c b/src/compositor-drm.c
index e01f6b9..621414c 100644
--- a/src/compositor-drm.c
+++ b/src/compositor-drm.c
@@ -2212,7 +2212,7 @@
 		    height == drm_mode->base.height)
 			configured = drm_mode;
 
-		if (memcmp(&current_mode, &drm_mode->mode_info,
+		if (memcmp(current_mode, &drm_mode->mode_info,
 			   sizeof *current_mode) == 0)
 			current = drm_mode;