CONF STOREVAL fix off-by-one string termination

The fact that this makes the server crash hard, proves that
this command hasn't seen any use.  In fact it probably doesn't
even work.

diff --git a/citadel/server/control.c b/citadel/server/control.c
index b3df0ae3fbf97dd78beddf0302c493a67926413a..9be3addeb75cd95d36e9116308711f9360f86c94 100644 (file)
--- a/citadel/server/control.c
+++ b/citadel/server/control.c
@@ -636,7 +636,7 @@ void cmd_conf(char *argbuf) {
                        char *valbuf = malloc(bytes + 1);
                        cprintf("%d %d\n", SEND_BINARY, bytes);
                        client_read(valbuf, bytes);
-                       valbuf[bytes+1] = 0;
+                       valbuf[bytes] = 0;
                        CtdlSetConfigStr(confname, valbuf);
                        free(valbuf);
                }