Re: [Qemu-devel] [PATCH][UPDATED] Fix path mangling in linux-user/path.c

2007-06-18 Thread Lauri Leukkunen
On 17/06/07 17:30 +0100, Thiemo Seufer wrote: Lauri Leukkunen wrote: Attached patch fixes the linux-user path mangling code for use with real target root filesystems that have nasty symlinks and lots of files. The old code is terribly slow and can easily end up going through the entire

Re: [Qemu-devel] [PATCH][UPDATED] Fix path mangling in linux-user/path.c

2007-06-18 Thread Lauri Leukkunen
On 18/06/07 09:02 +0300, Lauri Leukkunen wrote: On 17/06/07 17:30 +0100, Thiemo Seufer wrote: Lauri Leukkunen wrote: - It misses to do error handling on malloc() returns - It is broken, an emulated /bin/ls /.. shows it ascends above the root. I'll try to provide an updated

Re: [Qemu-devel] [PATCH][UPDATED] Fix path mangling in linux-user/path.c

2007-05-22 Thread Lauri Leukkunen
On 5/22/07, Kirill A. Shutemov [EMAIL PROTECTED] wrote: On [Tue, 22.05.2007 02:22], Lauri Leukkunen wrote: Attached patch fixes the linux-user path mangling code for use with real target root filesystems that have nasty symlinks and lots of files. The old code is terribly slow and can easily

[Qemu-devel] [PATCH][UPDATED] Fix path mangling in linux-user/path.c

2007-05-21 Thread Lauri Leukkunen
Attached patch fixes the linux-user path mangling code for use with real target root filesystems that have nasty symlinks and lots of files. The old code is terribly slow and can easily end up going through the entire host system /usr hierarchy in a recursive loop. Compared to the previous

[Qemu-devel] [PATCH][UPDATED] Add support to linux-user for dropping LD_PRELOAD

2007-05-21 Thread Lauri Leukkunen
This patch adds an option -drop-ld-preload which results in the target process not having LD_PRELOAD set in its environment. This is useful when running inside environments like scratchbox. This version of the patch doesn't unset(LD_PRELOAD) but modifies the envp argument to loader_exec()

[Qemu-devel] Re: [PATCH][UPDATED] Add support to linux-user for dropping LD_PRELOAD

2007-05-21 Thread Lauri Leukkunen
And here is the actual patch itself. On 5/22/07, Lauri Leukkunen [EMAIL PROTECTED] wrote: This patch adds an option -drop-ld-preload which results in the target process not having LD_PRELOAD set in its environment. This is useful when running inside environments like scratchbox. This version

[Qemu-devel] Re: [PATCH][UPDATED] Add support to linux-user for dropping LD_PRELOAD

2007-05-21 Thread Lauri Leukkunen
And while I'm at it here's the same with properly deallocated environ strings. On 5/22/07, Lauri Leukkunen [EMAIL PROTECTED] wrote: And here is the actual patch itself. On 5/22/07, Lauri Leukkunen [EMAIL PROTECTED] wrote: This patch adds an option -drop-ld-preload which results in the target

[Qemu-devel] [PATCH] Fix path mangling in linux-user/path.c

2007-04-22 Thread Lauri Leukkunen
This patch improves performance at user-mode emulation startup and fixes a nasty looping bug caused by a symlink such as /usr/bin/X11 - ../bin I've tested this using current CVS version on debian/testing on amd64 with arm user-mode target. /lauri Index: path.c

[Qemu-devel] [PATCH] Make user-mode emu drop LD_PRELOAD

2007-04-22 Thread Lauri Leukkunen
This patch makes user-mode emulation drop LD_PRELOAD environment variable by default and adds an option -keep-ld-preload to override this behaviour. /lauri Index: linux-user/main.c === RCS file:

Re: [Qemu-devel] [PATCH] Make user-mode emu drop LD_PRELOAD

2007-04-22 Thread Lauri Leukkunen
On 4/22/07, Paul Brook [EMAIL PROTECTED] wrote: On Sunday 22 April 2007 20:39, Lauri Leukkunen wrote: This patch makes user-mode emulation drop LD_PRELOAD environment variable by default and adds an option -keep-ld-preload to override this behaviour. Why? My use case is for scratchbox 2, I