Hello community, here is the log from the commit of package polkit-qt-1 for openSUSE:Factory checked in at Fri Jun 10 11:40:10 CEST 2011.
-------- --- KDE/polkit-qt-1/polkit-qt-1.changes 2011-04-19 00:01:44.000000000 +0200 +++ /mounts/work_src_done/STABLE/polkit-qt-1/polkit-qt-1.changes 2011-06-10 10:27:34.000000000 +0200 @@ -1,0 +2,5 @@ +Fri Jun 10 10:27:08 CEST 2011 - dmuel...@suse.de + +- fix crash when ConsoleKit is unavailable + +------------------------------------------------------------------- calling whatdependson for head-i586 New: ---- ck-avoid-crash.diff ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ polkit-qt-1.spec ++++++ --- /var/tmp/diff_new_pack.nPca4M/_old 2011-06-10 11:39:40.000000000 +0200 +++ /var/tmp/diff_new_pack.nPca4M/_new 2011-06-10 11:39:40.000000000 +0200 @@ -30,12 +30,13 @@ Group: Development/Libraries/KDE Summary: PolicyKit Library Qt Bindings Version: 0.99.1 -Release: 8 +Release: 10 Url: http://api.kde.org/kdesupport-api/kdesupport-apidocs/polkit-qt/html/ BuildRoot: %{_tmppath}/%{name}-%{version}-build # ftp://ftp.kde.org/pub/kde/stable/apps/KDE4.x/admin/ Source0: %{name}-%{version}.tar.bz2 Source99: baselibs.conf +Patch0: ck-avoid-crash.diff %description Polkit-qt-1 aims to make it easy for Qt developers to take advantage of @@ -95,6 +96,7 @@ %prep %setup +%patch0 %build %cmake_kde4 -d build ++++++ ck-avoid-crash.diff ++++++ --- core/polkitqt1-authority.cpp +++ core/polkitqt1-authority.cpp @@ -219,12 +219,14 @@ // then we need to extract all seats from ConsoleKit QDBusMessage msg = QDBusMessage::createMethodCall(consoleKitService, consoleKitManagerPath, consoleKitManagerInterface, "GetSeats"); msg = QDBusConnection::systemBus().call(msg); - // this method returns a list with present seats - QList<QString> seats; - qVariantValue<QDBusArgument> (msg.arguments()[0]) >> seats; - // it can be multiple seats present so connect all their signals - Q_FOREACH(const QString &seat, seats) { - seatSignalsConnect(seat); + if (!msg.arguments().isEmpty()) { + // this method returns a list with present seats + QList<QString> seats; + qVariantValue<QDBusArgument> (msg.arguments()[0]) >> seats; + // it can be multiple seats present so connect all their signals + Q_FOREACH(const QString &seat, seats) { + seatSignalsConnect(seat); + } } } ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Remember to have fun... -- To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org For additional commands, e-mail: opensuse-commit+h...@opensuse.org