From: oetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Date: Tue, 22 Jul 2008 15:15:44 +0000 (+0000)
Subject: Fix for #167 - rrdcreate is arguably missing a check for 'step>=1' for RRAs with... 
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=b4eb6801b793c0312f20f93de0cd147ea2bfe533;p=rrdtool-all.git

Fix for #167 - rrdcreate is arguably missing a check for 'step>=1' for RRAs with AVERAGE as a CF. I ran into this when specifying 0.6 as a step value, assuming floats were also accepted. Patch attached here. -- Bogdan


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

diff --git a/program/src/rrd_create.c b/program/src/rrd_create.c
index d9ab8f48..186b3b40 100644
--- a/program/src/rrd_create.c
+++ b/program/src/rrd_create.c
@@ -312,6 +312,8 @@ rrd_create_r(const char *filename,
                         break;
                     default:
                         rrd.rra_def[rrd.stat_head->rra_cnt].pdp_cnt = atoi(token);
+                        if (atoi(token) < 1) 
+                            rrd_set_error("Invalid step: must be >= 1"); 
                         break;
                     }
                     break;