Review Request 127237: Fix crash in kmore tools when DBus is missing

Gregor Mi codestruct at posteo.org
Tue Mar 1 19:46:08 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127237/#review93034
-----------------------------------------------------------



Hi Kåre,
thanks for fix. Two things:
1) What do you think about preventing to add the null value into kmtServiceList in the first place in case DBus is missing? I think it does not make sense to have null values in this list at all.
2) If point 1) seems to complicated I suggest to add code comments to make it obvious why the checks were added.

- Gregor Mi


On March 1, 2016, 6:50 a.m., Kåre Särs wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127237/
> -----------------------------------------------------------
> 
> (Updated March 1, 2016, 6:50 a.m.)
> 
> 
> Review request for KDE Frameworks and Gregor Mi.
> 
> 
> Repository: knewstuff
> 
> 
> Description
> -------
> 
> If DBus is missing we get null-pointers in stead of KMoreToolsService*. This patch adds checks for null-pointers in those places that made Kate crash in the project plugin.
> 
> 
> Diffs
> -----
> 
>   src/kmoretools/kmoretoolsmenufactory.cpp 30f4d02 
>   src/kmoretools/kmoretoolspresets.cpp 2405321 
> 
> Diff: https://git.reviewboard.kde.org/r/127237/diff/
> 
> 
> Testing
> -------
> 
> Compiled and run on windows. No crashes in Kate when running without DBus and right-clicking files in the project plugin.
> 
> 
> Thanks,
> 
> Kåre Särs
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160301/3b458d01/attachment.html>


More information about the Kde-frameworks-devel mailing list