Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 50090

Summary: env-update don't process LD_PRELOAD correctly
Product: Gentoo Linux Reporter: Nguyen Thai Ngoc Duy (RETIRED) <pclouds>
Component: [OLD] Core systemAssignee: Portage team <dev-portage>
Status: RESOLVED WONTFIX    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: All   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Patch against 2.0.50-r6

Description Nguyen Thai Ngoc Duy (RETIRED) gentoo-dev 2004-05-05 04:19:13 UTC
I think env-update should concate all LD_PRELOAD variables in env.d. Version 2.0.50-r6 treats LD_PRELOAD as a normal variable instead of a colon-separated variables.
Comment 1 Nguyen Thai Ngoc Duy (RETIRED) gentoo-dev 2004-05-05 04:21:34 UTC
Created attachment 30771 [details, diff]
Patch against 2.0.50-r6
Comment 2 Aron Griffis (RETIRED) gentoo-dev 2004-05-06 18:47:08 UTC
You're going to need to explain more here.  LD_PRELOAD isn't a colon-separated variable, it's space-separated.  See ld.so(8)  What behavior are you trying to achieve?

Re-assigning to portage team since env-update belongs to them.
Comment 3 SpanKY gentoo-dev 2004-05-06 19:36:22 UTC
yeah, like aron said, you're going to have to give a useful reason for why LD_PRELOAD should even bothered being parsed seeing as how it'll rarely get used by most people
Comment 4 Nguyen Thai Ngoc Duy (RETIRED) gentoo-dev 2004-05-07 02:43:18 UTC
Yes, my mistake. LD_PRELOAD is not colon-separated. And it's rarely used. But it is used somewhere (xvnkb, libtrash ...).
Imagine the scenario that somesone sets two LD_PRELOAD in env.d and it doesn't work as expected.
Comment 5 SpanKY gentoo-dev 2004-05-07 17:17:59 UTC
env.d is mostly used by portage packages to install environment pieces so that the final environment can be put together

something like LD_PRELOAD is used on a per-app basis and *never* on a system wide basis

so, unless there is a real need for it (which has yet to be demonstrated) LD_PRELOAD should not be in a global env