X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Frrd_resize.c;h=52c208b58f6c98869bafee934ba059b94f1cac12;hb=dd86fdf1b4d99437b9aa19934e02230c090cc1d5;hp=11efcdaac7617f3fffa4cc768c48be29dfb73b60;hpb=009ac7ac9dde1ba7c450b230b9118cff2cedc9f6;p=rrdtool.git

diff --git a/src/rrd_resize.c b/src/rrd_resize.c
index 11efcda..52c208b 100644
--- a/src/rrd_resize.c
+++ b/src/rrd_resize.c
@@ -57,20 +57,20 @@ int rrd_resize(
 
     rrd_file = rrd_open(infilename, &rrdold, RRD_READWRITE | RRD_COPY);
     if (rrd_file == NULL) {
-        rrd_set_error("could not open RRD");
+        rrd_free(&rrdold);
         return (-1);
     }
     if (LockRRD(rrd_file->fd) != 0) {
         rrd_set_error("could not lock original RRD");
         rrd_free(&rrdold);
-        close(rrd_file->fd);
+        rrd_close(rrd_file);
         return (-1);
     }
 
     if (target_rra >= rrdold.stat_head->rra_cnt) {
         rrd_set_error("no such RRA in this RRD");
         rrd_free(&rrdold);
-        close(rrd_file->fd);
+        rrd_close(rrd_file);
         return (-1);
     }
 
@@ -78,7 +78,7 @@ int rrd_resize(
         if ((long) rrdold.rra_def[target_rra].row_cnt <= -modify) {
             rrd_set_error("This RRA is not that big");
             rrd_free(&rrdold);
-            close(rrd_file->fd);
+            rrd_close(rrd_file);
             return (-1);
         }
 
@@ -86,13 +86,14 @@ int rrd_resize(
     if (rrd_out_file == NULL) {
         rrd_set_error("Can't create '%s': %s", outfilename,
                       rrd_strerror(errno));
+        rrd_free(&rrdnew);
         return (-1);
     }
     if (LockRRD(rrd_out_file->fd) != 0) {
         rrd_set_error("could not lock new RRD");
         rrd_free(&rrdold);
-        close(rrd_file->fd);
-        close(rrd_out_file->fd);
+        rrd_close(rrd_file);
+        rrd_close(rrd_out_file);
         return (-1);
     }
 /*XXX: do one write for those parts of header that are unchanged */
@@ -231,11 +232,9 @@ int rrd_resize(
               sizeof(rra_ptr_t) * rrdnew.stat_head->rra_cnt);
 
     rrd_free(&rrdold);
-    close(rrd_file->fd);
     rrd_close(rrd_file);
 
     rrd_free(&rrdnew);
-    close(rrd_out_file->fd);
     rrd_close(rrd_out_file);
 
     return (0);