Summary: | Akregator crashed when clicking a article | ||
---|---|---|---|
Product: | [Frameworks and Libraries] Phonon | Reporter: | Stefan Böhmann <kde> |
Component: | general | Assignee: | aKregator haxx0rz <akregator-devel> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | cfeck, martin.sandsmark, myriam, robert |
Priority: | NOR | ||
Version: | 4.5-git | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 4.4.3 | |
Attachments: | Fix phonon videoplayer crash |
Description
Stefan Böhmann
2010-07-28 22:53:29 UTC
I'm seeing this too. The problem seems to be an uninitialized pointer (q_ptr) in the PhononVideoPlayerPrivate class -- a variable which seems to really just be an alias for the "this" pointer. I am pretty sure that the problem is in the constructor. There are two constructors. One say: d->q_ptr = this; The other says d_ptr->q_ptr = this; Note the subtle _ptr difference. The second form is incorrect, because the Q_D macro hasn't been used anywhere. When I modify the second constructor so that it uses d->q_ptr = this, everything works fine and I can read the BlueDevil review article. I guess technically phonon is upstream now? How can we best contact the qt-phonon folks? Created attachment 49798 [details]
Fix phonon videoplayer crash
thanks for the patch, it has been committed. *** Bug 246070 has been marked as a duplicate of this bug. *** I can confirm that f8e430b5b465919d8e58f77744ad058e4aa5ad1b fixes the issue for me (reported as Konqueror bug 246070) Setting status correctly. |