From b127a86f1dbd542fbd9f7548ed303c6932b1a5c9 Mon Sep 17 00:00:00 2001
From: Johan Klokkhammer Helsing <johan.helsing@qt.io>
Date: Mon, 26 Feb 2018 13:27:37 +0100
Subject: [PATCH] Fix nullptr dereference in client tests

Change-Id: I932cd20f17b5486a1161569c5e9a3feebabeee8e
Reviewed-by: Pier Luigi Fiorini <pierluigi.fiorini@liri.io>
---
 tests/auto/wayland/shared/mockcompositor.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/auto/wayland/shared/mockcompositor.cpp b/tests/auto/wayland/shared/mockcompositor.cpp
index 1d76cec1200..ef2636b0f9f 100644
--- a/tests/auto/wayland/shared/mockcompositor.cpp
+++ b/tests/auto/wayland/shared/mockcompositor.cpp
@@ -245,7 +245,8 @@ QSharedPointer<MockOutput> MockCompositor::output(int index)
 {
     QSharedPointer<MockOutput> result;
     lock();
-    result = m_compositor->outputs().at(index)->mockOutput();
+    if (Impl::Output *output = m_compositor->outputs().value(index, nullptr))
+        result = output->mockOutput();
     unlock();
     return result;
 }