lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:	Sat,  3 Jan 2015 13:10:05 +0600
From:	Alexander Kuleshov <kuleshovmail@...il.com>
To:	"H. Peter Anvin" <hpa@...or.com>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	Ingo Molnar <mingo@...hat.com>, x86@...nel.org,
	linux-kernel@...r.kernel.org,
	Alexander Kuleshov <kuleshovmail@...il.com>
Subject: [PATCH v2] x86,setup: Rename myisspace to isspace and move it to ctype.h

Signed-off-by: Alexander Kuleshov <kuleshovmail@...il.com>
---
 arch/x86/boot/cmdline.c | 19 +++++++------------
 arch/x86/boot/ctype.h   |  5 +++++
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/x86/boot/cmdline.c b/arch/x86/boot/cmdline.c
index 625d21b..bc1e25f 100644
--- a/arch/x86/boot/cmdline.c
+++ b/arch/x86/boot/cmdline.c
@@ -14,11 +14,6 @@
 
 #include "boot.h"
 
-static inline int myisspace(u8 c)
-{
-	return c <= ' ';	/* Close enough approximation */
-}
-
 /*
  * Find a non-boolean option, that is, "option=argument".  In accordance
  * with standard Linux practice, if this option is repeated, this returns
@@ -50,7 +45,7 @@ int __cmdline_find_option(unsigned long cmdline_ptr, const char *option, char *b
 	while (cptr < 0x10000 && (c = rdfs8(cptr++))) {
 		switch (state) {
 		case st_wordstart:
-			if (myisspace(c))
+			if (isspace(c))
 				break;
 
 			/* else */
@@ -63,7 +58,7 @@ int __cmdline_find_option(unsigned long cmdline_ptr, const char *option, char *b
 				len = 0;
 				bufptr = buffer;
 				state = st_bufcpy;
-			} else if (myisspace(c)) {
+			} else if (isspace(c)) {
 				state = st_wordstart;
 			} else if (c != *opptr++) {
 				state = st_wordskip;
@@ -71,12 +66,12 @@ int __cmdline_find_option(unsigned long cmdline_ptr, const char *option, char *b
 			break;
 
 		case st_wordskip:
-			if (myisspace(c))
+			if (isspace(c))
 				state = st_wordstart;
 			break;
 
 		case st_bufcpy:
-			if (myisspace(c)) {
+			if (isspace(c)) {
 				state = st_wordstart;
 			} else {
 				if (len < bufsize-1)
@@ -125,7 +120,7 @@ int __cmdline_find_option_bool(unsigned long cmdline_ptr, const char *option)
 		case st_wordstart:
 			if (!c)
 				return 0;
-			else if (myisspace(c))
+			else if (isspace(c))
 				break;
 
 			state = st_wordcmp;
@@ -135,7 +130,7 @@ int __cmdline_find_option_bool(unsigned long cmdline_ptr, const char *option)
 
 		case st_wordcmp:
 			if (!*opptr)
-				if (!c || myisspace(c))
+				if (!c || isspace(c))
 					return wstart;
 				else
 					state = st_wordskip;
@@ -148,7 +143,7 @@ int __cmdline_find_option_bool(unsigned long cmdline_ptr, const char *option)
 		case st_wordskip:
 			if (!c)
 				return 0;
-			else if (myisspace(c))
+			else if (isspace(c))
 				state = st_wordstart;
 			break;
 		}
diff --git a/arch/x86/boot/ctype.h b/arch/x86/boot/ctype.h
index 25e1340..eb909b5 100644
--- a/arch/x86/boot/ctype.h
+++ b/arch/x86/boot/ctype.h
@@ -18,4 +18,9 @@ static inline int isxdigit(int ch)
 	return (ch >= 'A') && (ch <= 'F');
 }
 
+static inline int isspace(u8 c)
+{
+	return c <= ' ';	/* Close enough approximation */
+}
+
 #endif
-- 
2.2.1.202.g44ae4ee.dirty

--
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