Bug 140505

Summary: esddsp should not use absolute paths to libesddsp.so.0 and libesd.so.0
Product: [openSUSE] SUSE LINUX 10.0 Reporter: Peter Astrand <peter>
Component: SoundAssignee: Stanislav Brabec <sbrabec>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Normal    
Priority: P5 - None    
Version: Final   
Target Milestone: ---   
Hardware: x86-64   
OS: SuSE Linux 10.0   
Whiteboard:
Found By: Third Party Developer/Partner Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: esound-0.2.35-64bit_install.patch from Fedora

Description Peter Astrand 2005-12-21 12:35:43 UTC
Currently (as with package esound-0.2.36-3 on Suse 10.0), esddsp sets LD_PRELOAD to /usr/lib64/libesddsp.so.0 and /usr/lib64/libesd.so.0. This is bad, since this means that esddsp will not work for 32-bit applications. A better solution is to not use absolute paths. The Fedora package contains a patch that fixes this. I think Suse should use this patch as well. 

(I think it will be hard to get this patch accepted by the maintainer, since it only works when you are building with a prefix that it's in /etc/ld.so.conf.)
Comment 1 Peter Astrand 2005-12-21 12:37:00 UTC
Created attachment 61543 [details]
esound-0.2.35-64bit_install.patch from Fedora
Comment 2 Takashi Iwai 2006-01-03 19:36:08 UTC
Reassigned to maintainer.
Comment 3 Stanislav Brabec 2006-01-04 11:11:22 UTC
aoss use absolute path with $LIB, which is not vulnerable to current directory attack and works on both platforms:

LD_PRELOAD="$LD_PRELOAD ${exec_prefix}/\$LIB/libaoss.so" exec "$@"

I am not sure, whether $LIB works on all versions of glibc, but in SuSE Linux 10.0 it works.
Comment 4 Stanislav Brabec 2006-01-24 18:02:15 UTC
Added upper mentioned fix and added Suggests: esound-32bit.