[Kde-pim] [PATCH] Fix a D-Bus assert in libkleo

[Thomas McGuire]

Hi,

On Monday 14 July 2008 12:40:42 Alex Merry wrote:
> Currently, when configuring the security section of KMail, clicking
> Configure in the Crypto Backends tab and then clicking OK results in an
> assert in D-Bus:
>
> arguments to dbus_message_new_signal() were incorrect, assertion "path !=
> NULL"
>
> Using the backtrace, I found the problem was that QString() was being
> passed as the path to QDBusMessage::createSignal() at line 316 of
> backendconfigwidget.cpp in libkleo/ui.
>
> The attached patch changes it to "/", fixing the assert.
I'd say go ahead and commit (though I know nothing about D-Bus, but if this 
fixes a crash it is probably OK).
If you can, backport to the 4.1 branch as well, if you can't, I'll do that 
eventually.
Thanks for discovering this.

Regards,
Thomas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20080714/8a61f628/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/