symfony
diff --git a/‎src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php
+3-3Lines changed: 3 additions & 3 deletions b/‎src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php
+3-3Lines changed: 3 additions & 3 deletions
diff --git a/‎src/Symfony/Component/Ldap/Tests/Adapter/ExtLdap/EntryManagerTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Ldap/Tests/Adapter/ExtLdap/EntryManagerTest.php
+30Lines changed: 30 additions & 0 deletions b/‎src/Symfony/Component/Ldap/Tests/Adapter/ExtLdap/EntryManagerTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Ldap/Tests/Adapter/ExtLdap/EntryManagerTest.php
+30Lines changed: 30 additions & 0 deletions
@@ -79,7 +79,7 @@ public function addAttributeValues(Entry $entry, string $attribute, array $value
         $con = $this->getConnectionResource();
 
         if (!@ldap_mod_add($con, $entry->getDn(), [$attribute => $values])) {
-            throw new LdapException(sprintf('Could not add values to entry "%s", attribute %s: ', $entry->getDn(), $attribute).ldap_error($con));
+            throw new LdapException(sprintf('Could not add values to entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con));
         }
     }
 
@@ -94,7 +94,7 @@ public function removeAttributeValues(Entry $entry, string $attribute, array $va
         $con = $this->getConnectionResource();
 
         if (!@ldap_mod_del($con, $entry->getDn(), [$attribute => $values])) {
-            throw new LdapException(sprintf('Could not remove values from entry "%s", attribute %s: ', $entry->getDn(), $attribute).ldap_error($con));
+            throw new LdapException(sprintf('Could not remove values from entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con));
         }
     }
 
@@ -159,7 +159,7 @@ public function applyOperations(string $dn, iterable $operations): void
 
     private function parseRdnFromEntry(Entry $entry): string
     {
-        if (!preg_match('/^([^,]+),/', $entry->getDn(), $matches)) {
+        if (!preg_match('/(^[^,\\\\]*(?:\\\\.[^,\\\\]*)*),/', $entry->getDn(), $matches)) {
             throw new LdapException(sprintf('Entry "%s" malformed, could not parse RDN.', $entry->getDn()));
         }
 
 
@@ -44,4 +44,34 @@ public function testGetResources()
         $entryManager = new EntryManager($connection);
         $entryManager->update($entry);
     }
+
+    /**
+     * @see https://tools.ietf.org/html/rfc4514#section-3
+     *
+     * @dataProvider moveWithRFC4514DistinguishedNameProvider
+     */
+    public function testMoveWithRFC4514DistinguishedName(string $dn, string $expectedRdn)
+    {
+        $connection = $this->createMock(Connection::class);
+
+        $entry = new Entry($dn);
+        $entryManager = new EntryManager($connection);
+
+        $method = (new \ReflectionClass(EntryManager::class))->getMethod('parseRdnFromEntry');
+        $method->setAccessible(true);
+
+        $cn = $method->invokeArgs($entryManager, [$entry, 'a']);
+
+        $this->assertSame($expectedRdn, $cn);
+    }
+
+    public function moveWithRFC4514DistinguishedNameProvider(): array
+    {
+        return [
+            ['CN=Simple,DC=example,DC=net', 'CN=Simple'],
+            ['CN=James \"Jim\" Smith\, III,DC=example,DC=net', 'CN=James \"Jim\" Smith\, III'],
+            ['UID=jsmith,DC=example,DC=net', 'UID=jsmith'],
+            ["CN=Before\0dAfter,DC=example,DC=net", "CN=Before\0dAfter"],
+        ];
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public function addAttributeValues(Entry $entry, string $attribute, array $value`
`79`	`79`	`$con = $this->getConnectionResource();`
`80`	`80`
`81`	`81`	`if (!@ldap_mod_add($con, $entry->getDn(), [$attribute => $values])) {`
`82`		`- throw new LdapException(sprintf('Could not add values to entry "%s", attribute %s: ', $entry->getDn(), $attribute).ldap_error($con));`
	`82`	`+ throw new LdapException(sprintf('Could not add values to entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con));`
`83`	`83`	`}`
`84`	`84`	`}`
`85`	`85`
`@@ -94,7 +94,7 @@ public function removeAttributeValues(Entry $entry, string $attribute, array $va`
`94`	`94`	`$con = $this->getConnectionResource();`
`95`	`95`
`96`	`96`	`if (!@ldap_mod_del($con, $entry->getDn(), [$attribute => $values])) {`
`97`		`- throw new LdapException(sprintf('Could not remove values from entry "%s", attribute %s: ', $entry->getDn(), $attribute).ldap_error($con));`
	`97`	`+ throw new LdapException(sprintf('Could not remove values from entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con));`
`98`	`98`	`}`
`99`	`99`	`}`
`100`	`100`
`@@ -159,7 +159,7 @@ public function applyOperations(string $dn, iterable $operations): void`
`159`	`159`
`160`	`160`	`private function parseRdnFromEntry(Entry $entry): string`
`161`	`161`	`{`
`162`		`- if (!preg_match('/^([^,]+),/', $entry->getDn(), $matches)) {`
	`162`	`+ if (!preg_match('/(^[^,\\\\](?:\\\\.[^,\\\\])*),/', $entry->getDn(), $matches)) {`
`163`	`163`	`throw new LdapException(sprintf('Entry "%s" malformed, could not parse RDN.', $entry->getDn()));`
`164`	`164`	`}`
`165`	`165`