Code

be more careful when checking if a file is an rrdfile
authoroetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Sat, 11 Jun 2005 19:03:46 +0000 (19:03 +0000)
committeroetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Sat, 11 Jun 2005 19:03:46 +0000 (19:03 +0000)
git-svn-id: svn://svn.oetiker.ch/rrdtool/branches/1.2@632 a5681a0c-68f1-0310-ab6d-d61299d08faa

program/src/rrd_open.c

index 11476de43cb85bcec38ceb56cab7a42da86454c9..bf818e5d1ecba3921d5b946fed05d784e29e7abc 100644 (file)
@@ -112,7 +112,6 @@ rrd_open(const char *file_name, FILE **in_file, rrd_t *rrd, int rdwr)
        fclose(*in_file);
        return(-1);
     }  
-    version = atoi(rrd->stat_head->version);
 
        /* lets do some test if we are on track ... */
        if (strncmp(rrd->stat_head->cookie,RRD_COOKIE,4) != 0){
@@ -121,19 +120,22 @@ rrd_open(const char *file_name, FILE **in_file, rrd_t *rrd, int rdwr)
            fclose(*in_file);
            return(-1);}
 
-        if (version > atoi(RRD_VERSION)){
-           rrd_set_error("can't handle RRD file version %s",
-                       rrd->stat_head->version);
+       if (rrd->stat_head->float_cookie != FLOAT_COOKIE){
+           rrd_set_error("This RRD was created on other architecture");
            free(rrd->stat_head);
            fclose(*in_file);
            return(-1);}
 
-       if (rrd->stat_head->float_cookie != FLOAT_COOKIE){
-           rrd_set_error("This RRD was created on other architecture");
+    version = atoi(rrd->stat_head->version);
+
+        if (version > atoi(RRD_VERSION)){
+           rrd_set_error("can't handle RRD file version %s",
+                       rrd->stat_head->version);
            free(rrd->stat_head);
            fclose(*in_file);
            return(-1);}
 
+
     MYFREAD(rrd->ds_def,    ds_def_t,     rrd->stat_head->ds_cnt)
     MYFREAD(rrd->rra_def,   rra_def_t,    rrd->stat_head->rra_cnt)
     /* handle different format for the live_head */