diff options
author | David S. Miller <davem@sunset.davemloft.net> | 2007-09-30 17:00:34 -0700 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-09-30 17:00:34 -0700 |
commit | 8cc8c28a9acdceda0e60519167a052cc3408c5c3 (patch) | |
tree | 06c73a0904e4fa51d95cb51054312e72435e1a87 | |
parent | 9c908f97d0f9aee32fa6d4b0a44c4c5d0ccab2b0 (diff) | |
download | op-kernel-dev-8cc8c28a9acdceda0e60519167a052cc3408c5c3.zip op-kernel-dev-8cc8c28a9acdceda0e60519167a052cc3408c5c3.tar.gz |
[SPARC64]: Fix put_user() calls in binfmt_aout32.c
argv and envp are pointers to u32's in userspace, so don't
try to put_user() a NULL to them.
Aparently gcc-4.2.x now warns about this, and since we use
-Werror for arch/sparc64 code, this breaks the build.
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | arch/sparc64/kernel/binfmt_aout32.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/sparc64/kernel/binfmt_aout32.c b/arch/sparc64/kernel/binfmt_aout32.c index f205fc7..d208cc7 100644 --- a/arch/sparc64/kernel/binfmt_aout32.c +++ b/arch/sparc64/kernel/binfmt_aout32.c @@ -177,7 +177,7 @@ static u32 __user *create_aout32_tables(char __user *p, struct linux_binprm *bpr get_user(c,p++); } while (c); } - put_user(NULL,argv); + put_user(0,argv); current->mm->arg_end = current->mm->env_start = (unsigned long) p; while (envc-->0) { char c; @@ -186,7 +186,7 @@ static u32 __user *create_aout32_tables(char __user *p, struct linux_binprm *bpr get_user(c,p++); } while (c); } - put_user(NULL,envp); + put_user(0,envp); current->mm->env_end = (unsigned long) p; return sp; } |