[Typo3] Bug in newloginbox?

[Ries van Twisk]

Markus, peter,

I found that bug already a couple of months ago and communicated this 
with the current developer (not Kasper).
Please check if you have the latest version of newloginbox (I have 
version 2.2.9) before posting this to the everly growing bIgs.typo3.org.

cheers,
Ries

My lines:
87:             // Get storage PIDs:
88:        $spid=$GLOBALS['TSFE']->getStorageSiterootPids();
...
..
106:                foreach(split(',', $this->conf['domains']) as 
$d)       {
107:                    if (ereg('(^|\.)'.$d.'$',$redirect_domain))     {

So no collision there....


>Markus Stauffiger wrote:
>
>  
>
>>Hi Typo3-list
>>
>>I think I've discovered a bug in the newloginbox extension, but as I'm not 
>>completey sure about I'd like you to take a look:
>>
>>class.tx_newloginbox_pi1.php
>>
>>Line 88:
>>$d = $GLOBALS['TSFE']->getStorageSiterootPids();
>>
>>So far so good, but now on Line 106, $d is used as a "temporary" variable in 
>>a loop
>>foreach(split(',', $this->conf['domains']) as $d)
>>
>>So, of course, if the script tries to use $d afterwards as it won't find the 
>>Information (e.g. $d['_STORAGE_PID'])
>>
>>What do you think?
>>
>>Greetings
>>Markus
>>    
>>
>
>It might cause problems: file a bug at bugs.typo3.org if not already there.
>
>Regs. Peter.
>
>
>  
>