From: oetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Date: Thu, 4 May 2006 20:41:51 +0000 (+0000)
Subject: fix for debian bug 359071 ... reporting the long cdef in the error message was too... 
X-Git-Url: https://git.octo.it/?p=rrdtool.git;a=commitdiff_plain;h=5b9934cea6164cb4c1bb15a4400af2f861a4a834

fix for debian bug 359071 ... reporting the long cdef in the error message was too much ...
plus some other small cleanups in this context


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

diff --git a/src/Makefile.am b/src/Makefile.am
index dedab62..d6b3122 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -99,7 +99,7 @@ librrd_la_LDFLAGS         = -version-info 2:7:0
 
 librrd_th_la_SOURCES         = $(RRD_C_FILES) rrd_thread_safe.c
 librrd_th_la_CFLAGS          = $(MULTITHREAD_CFLAGS)
-librrd_th_la_LDFLAGS         = $(MULTITHREAD_LDFLAGS) -version-info 2:1:0
+librrd_th_la_LDFLAGS         = $(MULTITHREAD_LDFLAGS) -version-info 2:7:0
 
 
 include_HEADERS	= rrd.h
diff --git a/src/rrd_error.c b/src/rrd_error.c
index df675a2..ced26b5 100644
--- a/src/rrd_error.c
+++ b/src/rrd_error.c
@@ -111,8 +111,8 @@ rrd_new_context(void) {
 
     if (rrd_ctx) {
 	rrd_ctx->len = 0;
-	rrd_ctx->rrd_error = malloc(MAXLEN);
-	rrd_ctx->lib_errstr = malloc(ERRBUFLEN);
+	rrd_ctx->rrd_error = malloc(MAXLEN+10);
+	rrd_ctx->lib_errstr = malloc(ERRBUFLEN+10);
 	if (rrd_ctx->rrd_error && rrd_ctx->lib_errstr) {
 	    *rrd_ctx->rrd_error = 0;
 	    *rrd_ctx->lib_errstr = 0;
diff --git a/src/rrd_restore.c b/src/rrd_restore.c
index 51def7e..29a97b3 100644
--- a/src/rrd_restore.c
+++ b/src/rrd_restore.c
@@ -191,8 +191,9 @@ int xml2rrd(char* buf, rrd_t* rrd, char rc){
       read_tag(&ptr2,"max","%lf",&(rrd->ds_def[rrd->stat_head->ds_cnt-1].par[DS_max_val].u_val));
 	  } else { /* DST_CDEF */
 		 char buffer[1024];
-	     read_tag(&ptr2,"cdef","%s",buffer);
+	         read_tag(&ptr2,"cdef","%1000s",buffer);
 		 parseCDEF_DS(buffer,rrd,rrd -> stat_head -> ds_cnt - 1);
+		 if (rrd_test_error()) return -1;
 	  }
 
       read_tag(&ptr2,"last_ds","%30s",rrd->pdp_prep[rrd->stat_head->ds_cnt-1].last_ds);
diff --git a/src/rrd_rpncalc.c b/src/rrd_rpncalc.c
index 6a903f5..3c6510f 100644
--- a/src/rrd_rpncalc.c
+++ b/src/rrd_rpncalc.c
@@ -194,7 +194,7 @@ void parseCDEF_DS(char *def,rrd_t *rrd, int ds_idx)
     
     rpnp = rpn_parse((void*) rrd, def, &lookup_DS);
     if (rpnp == NULL) {
-        rrd_set_error("failed to parse computed data source %s", def);
+        rrd_set_error("failed to parse computed data source");
         return;
     }
     /* Check for OP nodes not permitted in COMPUTE DS.