uname: add support for -i and -o, fix printing of unknown -p

 value with -a option

function                                             old     new   delta
uname_main                                           166     185     +19
utsname_offset                                        12      16      +4
options                                              621     623      +2
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 3/0 up/down: 25/0)               Total: 25 bytes

1 files changed, 72 insertions, 33 deletions

diff --git a/coreutils/uname.c b/coreutils/uname.c
index e28285c..8e51aa8 100644
--- a/coreutils/uname.c
+++ b/coreutils/uname.c
@@ -9,25 +9,43 @@
 /* http://www.opengroup.org/onlinepubs/007904975/utilities/uname.html */
 
 /* Option		Example
+ * -s, --sysname	SunOS
+ * -n, --nodename	rocky8
+ * -r, --release	4.0
+ * -v, --version
+ * -m, --machine	sun
+ * -a, --all		SunOS rocky8 4.0  sun
+ *
+ * The default behavior is equivalent to '-s'.
+ *
+ * David MacKenzie <djm@gnu.ai.mit.edu>
+ *
+ * GNU coreutils 6.10:
+ * Option:                      struct   Example(s):
+ *                              utsname
+ *                              field:
+ * -s, --kernel-name            sysname  Linux
+ * -n, --nodename               nodename localhost.localdomain
+ * -r, --kernel-release         release  2.6.29
+ * -v, --kernel-version         version  #1 SMP Sun Jan 11 20:52:37 EST 2009
+ * -m, --machine                machine  x86_64   i686
+ * -p, --processor              (none)   x86_64   i686
+ * -i, --hardware-platform      (none)   x86_64   i386
+ *      NB: vanilla coreutils reports "unknown" -p and -i,
+ *      x86_64 and i686/i386 shown above are Fedora's inventions.
+ * -o, --operating-system       (none)   GNU/Linux
+ * -a, --all: all of the above, in the order shown.
+ *      If -p or -i is not known, don't show them
+ */
 
-   -s, --sysname	SunOS
-   -n, --nodename	rocky8
-   -r, --release	4.0
-   -v, --version
-   -m, --machine	sun
-   -a, --all		SunOS rocky8 4.0  sun
-
-   The default behavior is equivalent to '-s'.
-
-   David MacKenzie <djm@gnu.ai.mit.edu> */
-
-/* Busyboxed by Erik Andersen */
-
-/* Further size reductions by Glenn McGrath and Manuel Novoa III. */
-
-/* Mar 16, 2003      Manuel Novoa III   (mjn3@codepoet.org)
+/* Busyboxed by Erik Andersen
  *
- * Now does proper error checking on i/o.  Plus some further space savings.
+ * Before 2003: Glenn McGrath and Manuel Novoa III
+ *  Further size reductions.
+ * Mar 16, 2003: Manuel Novoa III (mjn3@codepoet.org)
+ *  Now does proper error checking on i/o.  Plus some further space savings.
+ * Jan 2009:
+ *  Fix handling of -a to not print "unknown", add -o and -i support.
  */
 
 #include <sys/utsname.h>
@@ -35,17 +53,21 @@
 
 typedef struct {
 	struct utsname name;
-	char processor[8];			/* for "unknown" */
+	char processor[sizeof(((struct utsname*)NULL)->machine)];
+	char platform[sizeof(((struct utsname*)NULL)->machine)];
+	char os[sizeof("GNU/Linux")];
 } uname_info_t;
 
-static const char options[] ALIGN1 = "snrvmpa";
+static const char options[] ALIGN1 = "snrvmpioa";
 static const unsigned short utsname_offset[] = {
-	offsetof(uname_info_t, name.sysname),
-	offsetof(uname_info_t, name.nodename),
-	offsetof(uname_info_t, name.release),
-	offsetof(uname_info_t, name.version),
-	offsetof(uname_info_t, name.machine),
-	offsetof(uname_info_t, processor)
+	offsetof(uname_info_t, name.sysname), /* -s */
+	offsetof(uname_info_t, name.nodename), /* -n */
+	offsetof(uname_info_t, name.release), /* -r */
+	offsetof(uname_info_t, name.version), /* -v */
+	offsetof(uname_info_t, name.machine), /* -m */
+	offsetof(uname_info_t, processor), /* -p */
+	offsetof(uname_info_t, platform), /* -i */
+	offsetof(uname_info_t, os), /* -o */
 };
 
 int uname_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
@@ -55,6 +77,8 @@ int uname_main(int argc UNUSED_PARAM, char **argv)
 #if defined(__sparc__) && defined(__linux__)
 	char *fake_sparc = getenv("FAKE_SPARC");
 #endif
+	const char *unknown_str = "unknown";
+	const char *fmt;
 	const unsigned short *delta;
 	char toprint;
 
@@ -64,8 +88,9 @@ int uname_main(int argc UNUSED_PARAM, char **argv)
 		bb_show_usage();
 	}
 
-	if (toprint & (1 << 6)) { /* -a => all opts on */
-		toprint = 0x3f;
+	if (toprint & (1 << 8)) { /* -a => all opts on */
+		toprint = (1 << 8) - 1;
+		unknown_str = ""; /* -a does not print unknown fields */
 	}
 
 	if (toprint == 0) { /* no opts => -s (sysname) */
@@ -79,16 +104,30 @@ int uname_main(int argc UNUSED_PARAM, char **argv)
 		strcpy(uname_info.name.machine, "sparc");
 	}
 #endif
-
-	strcpy(uname_info.processor, "unknown");
+	strcpy(uname_info.processor, unknown_str);
+	strcpy(uname_info.platform, unknown_str);
+	strcpy(uname_info.os, "GNU/Linux");
+#if 0
+	/* Fedora does something like this */
+	strcpy(uname_info.processor, uname_info.name.machine);
+	strcpy(uname_info.platform, uname_info.name.machine);
+	if (uname_info.platform[0] == 'i'
+	 && uname_info.platform[1]
+	 && uname_info.platform[2] == '8'
+	 && uname_info.platform[3] == '6'
+	) {
+		uname_info.platform[1] = '3';
+	}
+#endif
 
 	delta = utsname_offset;
+	fmt = " %s" + 1;
 	do {
 		if (toprint & 1) {
-			/* printf would not be safe here */
-			fputs((char *)(&uname_info) + *delta, stdout);
-			if (toprint > 1) {
-				bb_putchar(' ');
+			const char *p = (char *)(&uname_info) + *delta;
+			if (p[0]) {
+				printf(fmt, p);
+				fmt = " %s";
 			}
 		}
 		++delta;