Added "step=1800" and such to "DEF"
authoralex <alex@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Sun, 23 Jun 2002 22:29:40 +0000 (22:29 +0000)
committeralex <alex@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Sun, 23 Jun 2002 22:29:40 +0000 (22:29 +0000)
Cleaned some of the signed vs. unsigned problems

git-svn-id: svn://svn.oetiker.ch/rrdtool/trunk/program@148 a5681a0c-68f1-0310-ab6d-d61299d08faa

src/Makefile.am
src/rrd_fetch.c
src/rrd_graph.c
src/rrd_resize.c

index 9e64c24..87fdd2b 100644 (file)
@@ -63,6 +63,7 @@ RRD_C_FILES =         \
        rrd_tune.c      \
        rrd_update.c    \
        rrd_xport.c     \
+       rrd_nan_inf.c   \
        rrd_gfx.c rrd_gfx.h \
        rrd_afm.c rrd_afm_data.c \
        getopt.h ntconfig.h parsetime.h \
index c187b33..3260904 100644 (file)
@@ -5,6 +5,10 @@
  *****************************************************************************
  * $Id$
  * $Log$
+ * Revision 1.5  2002/06/23 22:29:40  alex
+ * Added "step=1800" and such to "DEF"
+ * Cleaned some of the signed vs. unsigned problems
+ *
  * Revision 1.4  2002/02/01 20:34:49  oetiker
  * fixed version number and date/time
  *
@@ -134,7 +138,7 @@ rrd_fetch(int argc,
        return -1;
     }
     
