From: Denys Vlasenko <vda.linux@googlemail.com>
Date: Thu, 4 Nov 2010 22:22:40 +0000 (+0100)
Subject: powertop: replace erroneous \n with \0; make numberic conversion more robust
X-Git-Tag: 1_18_0~25^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dcb163aa8e7808b59aef3d3319c707e7d9ac4ce4;p=platform%2Fupstream%2Fbusybox.git

powertop: replace erroneous \n with \0; make numberic conversion more robust

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---

diff --git a/procps/powertop.c b/procps/powertop.c
index 4c3c356..87efbe8 100644
--- a/procps/powertop.c
+++ b/procps/powertop.c
@@ -323,18 +323,16 @@ static void process_irq_counts(void)
 		 */
 		*p = '\0';
 		/* Deal with non-maskable interrupts -- make up fake numbers */
-		nr = index_in_strings("NMI\0RES\nCAL\0TLB\0TRM\0THR\0SPU\0", buf);
-		if (nr != -1) {
+		nr = index_in_strings("NMI\0RES\0CAL\0TLB\0TRM\0THR\0SPU\0", buf);
+		if (nr >= 0) {
 			nr += 20000;
 		} else {
 			/* bb_strtou doesn't eat leading spaces, using strtoul */
+			errno = 0;
 			nr = strtoul(buf, NULL, 10);
+			if (errno)
+				continue;
 		}
-		*p = ':';
-
-		if (nr == -1)
-			continue;
-
 		p++;
 		/*  0:  143646045  153901007   IO-APIC-edge      timer
 		 *    ^