summaryrefslogtreecommitdiffstats
path: root/mandriva/2010.2/applications/krandr/krandr-0.5.2.1-grab_server_avoiding_crashes.patch
blob: 57faa1cf6550c7f9e3bbd22719e1d2d7daa21bd3 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
diff -p -up krandr-0.5.2.1/randr/randrcrtc.cpp.orig krandr-0.5.2.1/randr/randrcrtc.cpp
--- krandr-0.5.2.1/randr/randrcrtc.cpp.orig	2008-03-31 15:52:59.000000000 -0300
+++ krandr-0.5.2.1/randr/randrcrtc.cpp	2008-03-31 15:56:11.000000000 -0300
@@ -199,6 +199,10 @@ bool RandRCrtc::applyProposed()
 	for (int i = 0; i < m_connectedOutputs.count(); ++i)
 		kdDebug() << "               - " << m_screen->output(m_connectedOutputs[i])->name() << endl;
 #endif
+
+	// Grab server while messing around
+	XGrabServer(qt_xdisplay());
+
 	RandRMode mode;
 	if (m_proposedRect.size() == m_currentRect.size() && m_proposedRate == m_currentRate)
 	{
@@ -242,7 +246,10 @@ bool RandRCrtc::applyProposed()
 	if (!m_connectedOutputs.count())
 		mode = RandRMode();
 	else if (!mode.isValid())
+	{
+		XUngrabServer(qt_xdisplay());
 		return false;
+	}
 
 	RROutput *outputs = new RROutput[m_connectedOutputs.count()];
 	int connectedCount = m_connectedOutputs.count();
@@ -259,7 +266,10 @@ bool RandRCrtc::applyProposed()
 		{
 			QRect r = QRect(0,0,0,0).unite(m_proposedRect);
 			if (r.width() > m_screen->maxSize().width() || r.height() > m_screen->maxSize().height())
+			{
+				XUngrabServer(qt_xdisplay());
 				return false;
+			}
 
 			// if the desired mode is bigger than the current screen size, first change the 
 			// screen size, and then the crtc size
@@ -267,7 +277,10 @@ bool RandRCrtc::applyProposed()
 			{
 				// try to adjust the screen size
 				if (!adjustScreenSize(r, true))
+				{
+					XUngrabServer(qt_xdisplay());
 					return false;
+				}
 			}
 
 		}
@@ -280,12 +293,18 @@ bool RandRCrtc::applyProposed()
 				// check if the rotated rect is smaller than the max screen size
 				r = m_screen->rect().unite(r);
 				if (r.width() > m_screen->maxSize().width() || r.height() > m_screen->maxSize().height())
+				{
+					XUngrabServer(qt_xdisplay());
 					return false;
+				}
 				
 				// adjust the screen size
 				r = r.unite(m_currentRect);
 				if (!adjustScreenSize(r,true))
+				{
+					XUngrabServer(qt_xdisplay());
 					return false;
+				}
 			}
 		}
 	}
@@ -311,6 +330,7 @@ bool RandRCrtc::applyProposed()
 	}
 
 	m_screen->adjustSize();
+	XUngrabServer(qt_xdisplay());
 	return ret;
 }