wapmorgan\UnifiedArchive\Formats\Gzip::gzipStat()

Complexity

Conditions	3
Paths	2

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function gzipStat($file)
    {
        $fp = fopen($file, 'rb');
        if (filesize($file) < 18 || strcmp(fread($fp, 2), "\x1f\x8b")) {

It seems like $fp can also be of type false; however, parameter $handle of fread() does only seem to accept resource, maybe add an additional type check?

            return false;  // Not GZIP format (See RFC 1952)
        }
        $method = fread($fp, 1);

The assignment to $method is dead and can be removed.

        $flags = fread($fp, 1);

The assignment to $flags is dead and can be removed.

        $stat = unpack('Vmtime', fread($fp, 4));
        fseek($fp, -4, SEEK_END);

It seems like $fp can also be of type false; however, parameter $handle of fseek() does only seem to accept resource, maybe add an additional type check?

        $stat += unpack('Vsize', fread($fp, 4));
        fclose($fp);

It seems like $fp can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

        return $stat;
    }
}