From fd1ae14fa3f13133923fa16ac593600a7410c57a Mon Sep 17 00:00:00 2001 From: Sascha Kruse Date: Fri, 8 Mar 2013 08:25:00 +0100 Subject: [PATCH] don't set buf[len - 1] to '\0' when reading dmenu input see github issue #94 Since the buffer is initialized to 0, we don't need to explicitly set the terminating null-byte. Thanks to losciamano --- menu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/menu.c b/menu.c index 3943993..338f848 100644 --- a/menu.c +++ b/menu.c @@ -165,7 +165,7 @@ void context_menu(void) if (!dmenu_input) return; - char buf[1024]; + char buf[1024] = {0}; int child_io[2]; int parent_io[2]; if (pipe(child_io) != 0) { @@ -204,7 +204,6 @@ void context_menu(void) size_t len = read(parent_io[0], buf, 1023); if (len == 0) return; - buf[len - 1] = '\0'; int status; waitpid(pid, &status, 0);