it makes no sense allocating an array and then asiging it to a fixed string ...

this caused stuff to crash badly as long error strings were assigned.

git-svn-id: svn://svn.oetiker.ch/rrdtool/branches/1.2/program@929 a5681a0c-68f1-0310-ab6d-d61299d08faa

diff --git a/src/rrd_not_thread_safe.c b/src/rrd_not_thread_safe.c
index 2d3b350..7022277 100644 (file)
--- a/src/rrd_not_thread_safe.c
+++ b/src/rrd_not_thread_safe.c
@@ -14,20 +14,20 @@
 #define MAXLEN 4096
 #define ERRBUFLEN 256
 
-static char rrd_error[MAXLEN] = "\0";
-static char rrd_liberror[ERRBUFLEN] = "\0";
+static char rrd_error[MAXLEN+10];
+static char rrd_liberror[ERRBUFLEN+10];
 /* The global context is very useful in the transition period to even
    more thread-safe stuff, it can be used whereever we need a context
    and do not need to worry about concurrency. */
 static struct rrd_context global_ctx = {
-    sizeof(rrd_error),
-    sizeof(rrd_liberror),
+    MAXLEN,
+    ERRBUFLEN,
     rrd_error, 
     rrd_liberror
 };
-#include <stdarg.h>
+/* #include <stdarg.h> */
 
-struct rrd_context *rrd_get_context() {
+struct rrd_context *rrd_get_context(void) {
     return &global_ctx;
 }