[Nepomuk] Review Request: Fix bug 311222 (crash)

[Vishesh Handa]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107639/#review23176
-----------------------------------------------------------

Ship it!


I'm trying to understand when would a case occur that m_client = 0. m_client can never change once it has been set in the Model constructor, so it should always have some value, unless it was explicitly set a 0. In that case this crash will still happen.

I think 690e59e13a9ba783f523f6129021fd4d5811642d fixes the issue, where the client connection is never deleted.

Both ways, this should go in.

If you want you can push this into the 2.8 branch and then merge 2.8 into master. Soprano releases are done when requested. I think we will need a new one for KDE RC1. I'll have to ask Sebastian to release a new version.

- Vishesh Handa


On Dec. 9, 2012, 3:44 a.m., Simeon Bird wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107639/
> -----------------------------------------------------------
> 
> (Updated Dec. 9, 2012, 3:44 a.m.)
> 
> 
> Review request for Nepomuk, Soprano, Vishesh Handa, and Sebastian Trueg.
> 
> 
> Description
> -------
> 
> Fix crash on deletion if nepomuk has been disabled, as per bug 311222.
> 
> (Should I submit this to master, or 2.8 branch as well? Where is the next soprano release coming from?)
> 
> 
> This addresses bug 311222.
>     http://bugs.kde.org/show_bug.cgi?id=311222
> 
> 
> Diffs
> -----
> 
>   client/clientmodel.cpp bdcbed3 
> 
> Diff: http://git.reviewboard.kde.org/r/107639/diff/
> 
> 
> Testing
> -------
> 
> Yup, it fixes the bug.
> 
> 
> Thanks,
> 
> Simeon Bird
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20121209/2bd62ca0/attachment-0001.html>