eregi replacement help

[synking]

I am having trouble trying to fix my eregi call in my script.

 

I have a reporting tool that grabs information from the user agent when they email help questions.  I used eregi to find the string in an array and print out when it finds a match.

 

I don't fully understand the PCRE functions so i don' t know if i am using them right.

 

what i have is

foreach($OSList as $CurrOS=>$Match) {
        // Find a match
        if (eregi($Match, $_SERVER['HTTP_USER_AGENT']))
        {
                // We found the correct match
                break;
        }

        
}

 

I know i need to change this around for PCRE preg_match maybe but i am not sure.  if anyone could send me a tutorial or help me out it would be great.

 

EDIT: notice the foreach was not correct

[ManiacDan]

Without seeing the contents of $Match, all we can tell you is that you're calling the ereg function properly, but you shouldn't be using it in the first place.

 

-Dan

[synking]

oh sorry and thanks.  I actually just found that the script i am using is here

 

http://pastebin.com/qBW6pyzv

 

I did not build the tool it was made by another person.

 

The $Match is the array $oslist  I would like to be able to update this script if anyone can point me in the right direction.

 

Also what it does if i just swap eregi with preg_match is print out the last option in the array.

[Maq]

eregi has been deprecated as of 5.3, you should not use this function.

 

If $match is not a regular expression you should not be using any regex function.

If you do need a regex function then use preg_match.

If you just want to check if $match exists anywhere in the USER_AGENT then use strpos.

[ManiacDan]

Yeah...you want:

if ( strpos( $_SERVER['HTTP_USER_AGENT'], $Match ) !== false )

-Dan