symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
+3Lines changed: 3 additions & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
+3Lines changed: 3 additions & 0 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveInstanceofConditionalsPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveInstanceofConditionalsPassTest.php
+23Lines changed: 23 additions & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveInstanceofConditionalsPassTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveInstanceofConditionalsPassTest.php
+23Lines changed: 23 additions & 0 deletions
@@ -107,6 +107,9 @@ private function processDefinition(ContainerBuilder $container, $id, Definition
             while (0 <= --$i) {
                 foreach ($instanceofTags[$i] as $k => $v) {
                     foreach ($v as $v) {
+                        if ($definition->hasTag($k) && in_array($v, $definition->getTag($k))) {
+                            continue;
+                        }
                         $definition->addTag($k, $v);
                     }
                 }
 
@@ -130,6 +130,29 @@ public function testProcessUsesAutoconfiguredInstanceof()
         $this->assertSame(array('local_instanceof_tag' => array(array()), 'autoconfigured_tag' => array(array())), $def->getTags());
     }
 
+    public function testAutoconfigureInstanceofDoesNotDuplicateTags()
+    {
+        $container = new ContainerBuilder();
+        $def = $container->register('normal_service', self::class);
+        $def
+            ->addTag('duplicated_tag')
+            ->addTag('duplicated_tag', array('and_attributes' => 1))
+        ;
+        $def->setInstanceofConditionals(array(
+            parent::class => (new ChildDefinition(''))->addTag('duplicated_tag'),
+        ));
+        $def->setAutoconfigured(true);
+        $container->registerForAutoconfiguration(parent::class)
+            ->addTag('duplicated_tag', array('and_attributes' => 1))
+        ;
+
+        (new ResolveInstanceofConditionalsPass())->process($container);
+        (new ResolveDefinitionTemplatesPass())->process($container);
+
+        $def = $container->getDefinition('normal_service');
+        $this->assertSame(array('duplicated_tag' => array(array(), array('and_attributes' => 1))), $def->getTags());
+    }
+
     public function testProcessDoesNotUseAutoconfiguredInstanceofIfNotEnabled()
     {
         $container = new ContainerBuilder();
Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,9 @@ private function processDefinition(ContainerBuilder $container, $id, Definition`
`107`	`107`	`while (0 <= --$i) {`
`108`	`108`	`foreach ($instanceofTags[$i] as $k => $v) {`
`109`	`109`	`foreach ($v as $v) {`
	`110`	`+ if ($definition->hasTag($k) && in_array($v, $definition->getTag($k))) {`
	`111`	`+ continue;`
	`112`	`+ }`
`110`	`113`	`$definition->addTag($k, $v);`
`111`	`114`	`}`
`112`	`115`	`}`