Review Request 117779: fix crash when textureNode->texture() is null

[Alexander Richardson]

> On April 25, 2014, 11:44 p.m., Thomas Lübking wrote:
> > src/declarativeimports/core/windowthumbnail.cpp, line 231
> > <https://git.reviewboard.kde.org/r/117779/diff/1/?file=268344#file268344line231>
> >
> >     try before this call.

Okay, just thought I'd exit early in that case and not bother with the other stuff. However is this an appropriate fix, or should it be done somewhere else?


- Alexander


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


On April 25, 2014, 11:41 p.m., Alexander Richardson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117779/
> -----------------------------------------------------------
> 
> (Updated April 25, 2014, 11:41 p.m.)
> 
> 
> Review request for kde-workspace and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> fix crash when textureNode->texture() is null
> 
> I get this crash very frequently on my system. This is probably only fixing
> the symptom and not the underlying issue, however at least plasma no longer
> crashes every few minutes
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/windowthumbnail.cpp 59255f75994adb96b30fb503c759b2e9110ab708 
> 
> Diff: https://git.reviewboard.kde.org/r/117779/diff/
> 
> 
> Testing
> -------
> 
> No longer crashes
> 
> 
> Thanks,
> 
> Alexander Richardson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140425/6750d805/attachment.htm>