libsemanage: free resources on failed connect attempt

Signed-off-by: Christian Göttsche <[email protected]>

libsemanage/src/database_activedb.c

@@ -139,6 +139,9 @@ int dbase_activedb_init(semanage_handle_t * handle,
 void dbase_activedb_release(dbase_activedb_t * dbase)
 {
 
+ if (!dbase)
+ return;
+
  dbase_llist_drop_cache(&dbase->llist);
  free(dbase);
 }

libsemanage/src/database_file.c

@@ -191,6 +191,9 @@ int dbase_file_init(semanage_handle_t * handle,
 void dbase_file_release(dbase_file_t * dbase)
 {
 
+ if (!dbase)
+ return;
+
  dbase_llist_drop_cache(&dbase->llist);
  free(dbase);
 }

libsemanage/src/database_join.c

@@ -271,6 +271,9 @@ int dbase_join_init(semanage_handle_t * handle,
 void dbase_join_release(dbase_join_t * dbase)
 {
 
+ if (!dbase)
+ return;
+
  dbase_llist_drop_cache(&dbase->llist);
  free(dbase);
 }

libsemanage/src/database_policydb.c

@@ -44,7 +44,7 @@ struct dbase_policydb {
 static void dbase_policydb_drop_cache(dbase_policydb_t * dbase)
 {
 
- if (dbase->cache_serial >= 0) {
+ if (dbase && dbase->cache_serial >= 0) {
  sepol_policydb_free(dbase->policydb);
  dbase->cache_serial = -1;
  dbase->modified = 0;

libsemanage/src/direct_api.c

@@ -321,6 +321,7 @@ int semanage_direct_connect(semanage_handle_t * sh)
 
  err:
  ERR(sh, "could not establish direct connection");
+ (void) semanage_direct_disconnect(sh);
  return STATUS_ERR;
 }
 

libsemanage/src/handle.c

@@ -368,12 +368,11 @@ int semanage_access_check(semanage_handle_t * sh)
 
 int semanage_disconnect(semanage_handle_t * sh)
 {
- assert(sh != NULL && sh->funcs != NULL
- && sh->funcs->disconnect != NULL);
+ assert(sh != NULL);
  if (!sh->is_connected) {
  return 0;
  }
- if (sh->funcs->disconnect(sh) < 0) {
+ if (sh->funcs && sh->funcs->disconnect(sh) < 0) {
  return -1;
  }
  sh->is_in_transaction = 0;