Bug 804363 - kmix crashes when pulseaudio exits
Summary: kmix crashes when pulseaudio exits
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: kdemultimedia
Version: 17
Hardware: All
OS: All
medium
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F17Blocker-kde
TreeView+ depends on / blocked
 
Reported: 2012-03-18 09:15 UTC by Stefan Becker
Modified: 2012-05-04 19:42 UTC (History)
14 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-03 18:15:26 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
screenshot (357.58 KB, image/png)
2012-03-18 14:24 UTC, Dan Mashal
no flags Details


Links
System ID Private Priority Status Summary Last Updated
KDE Software Compilation 290742 0 None None None Never

Description Stefan Becker 2012-03-18 09:15:17 UTC
Description of problem:

This seems to be a new problem with KDE 4.8.x: when pulseaudio exits then kmix crashes with segmentation fault.

Version-Release number of selected component (if applicable):
kdemultimedia-kmix-4.8.1-1.fc16.x86_64
pulseaudio-0.9.23-1.fc16.x86_64

How reproducible: Always


Steps to Reproduce:
1. login to KDE session
2. in a shell enter "pactl exit" to stop pulseaudio

  
Actual results:
KDE Crash handler opens showing "Executable: kmix PID: 3140 Signal: Segmentation fault (11)"

Expected results: no crash


Additional info:
My example shows a voluntary pulseaudio exit. This also works with "kill" and I'm pretty sure it happens also if pulseaudio itself should crash.

Comment 1 Stefan Becker 2012-03-18 09:35:24 UTC
Reproduced on my second system with Fedora 16 and KDE 4.8.1.

Comment 2 Stefan Becker 2012-03-18 14:07:08 UTC
Also reproduced on F17:

kdemultimedia-kmix-4.8.1-1.fc17.x86_64
pulseaudio-1.1-8.fc17.x86_64

Comment 3 Dan Mashal 2012-03-18 14:24:10 UTC
Created attachment 570893 [details]
screenshot

Comment 4 Dan Mashal 2012-03-18 14:25:28 UTC
I was able to reproduce this.

Comment 5 Sergio Basto 2012-03-27 21:11:00 UTC
I got this kmix crashs also

Comment 6 Stefan Becker 2012-03-29 11:00:22 UTC
I also noticed that kmix crashes when you change the PulseAudio configuration, e.g. in pavucontrol -> Tab Configuration: switch "Internal Audio" on or off.

Comment 7 Andreas Bierfert 2012-04-04 20:03:23 UTC
Same here on today's f17.

Comment 8 Sergio Basto 2012-04-04 21:08:31 UTC
Hi , reported also here: 
https://bugs.kde.org/show_bug.cgi?id=290742 
but don't find where change external tracker here .

Comment 9 Stefan Becker 2012-04-05 08:27:29 UTC
OK, I did the following

 - updated my machine to latest F17 KDE 4.8.2 packages
 - downloaded & unpacked kdemultimedia-4.8.2-1.fc17.src.rpm
 - replaced the kmix directory inside kdemultimedia-4.8.2 with the contents of git HEAD from here

   git://anongit.kde.org/kmix

   commit 9b4872a60555c6352078e7f11809006999d151d6
   Author: Christian Esken <esken>
   Date:   Wed Mar 21 23:09:15 2012 +0000

    More MixDevice reference cleanups, and fix some minor memory leaks.
    CCBUGS: 290742

 - ran the kdemultimedia build & install
 - used the output of "rpm -ql kdemultimedia-kmix" to create a tar archive from the installation directory that only contains the kmix files
 - killed kmix process
 - as root: tar -C / -xvf <the tar archive>
 - restarted kmix

After that I tested

 - pactl exit
 - pavucontrol: switch internal audio from OFF to Stereo Output
 - pavucontrol: switch internal audio from Stereo Output to OFF
 - suspend-to-ram & resume

All of these were confirmed cases for kmix crashes. Now everything work fine.

