mds: fix fd leak in dumper

CID 717084: Resource leak (RESOURCE_LEAK)
At (6): Handle variable "fd" going out of scope leaks the handle.

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/mds/Dumper.cc b/src/mds/Dumper.cc
index d68df50f0e34906aff1474d133cf4d1d8a5b50c3..793625581dc40bd2a8d9e5c2b9b04bb88837e8b4 100644 (file)
--- a/src/mds/Dumper.cc
+++ b/src/mds/Dumper.cc
@@ -170,8 +170,10 @@ void Dumper::undump(const char *dump_file)
 
   char buf[200];
   int r = safe_read(fd, buf, sizeof(buf));
-  if (r < 0)
+  if (r < 0) {
+    TEMP_FAILURE_RETRY(::close(fd));
     return;
+  }
 
   long long unsigned start, len;
   sscanf(strstr(buf, "start offset"), "start offset %llu", &start);
@@ -232,6 +234,7 @@ void Dumper::undump(const char *dump_file)
     left -= l;
   }
 
+  TEMP_FAILURE_RETRY(::close(fd));
   cout << "done." << std::endl;
 }