chore: some sanity checks

Author: rvazarkar

src/CommonLib/LdapConnectionPool.cs

@@ -811,17 +811,17 @@ public void ReleaseConnection(LdapConnectionWrapper connectionWrapper, bool conn
                 }
             }
             else {
-                connectionWrapper.Connection.Dispose();
+                connectionWrapper.Connection?.Dispose();
             }
         }
 
         public void Dispose() {
             while (_connections.TryTake(out var wrapper)) {
-                wrapper.Connection.Dispose();
+                wrapper.Connection?.Dispose();
             }
 
             while (_globalCatalogConnection.TryTake(out var wrapper)) {
-                wrapper.Connection.Dispose();
+                wrapper.Connection?.Dispose();
             }
         }
 

test/unit/LdapConnectionPoolTest.cs

@@ -23,7 +23,7 @@ private static ConcurrentBag<LdapConnectionWrapper> GetConnectionsBag(LdapConnec
             .GetField("_connections", BindingFlags.Instance | BindingFlags.NonPublic);
         return (ConcurrentBag<LdapConnectionWrapper>)field.GetValue(pool);
     }
-
+ 
     private static ConcurrentBag<LdapConnectionWrapper> GetGlobalCatalogConnectionsBag(LdapConnectionPool pool) {
         var field = typeof(LdapConnectionPool)
             .GetField("_globalCatalogConnection", BindingFlags.Instance | BindingFlags.NonPublic);