Summary: | knotify crash when a network change occurs | ||
---|---|---|---|
Product: | [Frameworks and Libraries] Phonon | Reporter: | Felipe Sologuren <sologuren> |
Component: | KDE platform plugin | Assignee: | Harald Sitter <sitter> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | martin.sandsmark, myriam |
Priority: | NOR | Keywords: | drkonqi |
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/phonon/ec5f8fc79338e15b69d24fd89b9fccf93049624f | Version Fixed In: | 4.8.3 |
Description
Felipe Sologuren
2014-11-26 16:58:43 UTC
Git commit ec5f8fc79338e15b69d24fd89b9fccf93049624f by Harald Sitter. Committed on 26/11/2014 at 22:28. Pushed by sitter into branch '4.8'. do not assume we have a backend when setting an output device random suggestion du jour: since debianesque distros are the only ones allowing running without a backend (to my knowledge anyway), this fix should be adopted there at the earliest convenient time. this change prevents a crash when run without a backend and getting device changes as the Iface handling later on would directly call the cast output without checking its validity. since the cast would always be null when there is no backend we can just as well save time and return early. note: this is ultimately fallout from too excessive init() setup when there is no backend, so the ideal fix would have been to abort the setup there. given the limited testing exposure of alsa+phononserver+phonon this less invasive fix is deemed more suitable for the existing code base though. (this lineup only happens with phononserver in the picture and that bugger has already been killed off for plasma5) CCMAIL: pkg-kde-talk@lists.alioth.debian.org M +3 -0 phonon/audiooutput.cpp http://commits.kde.org/phonon/ec5f8fc79338e15b69d24fd89b9fccf93049624f |