MimeMagic: Prevent PHP warning when trying to read zero bytes
authorBartosz Dziewoński <matma.rex@gmail.com>
Thu, 24 Mar 2016 09:06:34 +0000 (10:06 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Thu, 24 Mar 2016 09:14:31 +0000 (10:14 +0100)
Honestly, there's no reason why fread() should warn when asked to read
zero bytes. It should just return zero bytes of content. Eh, PHP.

Bug: T130813
Change-Id: I56c4826759390bdbf6af04da28e2d9ed07781bca

includes/MimeMagic.php

index aedfd8f..a432d44 100644 (file)
@@ -645,7 +645,7 @@ class MimeMagic {
                        throw new MWException(
                                "Seeking $tailLength bytes from EOF failed in " . __METHOD__ );
                }
-               $tail = fread( $f, $tailLength );
+               $tail = $tailLength ? fread( $f, $tailLength ) : '';
                fclose( $f );
 
                wfDebug( __METHOD__ . ": analyzing head and tail of $file for magic numbers.\n" );