From fd581d13d941e86ed5069c3fa49537035098e500 Mon Sep 17 00:00:00 2001
From: Radim Kolar <>
Date: Sun, 13 Sep 2009 17:35:42 +0200
Subject: [PATCH] Dont terminate version string by 0x0a

---
 ChangeLog       | 5 +++++
 server/server.c | 1 -
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 907761a..e4875e1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,11 @@ Version NEXT
 	fver now reports if server can process extra data in input packets.
 	  fspclient did that for a long time. This field is used by fspd
 	  2.8.1 beta 21 or newer.
+	Server version string sent as response to CC_VERSION command have
+	  no longer \n appended to it. Old FSP used it but fver client
+	  command can handle server strings without \n for quite a long
+	  time, so i decided to remove it. It is bad idea to terminate
+	  version strings by \n while it is not required by FSP protocol.
 
 Version 2.8.1b25 - 10 Sep 2009
 	GNU autotools build system was entirely replaced by SCons
diff --git a/server/server.c b/server/server.c
index 8ab4a15..041e647 100644
--- a/server/server.c
+++ b/server/server.c
@@ -429,7 +429,6 @@ static void server_show_version (struct sockaddr_in * from, UBUF * ub)
   char buf[UBUF_SPACE], verflags = 0;
   unsigned int xtra = VER_BYTES;
   strcpy(buf, "fspd " PACKAGE_VERSION);
-  strcat(buf, "\n");
 
   if (logging)       verflags |= VER_LOG;
   if (read_only)     verflags |= VER_READONLY;