diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2011-02-07 01:05:53 -0500 |
|---|---|---|
| committer | Riku Voipio <riku.voipio@nokia.com> | 2011-02-09 10:33:54 +0200 |
| commit | 82a39595f7c70aecfb4649ae5a125147b62131f8 (patch) | |
| tree | b9a52ae6138f83ac740227e7de29b9b0423b9f87 /linux-user | |
| parent | 906c1b8ec8ed8987662e2697af20b9ca19c659b5 (diff) | |
linux-user/FLAT: fix auto-stack sizing
The current auto-stack sizing works like it does on a NOMMU system; the
problem is that this only works if the envp/argv arrays are fairly slim.
On a desktop system, this is rarely the case, and can easily blow past
the stack and into data/text regions as the default stack for FLAT progs
is a mere 4KiB. So rather than rely on the NOMMU calculation (which is
only there because NOMMU can't easily allocate gobs of contiguous mem),
calc the full space actually needed and let the MMU host make space.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Riku Voipio <riku.voipio@nokia.com>
Diffstat (limited to 'linux-user')
| -rw-r--r-- | linux-user/flatload.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/linux-user/flatload.c b/linux-user/flatload.c index 8f9f4a5fcc..d8b4476325 100644 --- a/linux-user/flatload.c +++ b/linux-user/flatload.c @@ -733,8 +733,15 @@ int load_flt_binary(struct linux_binprm * bprm, struct target_pt_regs * regs, * pedantic and include space for the argv/envp array as it may have * a lot of entries. */ -#define TOP_OF_ARGS (TARGET_PAGE_SIZE * MAX_ARG_PAGES - sizeof(void *)) - stack_len = TOP_OF_ARGS - bprm->p; /* the strings */ + stack_len = 0; + for (i = 0; i < bprm->argc; ++i) { + /* the argv strings */ + stack_len += strlen(bprm->argv[i]); + } + for (i = 0; i < bprm->envc; ++i) { + /* the envp strings */ + stack_len += strlen(bprm->envp[i]); + } stack_len += (bprm->argc + 1) * 4; /* the argv array */ stack_len += (bprm->envc + 1) * 4; /* the envp array */ |