Fixed memory leak when running cf-check on non-existent file

Ticket: CFE-4508
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>

Author: larsewi

cf-check/validate.c

@@ -592,27 +592,34 @@ int CFCheck_Validate(const char *path)
     rc = mdb_env_open(env, path, MDB_NOSUBDIR | MDB_RDONLY, 0644);
     if (rc != 0)
     {
+        mdb_env_close(env);
         return rc;
     }
 
     MDB_txn *txn;
     rc = mdb_txn_begin(env, NULL, MDB_RDONLY, &txn);
     if (rc != 0)
     {
+        mdb_env_close(env);
         return rc;
     }
 
     MDB_dbi dbi;
     rc = mdb_open(txn, NULL, 0, &dbi);
     if (rc != 0)
     {
+        mdb_txn_abort(txn);
+        mdb_env_close(env);
         return rc;
     }
 
     MDB_cursor *cursor;
     rc = mdb_cursor_open(txn, dbi, &cursor);
     if (rc != 0)
     {
+        mdb_close(env, dbi);
+        mdb_txn_abort(txn);
+        mdb_env_close(env);
         return rc;
     }
 
@@ -627,6 +634,10 @@ int CFCheck_Validate(const char *path)
     {
         // At this point, not found is expected, anything else is an error
         DestroyValidator(&state);
+        mdb_cursor_close(cursor);
+        mdb_close(env, dbi);
+        mdb_txn_abort(txn);
+        mdb_env_close(env);
         return rc;
     }
     mdb_cursor_close(cursor);