Deny the usage of animated avatars

[iblis]

Could you guys try the following code, and report back bugs if there are any?
This snipplet detects if an avatar is animated (or any other .gif)

PHP Code:

function isAnimated($filename)
{
    $fp = fopen($filename, "rb");
    $whole = fread($fp, filesize($filename));
    $i = -1;
    while($i = strpos($whole, 0x2C, $i+1))
    {
        if(strpos($whole, 0x00, $i-1) == $i-1)
            $c++;
    }
    if($c > 1)
        return true;
    return false;
} 


[tehste]

do php if statements work without brackets and without telling an if to end?

PHP Code:

function isAnimated($filename)
{
    $fp = fopen($filename, "rb");
    $whole = fread($fp, filesize($filename));
    $i = -1;
    while($i = strpos($whole, 0x2C, $i+1))
    {
        if(strpos($whole, 0x00, $i-1) == $i-1) {
            $c++;
        }
    }
    if($c > 1) {
        return true;
    } else {
    return false;
    }
} 


[iblis]

Yes they do.
If you do not enclose the content in brackets, it assumes the next line is to be executed if the statement returns true - else it ignores it

[tehste]

Originally Posted by iblis

Yes they do.
If you do not enclose the content in brackets, it assumes the next line is to be executed if the statement returns true - else it ignores it

you learn something new everyday