[Ldap] Do not run ldap_set_option on failed connection

tatankat · nicolas-grekas · commit 8f4f2b1e58a0 · 2022-09-28T15:02:08.000+02:00
diff --git a/src/Symfony/Component/Ldap/Adapter/ExtLdap/Connection.php b/src/Symfony/Component/Ldap/Adapter/ExtLdap/Connection.php
@@ -159,18 +159,18 @@ private function connect()
             }
         }
 
-        $this->connection = ldap_connect($this->config['connection_string']);
+        if (false === $connection = ldap_connect($this->config['connection_string'])) {
+            throw new LdapException('Invalid connection string: '.$this->config['connection_string']);
+        } else {
+            $this->connection = $connection;
+        }
 
         foreach ($this->config['options'] as $name => $value) {
             if (!\in_array(ConnectionOptions::getOption($name), self::PRECONNECT_OPTIONS, true)) {
                 $this->setOption($name, $value);
             }
         }
 
-        if (false === $this->connection) {
-            throw new LdapException('Could not connect to Ldap server: '.ldap_error($this->connection));
-        }
-
         if ('tls' === $this->config['encryption'] && false === @ldap_start_tls($this->connection)) {
             throw new LdapException('Could not initiate TLS connection: '.ldap_error($this->connection));
         }

Original file line number	Diff line number	Diff line change
`@@ -159,18 +159,18 @@ private function connect()`
`159`	`159`	`}`
`160`	`160`	`}`
`161`	`161`
`162`		`- $this->connection = ldap_connect($this->config['connection_string']);`
	`162`	`+ if (false === $connection = ldap_connect($this->config['connection_string'])) {`
	`163`	`+ throw new LdapException('Invalid connection string: '.$this->config['connection_string']);`
	`164`	`+ } else {`
	`165`	`+ $this->connection = $connection;`
	`166`	`+ }`
`163`	`167`
`164`	`168`	`foreach ($this->config['options'] as $name => $value) {`
`165`	`169`	`if (!\in_array(ConnectionOptions::getOption($name), self::PRECONNECT_OPTIONS, true)) {`
`166`	`170`	`$this->setOption($name, $value);`
`167`	`171`	`}`
`168`	`172`	`}`
`169`	`173`
`170`		`- if (false === $this->connection) {`
`171`		`- throw new LdapException('Could not connect to Ldap server: '.ldap_error($this->connection));`
`172`		`- }`
`173`		`-`
`174`	`174`	`if ('tls' === $this->config['encryption'] && false === @ldap_start_tls($this->connection)) {`
`175`	`175`	`throw new LdapException('Could not initiate TLS connection: '.ldap_error($this->connection));`
`176`	`176`	`}`