Would it be acceptable to put kmix git HEAD as mega-patch into kdemultimedia 4.8.2 RPM package?

Comment 10 Rex Dieter 2012-04-05 14:06:58 UTC
blindly rabase to HEAD?  probably not, but if we can find the particular commits/fixes, then backporting to 4.8 branch and our packaging is certainly doable.

Comment 11 Stefan Becker 2012-04-05 15:11:08 UTC
That's exactly the problem. According to the KDE bug comments the author started with something small which turned into something invasive, which he himself feels not comfortable to backport to 4.8.

git HEAD seems to compile without problems against 4.8 kdelibs, so I think we'll have to bite the bullet on this one.

Comment 12 Stefan Becker 2012-04-05 15:11:52 UTC
That's kmix git HEAD, nothing else from kdemultimedia BTW.

Comment 13 Rex Dieter 2012-04-05 16:15:30 UTC
Where did upstream say they weren't willing to backport/fix 4.8?

Comment 14 Rex Dieter 2012-04-24 12:31:15 UTC
yay,
http://websvn.kde.org/?view=rev&revision=1291315

I can confirm that this upstream patch set seems to fix it for me, I'll work to get this backported asap.

Comment 15 Fedora Update System 2012-04-24 14:10:24 UTC
kdemultimedia-4.8.2-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/kdemultimedia-4.8.2-4.fc17

Comment 16 Rex Dieter 2012-04-24 14:11:30 UTC
nominating f17blocker, mostly so we can get it as a NTH since we do have a fix (candidate) in hand.

Comment 17 Fedora Update System 2012-04-25 04:47:51 UTC
Package kdemultimedia-4.8.2-4.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kdemultimedia-4.8.2-4.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-6596/kdemultimedia-4.8.2-4.fc17
then log in and leave karma (feedback).

Comment 18 Stefan Becker 2012-04-26 08:24:43 UTC
After updating kmix no longer crashes.

Comment 19 Tim Flink 2012-04-26 17:38:17 UTC
How common are pa crashes/exits? That seems like something that would be rather rare in the final release.

Since this could be fixed with an update for an installed system and unless I'm way off on the frequency of occurrence, I don't think that this would be terribly common on livecds and thus, not blocker or nth worthy.

-1 blocker, -1 nth

Comment 20 Sergio Basto 2012-04-26 19:07:42 UTC
And on F16 , we don't have nothing on testing ?

Comment 21 Fedora Update System 2012-04-26 19:49:59 UTC
kdemultimedia-4.8.2-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/kdemultimedia-4.8.2-4.fc16

Comment 22 Rex Dieter 2012-04-26 19:51:43 UTC
OK, I caved in ^^ see Comment #21, please test/karma, frolick, and enjoy.

Comment 23 Rex Dieter 2012-04-28 17:14:03 UTC
Wrt being this being F17 NTH-worthy, let me expand on this bug's impact, see upstream bug https://bugs.kde.org/show_bug.cgi?id=290742
and the many plethora (> 100) of dup'd reported bugs.  There's crashes beyond the relatively simple (pulseaudio exit) reproducer here.

Comment 24 Fedora Update System 2012-05-01 00:50:14 UTC
kdemultimedia-4.8.2-4.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 25 Fedora Update System 2012-05-02 04:43:10 UTC
kdemultimedia-4.8.2-4.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 26 Adam Williamson 2012-05-03 18:15:26 UTC
Dunno why Bodhi didn't close this, if the updates are pushed.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 27 Rex Dieter 2012-05-03 18:22:32 UTC
thanks.

Comment 28 Sergio Basto 2012-05-04 08:31:09 UTC
hi ,
I have a new problem with kmix and events sounds , as I can understand event sounds are binary (0 or 1 ) but also have ruler, no I can't change volume of event sounds after muted for a while .
(tested in F16 and 17)
Thanks

Comment 29 Kevin Kofler 2012-05-04 19:42:13 UTC
Please file separate bug reports for separate bugs.


Note You need to log in before you can comment on or make changes to this bug.