symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/Container.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Container.php
+3-1Lines changed: 3 additions & 1 deletion b/‎src/Symfony/Component/DependencyInjection/Container.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Container.php
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
+17Lines changed: 17 additions & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
+17Lines changed: 17 additions & 0 deletions
@@ -469,7 +469,9 @@ protected function getEnv($name)
             return $this->envCache[$name];
         }
         if (!$this->has($id = 'container.env_var_processors_locator')) {
-            $this->set($id, new ServiceLocator(array()));
+            // We don't call set() to prevent an infinite loop:
+            // set() calls getRemovedIds(), getRemovedIds() calls getEnv() in some cases
+            $this->services[$id] = new ServiceLocator(array());
         }
         if (!$this->getEnv) {
             $this->getEnv = new \ReflectionMethod($this, __FUNCTION__);
 
@@ -538,6 +538,23 @@ public function testReplacingAPreDefinedServiceIsDeprecated()
 
         $this->assertSame($bar, $c->get('bar'), '->set() replaces a pre-defined service');
     }
+
+    public function testNoInfiniteLoopInGetEnv()
+    {
+        $_ENV['bar'] = 'hello';
+        $this->assertCount(1, (new RemovedIdsCallGetEnvContainer())->getRemovedIds());
+        unset($_ENV['bar']);
+    }
+}
+
+class RemovedIdsCallGetEnvContainer extends Container
+{
+    public function getRemovedIds()
+    {
+        return array(
+            'foo.'.$this->getEnv('bar') => true,
+        );
+    }
 }
 
 class ProjectServiceContainer extends Container
Original file line number	Diff line number	Diff line change
`@@ -469,7 +469,9 @@ protected function getEnv($name)`
`469`	`469`	`return $this->envCache[$name];`
`470`	`470`	`}`
`471`	`471`	`if (!$this->has($id = 'container.env_var_processors_locator')) {`
`472`		`- $this->set($id, new ServiceLocator(array()));`
	`472`	`+ // We don't call set() to prevent an infinite loop:`
	`473`	`+ // set() calls getRemovedIds(), getRemovedIds() calls getEnv() in some cases`
	`474`	`+ $this->services[$id] = new ServiceLocator(array());`
`473`	`475`	`}`
`474`	`476`	`if (!$this->getEnv) {`
`475`	`477`	`$this->getEnv = new \ReflectionMethod($this, __FUNCTION__);`