rpms/qt/devel qt-x11-free-3.3.7-umask.patch, NONE, 1.1 qt.spec, 1.125, 1.126

fedora-cvs-commits at redhat.com fedora-cvs-commits at redhat.com
Thu Nov 9 20:16:36 UTC 2006


Author: than

Update of /cvs/dist/rpms/qt/devel
In directory cvs.devel.redhat.com:/tmp/cvs-serv11417

Modified Files:
	qt.spec 
Added Files:
	qt-x11-free-3.3.7-umask.patch 
Log Message:
fix #189012, qt settings should be readable for other



qt-x11-free-3.3.7-umask.patch:
 qsettings.cpp |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletion(-)

--- NEW FILE qt-x11-free-3.3.7-umask.patch ---
--- qt-x11-free-3.3.7/src/tools/qsettings.cpp.tn	2006-11-09 20:53:25.000000000 +0100
+++ qt-x11-free-3.3.7/src/tools/qsettings.cpp	2006-11-09 20:59:53.000000000 +0100
@@ -994,7 +994,10 @@
 #endif
 
 	QFile file( filename + ".tmp" );
-	if (! file.open(IO_WriteOnly)) {
+	mode_t old_umask = umask(0022);
+	bool f = file.open(IO_WriteOnly);
+	umask(old_umask);
+	if (! f) {
 
 #ifdef QT_CHECK_STATE
 	    qWarning("QSettings::sync: failed to open '%s' for writing",


Index: qt.spec
===================================================================
RCS file: /cvs/dist/rpms/qt/devel/qt.spec,v
retrieving revision 1.125
retrieving revision 1.126
diff -u -r1.125 -r1.126
--- qt.spec	9 Nov 2006 17:33:00 -0000	1.125
+++ qt.spec	9 Nov 2006 20:16:22 -0000	1.126
@@ -18,6 +18,7 @@
 Patch1: qt-3.3.4-print-CJK.patch
 Patch2: qt-3.0.5-nodebug.patch
 Patch3: qt-3.1.0-makefile.patch
+Patch4: qt-x11-free-3.3.7-umask.patch
 Patch5: qt-x11-free-3.3.6-strip.patch
 Patch7: qt-x11-free-3.3.2-quiet.patch
 Patch8: qt-x11-free-3.3.3-qembed.patch
@@ -247,6 +248,7 @@
 %patch1 -p1 -b .cjk
 %patch2 -p1 -b .ndebug
 %patch3 -p1 -b .makefile
+%patch4 -p1 -b .umask
 %patch5 -p1
 %patch7 -p1 -b .quiet
 %patch8 -p1 -b .qembed
@@ -563,6 +565,7 @@
   thanks Sachin Tawniya, LingNing Zhang for the fixes
 - move html files to devel
 - add sqlite plugin
+- fix #189012, qt settings should be readable for other
 
 * Thu Aug 31 2006 Than Ngo <than at redhat.com> 1:3.3.6-13
 - add missing desktop files




More information about the fedora-cvs-commits mailing list