[<prev] [next>] [day] [month] [year] [list]
Message-ID: <tip-46ed99d1b7c92920ce9e313152522847647aae4f@git.kernel.org>
Date: Fri, 6 Apr 2012 03:03:49 -0700
From: tip-bot for Emil Goode <emilgoode@...il.com>
To: linux-tip-commits@...r.kernel.org
Cc: linux-kernel@...r.kernel.org, luto@...capital.net, hpa@...or.com,
mingo@...nel.org, emilgoode@...il.com, tglx@...utronix.de
Subject: [tip:x86/urgent] x86: vsyscall:
Use NULL instead 0 for a pointer argument
Commit-ID: 46ed99d1b7c92920ce9e313152522847647aae4f
Gitweb: http://git.kernel.org/tip/46ed99d1b7c92920ce9e313152522847647aae4f
Author: Emil Goode <emilgoode@...il.com>
AuthorDate: Sun, 1 Apr 2012 20:48:04 +0200
Committer: Thomas Gleixner <tglx@...utronix.de>
CommitDate: Fri, 6 Apr 2012 11:49:59 +0200
x86: vsyscall: Use NULL instead 0 for a pointer argument
This patch silences the following sparse warning:
arch/x86/kernel/vsyscall_64.c:250:34:
warning: Using plain integer as NULL pointer
Signed-off-by: Emil Goode <emilgoode@...il.com>
Acked-by: Andy Lutomirski <luto@...capital.net>
Cc: john.stultz@...aro.org
Link: http://lkml.kernel.org/r/1333306084-3776-1-git-send-email-emilgoode@gmail.com
Signed-off-by: Thomas Gleixner <tglx@...utronix.de>
---
arch/x86/kernel/vsyscall_64.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/x86/kernel/vsyscall_64.c b/arch/x86/kernel/vsyscall_64.c
index f386dc4..7515cf0 100644
--- a/arch/x86/kernel/vsyscall_64.c
+++ b/arch/x86/kernel/vsyscall_64.c
@@ -216,9 +216,9 @@ bool emulate_vsyscall(struct pt_regs *regs, unsigned long address)
current_thread_info()->sig_on_uaccess_error = 1;
/*
- * 0 is a valid user pointer (in the access_ok sense) on 32-bit and
+ * NULL is a valid user pointer (in the access_ok sense) on 32-bit and
* 64-bit, so we don't need to special-case it here. For all the
- * vsyscalls, 0 means "don't write anything" not "write it at
+ * vsyscalls, NULL means "don't write anything" not "write it at
* address 0".
*/
ret = -EFAULT;
@@ -247,7 +247,7 @@ bool emulate_vsyscall(struct pt_regs *regs, unsigned long address)
ret = sys_getcpu((unsigned __user *)regs->di,
(unsigned __user *)regs->si,
- 0);
+ NULL);
break;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists