]> git.pond.sub.org Git - empserver/commitdiff
client: Drop extra newlines from the client's messages
authorMarkus Armbruster <armbru@pond.sub.org>
Sun, 27 Dec 2015 14:37:06 +0000 (15:37 +0100)
committerMarkus Armbruster <armbru@pond.sub.org>
Sat, 8 Jul 2017 17:17:07 +0000 (19:17 +0200)
servercmd()'s argument arg ends with a newline already.  Broken in
commit 8b7d0b9, v4.3.11.

Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
src/client/servcmd.c

index 0c33758f18ab9ec81450b119220a46eedfc91d19..b8264651cd65745d676f865174e578985765425c 100644 (file)
@@ -30,7 +30,7 @@
  *     Dave Pare, 1989
  *     Steve McClure, 1998
  *     Ron Koenderink, 2005
- *     Markus Armbruster, 2005-2010
+ *     Markus Armbruster, 2005-2015
  */
 
 #include <config.h>
@@ -69,7 +69,7 @@ servercmd(int code, char *arg, int len)
     switch (code) {
     case C_PROMPT:
        if (sscanf(arg, "%d %d", &nmin, &nbtu) != 2) {
-           fprintf(stderr, "prompt: bad server prompt %s\n", arg);
+           fprintf(stderr, "prompt: bad server prompt %s", arg);
        }
        snprintf(the_prompt, sizeof(the_prompt), "[%d:%d] Command : ",
                 nmin, nbtu);
@@ -179,7 +179,7 @@ redir_authorized(char *arg, char *attempt, int expected)
     }
 
     if (!seen || (input_to_forget && input_to_forget != seen)) {
-       fprintf(stderr, "WARNING!  Server attempted to %s %s\n",
+       fprintf(stderr, "WARNING!  Server attempted to %s %s",
                attempt, arg);
        return 0;
     }
@@ -241,6 +241,9 @@ dopipe(char *p)
        return;
     }
 
+    /* strip newline */
+    p[strlen(p) - 1] = 0;
+
     redir_is_pipe = 1;
     if ((redir_fp = popen(p, "w")) == NULL) {
        fprintf(stderr, "Can't redirect to pipe %s: %s\n",