#17 fix a memory leak

CTestCustom.cmake

@@ -29,6 +29,8 @@ list(APPEND CTEST_CUSTOM_MEMCHECK_IGNORE
   ydb/loader-cleanup-test3.tdb
   ydb/loader-stress-test4.tdb
   ydb/maxsize-for-loader-B.tdb
+  ydb/openlimit17.tdb
+  ydb/openlimit17-locktree.tdb
   ydb/preload-db-nested.tdb
   ydb/stress-gc.tdb
   ydb/stress-gc2.tdb

ft/logger.cc

@@ -300,6 +300,8 @@ toku_logger_open_rollback(TOKULOGGER logger, CACHETABLE cachetable, bool create)
         bool is_empty;
         is_empty = toku_ft_is_empty_fast(t);
         assert(is_empty);
+    } else {
+        toku_ft_handle_close(t);
     }
     return r;
 }

src/tests/openlimit17-locktree.cc

@@ -140,7 +140,7 @@ int test_main (int argc __attribute__((__unused__)), char *const argv[] __attrib
     for (int i = 0; i < N; i++) {
         dbs[i] = NULL;
     }
-    bool emfile_happened = false;
+    bool emfile_happened = false; // should happen since there are less than N unused file descriptors
     for (int i = 0; i < N; i++) {
         r = db_create(&dbs[i], env, 0);
         assert(r == 0);

src/tests/openlimit17-metafiles.cc

@@ -109,11 +109,9 @@ int test_main (int argc __attribute__((__unused__)), char *const argv[] __attrib
     r = env->close(env, 0);
     assert(r == 0);
 
-
     struct rlimit nofile_limit;
     r = getrlimit(RLIMIT_NOFILE, &nofile_limit);
     assert(r == 0);
-
     const int N = 100;
     nofile_limit.rlim_cur = N;
     r = setrlimit(RLIMIT_NOFILE, &nofile_limit);

src/tests/openlimit17.cc

@@ -96,7 +96,7 @@ PATENT RIGHTS GRANT:
 int test_main (int argc __attribute__((__unused__)), char *const argv[] __attribute__((__unused__))) {
     int r;
 
-    const int N = 100;
+    const int N = 200;
 
     struct rlimit nofile_limit = { N, N };
     r = setrlimit(RLIMIT_NOFILE, &nofile_limit);
@@ -120,7 +120,7 @@ int test_main (int argc __attribute__((__unused__)), char *const argv[] __attrib
     for (int i = 0; i < N; i++) {
         dbs[i] = NULL;
     }
-    bool emfile_happened = false;
+    bool emfile_happened = false; // should happen since there are less than N unused file descriptors
     for (int i = 0; i < N; i++) {
         r = db_create(&dbs[i], env, 0);
         assert(r == 0);