Erik Pilsits

Thank you for the reply. I have a Verizon Galaxy Nexus running AOKP b23 (4.0.3). The segmentation fault is definitely still an issue on ICS and ICS ports, failing on...

I'd like to test a version of your patch that inherits LD_LIBRARY_PATH only. It seems that is the only one that matters, so it would be nice if that's all...

I found the target setting and compiled for armv5, thanks for that! However the patch doesn't seem to be working. After running su there is still no LD_LIBRARY_PATH environment string,...

> move call to populate_environment(ctx) from line 378 to line 535 where it shall look like I think our line numbers are a bit off. In the current su.c, the...

Ok, I fixed it. I moved the call to populate_environment(ctx) down about 9 lines, just above #define PARG(arg), and left it as is, not (&ctx). Tested with no errors or...

Thanks for the clarification. I got your patch to work, so it's good for now until you finalize it. Just a question... should you be able to 'su 1000' from...

Well the reason I ask that, is that 'su 1000' from a non-root shell fails. ``` E/su (10805): sudb - Opening database E/su (10805): sudb - Database opened E/su (10805):...

Heh, thanks :) I think it's a real bug though, I did a fresh git clone when you merged master into ics. But I can do a fresh clone when...

Ok, so I re-cloned ChainsDD's gc-ics branch, now I see your call to seteuid in the get_parent_env function. However after incorporating the above change to move the call to populate_environment,...

Bah, nevermind. I just realized your patch supersedes the earlier one. Successful test just now, inheriting LD_LIBRARY_PATH works and so does 'su 1000' from non-root shell. Nice :)