diff options
author | Stefani Seibold <stefani@seibold.net> | 2014-03-23 12:38:14 -0400 |
---|---|---|
committer | H. Peter Anvin <hpa@linux.intel.com> | 2014-03-24 12:31:23 -0400 |
commit | 645a387ecbdb4aa78c8451a66416340616134537 (patch) | |
tree | 48a948fbd4cf3cb3d18875ea7054b93564ae0724 | |
parent | 3c1b63b9e4862fb16352a0646439c2dd6d9e0e5c (diff) |
x86, vdso: Fix size of get_unmapped_area()
The size of the reserved memory for a 32 bit vdso must be the size of the
32 bit vDSO in pages + HPET page + VVAR page.
One page is not enough for this. Grrrr.... silly copy and paste bug,
was right in previous patch.
Signed-off-by: Stefani Seibold <stefani@seibold.net>
Cc: Andy Lutomirski <luto@amacapital.net>
Link: http://lkml.kernel.org/r/1395592694-20571-1-git-send-email-stefani@seibold.net
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
-rw-r--r-- | arch/x86/vdso/vdso32-setup.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/arch/x86/vdso/vdso32-setup.c b/arch/x86/vdso/vdso32-setup.c index 791c1cb822c6..00348980a3a6 100644 --- a/arch/x86/vdso/vdso32-setup.c +++ b/arch/x86/vdso/vdso32-setup.c | |||
@@ -165,12 +165,14 @@ int arch_setup_additional_pages(struct linux_binprm *bprm, int uses_interp) | |||
165 | 165 | ||
166 | down_write(&mm->mmap_sem); | 166 | down_write(&mm->mmap_sem); |
167 | 167 | ||
168 | addr = get_unmapped_area(NULL, 0, PAGE_SIZE, 0, 0); | 168 | addr = get_unmapped_area(NULL, 0, vdso32_size + VDSO_OFFSET(VDSO_PREV_PAGES), 0, 0); |
169 | if (IS_ERR_VALUE(addr)) { | 169 | if (IS_ERR_VALUE(addr)) { |
170 | ret = addr; | 170 | ret = addr; |
171 | goto up_fail; | 171 | goto up_fail; |
172 | } | 172 | } |
173 | 173 | ||
174 | addr += VDSO_OFFSET(VDSO_PREV_PAGES); | ||
175 | |||
174 | current->mm->context.vdso = (void *)addr; | 176 | current->mm->context.vdso = (void *)addr; |
175 | 177 | ||
176 | /* | 178 | /* |