Bug 114326

Summary: [patch] Segfaults when built with alsa
Product: [Retired] Red Hat Raw Hide Reporter: Enrico Scholz <rh-bugzilla>
Component: esoundAssignee: Havoc Pennington <hp>
Status: CLOSED RAWHIDE QA Contact: David Lawrence <dkl>
Severity: high Docs Contact:
Priority: medium    
Version: 1.0CC: aleksey, cyril.jaquier, danton67, karsten, me, pri.rhl1, wtogami, zgobolos
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-03-19 11:05:57 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 114961    
Attachments:
Description Flags
fixes segfault with alsa-1 none

Description Enrico Scholz 2004-01-26 17:58:28 UTC
Description of problem:

when compiling in alsa support (default, when alsa-lib is installed),
esd coredumps at startup. This might be caused by a changed alsa API,
or unsufficient testing ;)

Attached patch fixes it, although I am not really sure about the
BUFFERSIZE related hunks.


Version-Release number of selected component (if applicable):

esound-0.2.32-2

Comment 1 Enrico Scholz 2004-01-26 17:59:35 UTC
Created attachment 97258 [details]
fixes segfault with alsa-1

Comment 2 Aleksey Nogin 2004-02-20 11:18:11 UTC
Here is a stack trace:

(gdb) bt
#0  snd_pcm_hw_param_set_near (pcm=0x8e7d140, params=0xbfe41270,
var=SNDRV_PCM_HW_PARAM_RATE, val=0xac44, dir=0x0)
    at pcm_params.c:786
#1  0x00ea812d in __snd_pcm_hw_params_set_rate_near (pcm=0xac44,
params=0xac44, val=0xac44, dir=0xac44) at pcm.c:3599
#2  0x00cb5250 in initAlsa () from /usr/lib/libesd.so.0
#3  0x00cb5767 in esd_audio_open () from /usr/lib/libesd.so.0
#4  0x0804a51b in ?? ()


Comment 3 Aleksey Nogin 2004-02-20 11:18:55 UTC
*** Bug 115055 has been marked as a duplicate of this bug. ***

Comment 4 Aleksey Nogin 2004-02-20 11:19:44 UTC
*** Bug 115607 has been marked as a duplicate of this bug. ***

Comment 5 Aleksey Nogin 2004-02-20 11:20:46 UTC
Copying "blocks bug 114961" from the dup bug 115607.

Comment 6 Bill Nottingham 2004-02-23 23:52:05 UTC
*** Bug 116544 has been marked as a duplicate of this bug. ***

Comment 7 Bill Nottingham 2004-02-23 23:52:09 UTC
*** Bug 116585 has been marked as a duplicate of this bug. ***

Comment 8 Aleksey Nogin 2004-02-24 00:05:51 UTC
I believe this was fixed upstream -
http://cvs.gnome.org/bonsai/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvs/gnome&subdir=esound&command=DIFF_FRAMESET&file=audio_alsa09.c&rev2=1.10&rev1=1.9

The corresponding changelog entry is

2004-01-15  Frederic Crozat  <fcrozat>

	* acconfig.h:
	* audio_alsa09.c:
	* configure.in: Use new ALSA PCM API if available
	(fixes from Lukasz Mach <lukasz.pl> and 
	Eddy Mulyono <eddymulyono>)


Comment 9 Aleksey Nogin 2004-02-25 06:32:30 UTC
triage->easyfix, triage->upstream (a patch is attached - see comment
#2 and upstream have already fixed it as well - see comment #8).


Comment 10 Alexander Larsson 2004-03-03 15:42:26 UTC
I built esound-0.2.33-1, which should have that change. Could you test it?

Comment 11 Aleksey Nogin 2004-03-03 17:13:54 UTC
Would be happy to. Where do I get it?

Comment 12 Aleksey Nogin 2004-03-04 22:15:20 UTC
esound-0.2.33-1 - no segfault. 

P.S. See bug 117522.

Comment 13 Warren Togami 2004-03-19 04:58:04 UTC
So this is fixed?  Why is it not closed?