-    if ((cf_idx=cf_conv(argv[optind+1])) == -1 ){
+    if ((int)(cf_idx=cf_conv(argv[optind+1])) == -1 ){
        return -1;
     }
 
@@ -187,7 +191,7 @@ fprintf(stderr,"Looking for: start %10lu end %10lu step %5lu rows  %lu\n",
        return(-1);
     }
     
-    for(i=0;i<rrd.stat_head->ds_cnt;i++){
+    for(i=0;(unsigned long)i<rrd.stat_head->ds_cnt;i++){
        if ((((*ds_namv)[i]) = malloc(sizeof(char) * DS_NAM_SIZE))==NULL){
            rrd_set_error("malloc fetch ds_namv entry");
            rrd_free(&rrd);
@@ -201,7 +205,7 @@ fprintf(stderr,"Looking for: start %10lu end %10lu step %5lu rows  %lu\n",
     }
     
     /* find the rra which best matches the requirements */
-    for(i=0;i<rrd.stat_head->rra_cnt;i++){
+    for(i=0;(unsigned)i<rrd.stat_head->rra_cnt;i++){
        if(cf_conv(rrd.rra_def[i].cf_nam) == cf_idx){
            
            cal_end = (rrd.live_head->last_up - (rrd.live_head->last_up 
@@ -291,7 +295,7 @@ fprintf(stderr,"partial match, not best\n");
     *ds_cnt =   rrd.stat_head->ds_cnt; 
     if (((*data) = malloc(*ds_cnt * rows * sizeof(rrd_value_t)))==NULL){
        rrd_set_error("malloc fetch data area");
-       for (i=0;i<*ds_cnt;i++)
+       for (i=0;(unsigned long)i<*ds_cnt;i++)
              free((*ds_namv)[i]);
        free(*ds_namv);
        rrd_free(&rrd);
@@ -333,7 +337,7 @@ fprintf(stderr,"partial match, not best\n");
                      * *ds_cnt
                      * sizeof(rrd_value_t))),SEEK_SET) != 0){
        rrd_set_error("seek error in RRA");
-       for (i=0;i<*ds_cnt;i++)
+       for (i=0;(unsigned)i<*ds_cnt;i++)
              free((*ds_namv)[i]);
        free(*ds_namv);
        rrd_free(&rrd);
@@ -350,14 +354,14 @@ fprintf(stderr,"partial match, not best\n");
     /* step trough the array */
 
     for (i=start_offset;
-        i<(long)(rrd.rra_def[chosen_rra].row_cnt-end_offset);
+        i< (signed)rrd.rra_def[chosen_rra].row_cnt - end_offset;
         i++){
        /* no valid data yet */
        if (i<0) {
 #ifdef DEBUG
            fprintf(stderr,"pre fetch %li -- ",i);
 #endif
-           for(ii=0;ii<*ds_cnt;ii++){
+           for(ii=0;(unsigned)ii<*ds_cnt;ii++){
                *(data_ptr++) = DNAN;
 #ifdef DEBUG
                fprintf(stderr,"%10.2f ",*(data_ptr-1));
@@ -365,11 +369,11 @@ fprintf(stderr,"partial match, not best\n");
            }
        } 
        /* past the valid data area */
-       else if (i>=rrd.rra_def[chosen_rra].row_cnt) {
+       else if (i >= (signed)rrd.rra_def[chosen_rra].row_cnt) {
 #ifdef DEBUG
            fprintf(stderr,"post fetch %li -- ",i);
 #endif
-           for(ii=0;ii<*ds_cnt;ii++){
+           for(ii=0;(unsigned)ii<*ds_cnt;ii++){
                *(data_ptr++) = DNAN;
 #ifdef DEBUG
                fprintf(stderr,"%10.2f ",*(data_ptr-1));
@@ -378,13 +382,13 @@ fprintf(stderr,"partial match, not best\n");
        } else {
            /* OK we are inside the valid area but the pointer has to 
             * be wrapped*/
-           if (rra_pointer >= rrd.rra_def[chosen_rra].row_cnt) {
+           if (rra_pointer >= (signed)rrd.rra_def[chosen_rra].row_cnt) {
                rra_pointer -= rrd.rra_def[chosen_rra].row_cnt;
                if(fseek(in_file,(rra_base+rra_pointer
                               * *ds_cnt
                               * sizeof(rrd_value_t)),SEEK_SET) != 0){
                    rrd_set_error("wrap seek in RRA did fail");
-                   for (ii=0;ii<*ds_cnt;ii++)
+                   for (ii=0;(unsigned)ii<*ds_cnt;ii++)
                        free((*ds_namv)[ii]);
                    free(*ds_namv);
                    rrd_free(&rrd);
@@ -402,7 +406,7 @@ fprintf(stderr,"partial match, not best\n");
                     sizeof(rrd_value_t),
                     *ds_cnt,in_file) != rrd.stat_head->ds_cnt){
                rrd_set_error("fetching cdp from rra");
-               for (ii=0;ii<*ds_cnt;ii++)
+               for (ii=0;(unsigned)ii<*ds_cnt;ii++)
                    free((*ds_namv)[ii]);
                free(*ds_namv);
                rrd_free(&rrd);
index a73dbbc..f0ec67e 100644 (file)
@@ -681,8 +681,9 @@ for (col=0;col<row_cnt;col++) {
 int
 data_fetch( image_desc_t *im )
 {
-    int       i,ii;
-    int skip;
+    int                i,ii;
+    int                skip;
+
     /* pull the data from the log files ... */
     for (i=0;i<im->gdes_c;i++){
        /* only GF_DEF elements fetch data */
@@ -691,13 +692,17 @@ data_fetch( image_desc_t *im )
 
        skip=0;
        /* do we have it already ?*/
-       for (ii=0;ii<i;ii++){
+       for (ii=0;ii<i;ii++) {
            if (im->gdes[ii].gf != GF_DEF) 
                continue;
-           if((strcmp(im->gdes[i].rrd,im->gdes[ii].rrd) == 0)
-               && (im->gdes[i].cf == im->gdes[ii].cf)){
-               /* OK the data it is here already ... 
-                * we just copy the header portion */
+           if ((strcmp(im->gdes[i].rrd, im->gdes[ii].rrd) == 0)
+                       && (im->gdes[i].cf    == im->gdes[ii].cf)
+                       && (im->gdes[i].start == im->gdes[ii].start)
+                       && (im->gdes[i].end   == im->gdes[ii].end)
+                       && (im->gdes[i].step  == im->gdes[ii].step)) {
+               /* OK, the data is already there.
+               ** Just copy the header portion
+               */
                im->gdes[i].start = im->gdes[ii].start;
                im->gdes[i].end = im->gdes[ii].end;
                im->gdes[i].step = im->gdes[ii].step;
@@ -2524,7 +2529,6 @@ scan_for_col(char *input, int len, char *output)
     output[outp] = '\0';
     return inp;
 }
-
 /* Some surgery done on this function, it became ridiculously big.
 ** Things moved:
 ** - initializing     now in rrd_graph_init()
@@ -2951,31 +2955,285 @@ rrd_graph_options(int argc, char *argv[],image_desc_t *im)
     im->end = end_tmp;
 }
 
+/* rrd_name_or_num()
+**
+** Scans for a VDEF-variable or a number
+**
+** Returns an integer describing what was found:
+**
+** 0: error
+** 1: found an integer; it is returned in both l and d
+** 2: found a float; it is returned in d
+** 3: found a vname; its index is returned in l
+**
+** l and d are undefined unless described above
+*/
+static int
+rrd_name_or_num(image_desc_t *im, char *param, long *l, double *d)
+{
+    int i1=0,i2=0,i3=0,i4=0,i5=0,i6=0;
+    char vname[MAX_VNAME_LEN+1];
+
+    sscanf(param, "%li%n%*s%n", l,&i1,&i2);
+    sscanf(param, "%lf%n%*s%n", d,&i3,&i4);
+    sscanf(param, DEF_NAM_FMT "%n%*s%n", vname, &i5,&i6);
+
+    if ( (i1) && (!i2) ) return 1;
+    if ( (i3) && (!i4) ) return 2;
+    if ( (i5) && (!i6) ) {
+       if ((*l = find_var(im,vname))!=-1) return 3;
+    }
+    return 0;
+}
+
+/* rrd_vname_color()
+**
+** Parses "[<vname|number>][#color]" where at least one
+** of the optional strings must exist.
+**
+** Returns an integer describing what was found.
+** If the result is 0, the rrd_error string may be set.
+**
+** ...CVVVV 
+** ---------:-----------------------------------
+** 00000000 : error
+** ....0000 : a value/variable was not found
+** ....0001 : an integer number was found, returned in both l and d
+** ....0010 : a floating point number was found, returned in d
+** ....0011 : reserved for future values
+** ....01xx : reserved for future values
+** ....1000 : an existing  DEF vname was found, idx returned in l
+** ....1001 : an existing CDEF vname was found, idx returned in l
+** ....1010 : an existing VDEF vname was found, idx returned in l
+** ....1011 : reserved for future variables
+** ....11xx : reserved for future variables
+** ...0.... : a color was not found, returned in color
+** ...1.... : a color was found, returned in color
+*/
+static int
+rrd_vname_color(image_desc_t *im, char * param,
+                       long *l,
+                       double *d,
+                       gfx_color_t *color)
+{
+    int result=0,i=0;
+
+    if (param[0]!='#') { /* vname or num present or empty string */
+       char *s,*c=param;
+       while ((*c!='\0')&&(*c!='#')) c++,i++;
+       if (*c!='\0') {
+           s=malloc(i+1);
+           if (s==NULL) {
+               rrd_set_error("Out of memory in function rrd_vname_color");
+               return 0;
+           }
+           strncpy(s,param,i);
+           s[i]='\0';
+           result=rrd_name_or_num(im, s, l, d);
+           if (!result) {
+               rrd_set_error("Use of uninitialized vname %s",s);
+               free(s);
+           }
+       } else {
+           result=rrd_name_or_num(im, param, l, d);
+           if (!result) {
+               rrd_set_error("Use of uninitialized vname %s",param);
+           }
+       }
+       switch (result) {
+           case 0: return 0; /* error set above */
+           case 1:
+           case 2: break;
+           case 3:
+               switch (im->gdes[*l].gf) {
+                   case GF_DEF:        result=0x08;break;
+                   case GF_CDEF:       result=0x09;break;
+                   case GF_VDEF:       result=0x0A;break;
+                   default:
+                       rrd_set_error("Unexpected GF result from function "
+                       "rrd_name_or_num() called from rrd_vname_color");
+                       return 0;
+               }
+               break;
+           default:
+               rrd_set_error("Unexpected result from function "
+                       "rrd_name_or_num() called from rrd_vname_color");
+               return 0;
+       }
+    }
+    /* Parse color, if any. */
+    if (param[i] == '\0') return result;
+    else {
+       unsigned int r=0,g=0,b=0,a=0xFF;
+       int i1=0,i2=0;
+       sscanf(&param[i], "#%02x%02x%02x%n%02x%n",
+                               &r,&g,&b,&i1,&a,&i2);
+       if (!i1) {
+           rrd_set_error("Unparsable color %s",&param[i]);
+           return 0;
+       }
+       if (i2) i1=i2;
+       i2=0;
+       sscanf(&param[i+i1],"%*s%n",&i2);
+       if (i2) {
+           rrd_set_error("Garbage after color %s",param[i]);
+           return 0;
+       }
+       *color=r<<24|g<<16|b<<8|a;
+       return result|0x10;
+    }
+}
+
+/* rrd_find_function()
+**
+** Checks if the parameter is a valid function and
+** if so, returns it in the graph description pointer.
+**
+** The return value is a boolean; true if found
+*/
+static int
+rrd_find_function(char *param, graph_desc_t *gdp)
+{
+    size_t i1=0,i2=0;
+    char funcname[11];
+
+    sscanf(param,"%10[A-Z]%n%*1[1-3]%n",funcname,(int *)&i1,(int *)&i2);
+    gdp->gf=gf_conv(funcname);
+    if ((int)gdp->gf == -1) {
+       rrd_set_error("'%s' is not a valid function name",funcname);
+       return 0;
+    }
+    if (gdp->gf==GF_LINE) {
+       if (i2) {
+           gdp->linewidth=param[i1]-'0';
+       } else {
+           rrd_set_error("LINE should have a width");
+           return 0;
+       }
+    } else {
+       if (i2) {
+           rrd_set_error("Only LINE should have a width: %s",param);
+           return 0;
+       } else {
+           i2=i1;
+       }
+    }
+    if (strlen(param) != i2) {
+       rrd_set_error("Garbage after function name: %s",param);
+       return 0;
+    }
+    return 1;
+}
+/* rrd_split_line()
+**
+** Takes a string as input; splits this line into multiple
+** parameters on each ":" boundary.
+**
+** If this function returns successful, the caller will have
+** to free() the allocated memory for param.
+**
+** The input string is destroyed, its memory is used by the
+** output array.
+*/
+static int
+rrd_split_line(char *line,char ***param)
+{
+    int i=0,n=0;
+    char *c=line;
+
+    /* scan the amount of colons in the line.  We need
+    ** at most this amount+1 pointers for the array. If
+    ** any colons are escaped we waste some space.
+    */
+    if (*c!='\0') n=1;
+    while (*c != '\0')
+       if (*c++ == ':') n++;
+
+    if (n==0) {
+       rrd_set_error("No line to split. rrd_split_line was given the empty string.");
+       return -1;
+    }
+
+    /* Allocate memory for an array of n char pointers */
+    *param=calloc(n,sizeof(char *));
+    if (*param==NULL) {
+       rrd_set_error("Memory allocation failed inside rrd_split_line");
+       return -1;
+    }
+
+    /* split the line and fill the array */
+    c = line;
+    i=0;
+    (*param)[i] = c;
+    while (*c != '\0') {
+       switch (*c) {
+           case '\\':
+               c++;
+               if (*c=='\0') {
+                   free(*param);
+                   rrd_set_error("Lone backslash found inside rrd_split_line");
+                   return -1;
+               }
+               c++;
+               break;
+           case ':':
+               *c = '\0';
+               c++;
+               i++;
+               (*param)[i] = c;
+               break;
+           default:
+               c++;
+       }
+    }
+    i++; /* i separators means i+1 parameters */
+
+    return i;
+}
 void
 rrd_graph_script(int argc, char *argv[], image_desc_t *im)
 {
     int                i;
     char       symname[100];
     int                linepass = 0; /* stack must follow LINE*, AREA or STACK */    
+    char **    param;
+    int                paramcnt,paramused;
 
     for (i=optind+1;i<argc;i++) {
        int             argstart=0;
        int             strstart=0;
        graph_desc_t    *gdp;
        char            *line;
-       char            funcname[10],vname[MAX_VNAME_LEN+1],sep[1];
-       double          d;
-       double          linewidth;
-       int             j,k,l,m;
+       char    tmpline[256];
+       char            vname[MAX_VNAME_LEN+1],sep[1];
+/*     double          d;      */
+       int             j,k,l/*,m*/;
 
-       /* Each command is one element from *argv[], we call this "line".
+       /* Each command is one element from *argv[].  This command is
+       ** split at every unescaped colon.
        **
        ** Each command defines the most current gdes inside struct im.
        ** In stead of typing "im->gdes[im->gdes_c-1]" we use "gdp".
        */
        gdes_alloc(im);
        gdp=&im->gdes[im->gdes_c-1];
-       line=argv[i];
+       strcpy(tmpline,argv[i]);
+       line=tmpline;
+       if ((paramcnt=rrd_split_line(argv[i],&param))==-1) return;
+       paramused=0;
+
+#ifdef DEBUG
+       printf("DEBUG: after splitting line:\n");
+       for (j=0;j<paramcnt;j++)
+           printf("DEBUG: %3i: %s\n",j,param[j]);
+#endif
+
+       if (!rrd_find_function(param[paramused],gdp)) {
+           im_free(im);
+           free(param);
+           return;
+       }
+       paramused++;
 
        /* function:newvname=string[:ds-name:CF]        for xDEF
        ** function:vname[#color[:string]]              for LINEx,AREA,STACK
@@ -2984,57 +3242,71 @@ rrd_graph_script(int argc, char *argv[], image_desc_t *im)
        ** function:vname:CF:string                     for xPRINT
        ** function:string                              for COMMENT
        */
-       argstart=0;
 
-       sscanf(line, "%10[A-Z0-9]:%n", funcname,&argstart);
-       if (argstart==0) {
-           rrd_set_error("Cannot parse function in line: %s",line);
-           im_free(im);
-           return;
-       }
-        if(sscanf(funcname,"LINE%lf",&linewidth)){
-                im->gdes[im->gdes_c-1].gf = GF_LINE;
-                im->gdes[im->gdes_c-1].linewidth = linewidth;
-        } else {
-         if ((gdp->gf=gf_conv(funcname))==-1) {
-             rrd_set_error("'%s' is not a valid function name",funcname);
-             im_free(im);
-             return;
-         }
-        }
+/*TEMP*/argstart=strlen(param[paramused-1])+1;
 
-       /* If the error string is set, we exit at the end of the switch */
+       /* If anything fails just use rrd_set_error() and break from the
+       ** switch.  Just after the switch we call rrd_test_error() and
+       ** clean up if it is set.
+       */
        switch (gdp->gf) {
            case GF_XPORT:
                break;
            case GF_COMMENT:
-               if (rrd_graph_legend(gdp,&line[argstart])==0)
-                   rrd_set_error("Cannot parse comment in line: %s",line);
+               if (paramcnt<2) {
+                   rrd_set_error("Not enough parameters for %s",param[0]);
+                   break;
+               }
+               if (strlen(param[1])>FMT_LEG_LEN) {
+                   rrd_set_error("Comment too long: %s:%s",param[0],param[1]);
+                   break;
+               }
+               strcpy(gdp->legend,param[1]);
+               paramused++;
                break;
            case GF_PART:
            case GF_VRULE:
            case GF_HRULE:
-               j=k=l=m=0;
-               sscanf(&line[argstart], "%lf%n#%n", &d, &j, &k);
-               sscanf(&line[argstart], DEF_NAM_FMT "%n#%n", vname, &l, &m);
-               if (k+m==0) {
-                   rrd_set_error("Cannot parse name or num in line: %s",line);
+               if (paramcnt<2) {
+                   rrd_set_error("No name or number in %s",param[0]);
                    break;
                }
-               if (j!=0) {
-                   gdp->xrule=d;
-                   gdp->yrule=d;
-                   argstart+=j;
-               } else if (!rrd_graph_check_vname(im,vname,line)) {
-                   gdp->xrule=0;
-                   gdp->yrule=DNAN;
-                   argstart+=l;
-               } else break; /* exit due to wrong vname */
-               if ((j=rrd_graph_color(im,&line[argstart],line,0))==0) break;
-               argstart+=j;
-               if (strlen(&line[argstart])!=0) {
-                   if (rrd_graph_legend(gdp,&line[++argstart])==0)
-                       rrd_set_error("Cannot parse comment in line: %s",line);
+               j=rrd_vname_color(im,param[1],
+                               &gdp->xrule,&gdp->yrule,&gdp->col);
+               paramused++;
+               if (!j) break; /* error string set by function */
+               switch (j&0x0F) {
+                   case 0x00:
+                       rrd_set_error("Cannot parse name nor number "
+                               "in %s:%s",param[0],param[1]);
+                       break;
+                   case 0x08:
+                   case 0x09:
+                       rrd_set_error("Cannot use DEF or CDEF based "
+                               "variable in %s:%s",param[0],param[1]);
+                       break;
+                   case 0x0A:
+                       gdp->vidx=gdp->xrule;
+                       gdp->xrule=0;
+                       gdp->yrule=DNAN;
+                       break;
+                   case 0x01:
+                   case 0x02:
+                       break;
+                   default:
+                       rrd_set_error("Unexpected result while parsing "
+                               "%s:%s, program error",param[0],param[1]);
+               }
+               if (rrd_test_error()) break;
+
+               if (paramcnt>paramused) {
+                   if (strlen(param[paramused])>FMT_LEG_LEN) {
+                       rrd_set_error("Comment too long: %s:%s",
+                                               param[0],param[1]);
+                       break;
+                   }
+                   strcpy(gdp->legend,param[paramused]);
+                   paramused++;
                }
                break;
            case GF_STACK:
@@ -3110,29 +3382,73 @@ rrd_graph_script(int argc, char *argv[], image_desc_t *im)
            case GF_VDEF:
            case GF_CDEF:
                j=0;
-               sscanf(&line[argstart], DEF_NAM_FMT "=%n",gdp->vname,&j);
+               if (paramcnt<2) {
+                   rrd_set_error("Nothing following %s",param[0]);
+                   break;
+               }
+               sscanf(param[1], DEF_NAM_FMT "=%n",gdp->vname,&j);
                if (j==0) {
-                   rrd_set_error("Could not parse line: %s",line);
+                   rrd_set_error("Could not parse %s:%s",param[0],param[1]);
                    break;
                }
                if (find_var(im,gdp->vname)!=-1) {
-                   rrd_set_error("Variable '%s' in line '%s' already in use\n",
-                                                       gdp->vname,line);
+                   rrd_set_error("Variable '%s' in %s:%s' already in use\n",
+                                               gdp->vname,param[0],param[1]);
                    break;
                }
+               paramused++;
                argstart+=j;
                switch (gdp->gf) {
                    case GF_DEF:
-                       argstart+=scan_for_col(&line[argstart],MAXPATH,gdp->rrd);
+                       if (strlen(&param[1][j])>MAXPATH) {
+                           rrd_set_error("Path too long: %s:%s",param[0],param[1]);
+                           break;
+                       }
+                       strcpy(gdp->rrd,&param[1][j]);
+
+                       if (paramcnt<3) {
+                           rrd_set_error("No DS for %s:%s",param[0],param[1]);
+                           break;
+                       }
+                       j=k=0;
+                       sscanf(param[2],DS_NAM_FMT "%n%*s%n",gdp->ds_nam,&j,&k);
+                       if ((j==0)||(k!=0)) {
+                           rrd_set_error("Cannot parse DS in %s:%s:%s",
+                                               param[0],param[1],param[2]);
+                           break;
+                       }
+                       paramused++;
+                       if (paramcnt<4) {
+                           rrd_set_error("No CF for %s:%s:%s",
+                                       param[0],param[1],param[2]);
+                           break;
+                       }
                        j=k=0;
-                       sscanf(&line[argstart],
-                               ":" DS_NAM_FMT ":" CF_NAM_FMT "%n%*s%n",
-                               gdp->ds_nam, symname, &j, &k);
+                       sscanf(param[3],CF_NAM_FMT "%n%*s%n",symname,&j,&k);
                        if ((j==0)||(k!=0)) {
-                           rrd_set_error("Cannot parse DS or CF in '%s'",line);
+                           rrd_set_error("Cannot parse CF in %s:%s:%s:%s",
+                                       param[0],param[1],param[2],param[3]);
                            break;
                        }
-                       rrd_graph_check_CF(im,symname,line);
+                       if ((gdp->cf = cf_conv(symname))==-1) {
+                           rrd_set_error("Unknown CF '%s' in %s:%s:%s:%s",
+                                       param[0],param[1],param[2],param[3]);
+                           break;
+                       }
+                       paramused++;
+                       if (paramcnt>paramused) {
+                           k=0;l=0;
+                           sscanf(param[4],
+                                       "step=%lu%n%*s%n",
+                                &gdp->step,&k,&l);
+                            if ((k==0)||(l!=0)) {
+                                rrd_set_error("Cannot parse step in "
+                                                       "%s:%s:%s:%s:%s",
+                               param[0],param[1],param[2],param[3],param[4]);
+                               break;
+                            }
+                           paramused++;
+                       }
                        break;
                    case GF_VDEF:
                        j=0;
@@ -3180,6 +3496,7 @@ rrd_graph_script(int argc, char *argv[], image_desc_t *im)
        return; 
     }
 }
+
 int
 rrd_graph_check_vname(image_desc_t *im, char *varname, char *err)
 {
index 877120e..cdf88c3 100644 (file)
@@ -43,7 +43,7 @@ rrd_resize(int argc, char **argv)
     modify=strtol(argv[4],&endptr,0);
 
     if ((modify<1)) {
-        rrd_set_error("you must have at least one row in the RRA");
+        rrd_set_error("Please grow or shrink with at least 1 row");
         return(-1);
     }
 
@@ -67,12 +67,14 @@ rrd_resize(int argc, char **argv)
         fclose(infile);
         return(-1);
     }
-    if ((rrdold.rra_def[target_rra].row_cnt+modify)<0) {
-        rrd_set_error("This RRA is not that big");
-        rrd_free(&rrdold);
-        fclose(infile);
-        return(-1);
-    }
+
+    if (modify < 0)
+       if (rrdold.rra_def[target_rra].row_cnt <= modify) {
+           rrd_set_error("This RRA is not that big");
+           rrd_free(&rrdold);
+           fclose(infile);
+           return(-1);
+       }
 
     rrdnew.stat_head = rrdold.stat_head;
     rrdnew.ds_def    = rrdold.ds_def;