Bug#782692: kmod: spurious spaces in lsmod output

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- Forwarded message from Santiago Vila <sanvila@xxxxxxx> -----

From: Santiago Vila <sanvila@xxxxxxx>
To: submit@xxxxxxxxxxxxxxx
Subject: Bug#782692: kmod: spurious spaces in lsmod output

Package: kmod
Version: 18-3
Severity: minor

While using "localyesconfig" to build a custom kernel I noticed that lsmod
output now has trailing spaces when the list of "Used by" modules is empty.

The following (untested but trivial) patch should fix it. It just delays
the space to the point where we are sure that there are more things to print.

Thanks.

diff --git a/tools/lsmod.c b/tools/lsmod.c
index e953c7e..c1bab7f 100644
--- a/tools/lsmod.c
+++ b/tools/lsmod.c
@@ -64,15 +64,17 @@ static int do_lsmod(int argc, char *argv[])
 		struct kmod_list *holders, *hitr;
 		int first = 1;
 
-		printf("%-19s %8ld  %d ", name, size, use_count);
+		printf("%-19s %8ld  %d", name, size, use_count);
 		holders = kmod_module_get_holders(mod);
 		kmod_list_foreach(hitr, holders) {
 			struct kmod_module *hm = kmod_module_get_module(hitr);
 
 			if (!first)
 				putchar(',');
-			else
+			else {
+				putchar(' ');
 				first = 0;
+			}
 
 			fputs(kmod_module_get_name(hm), stdout);
 			kmod_module_unref(hm);

----- End forwarded message -----

-- 
ciao,
Marco
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux