Merge pull request #28 from kesselb/guard-get-cover

fix: add string as return for getCover and make sure to not pass false to fseek
wapmorgan · May 28, 2023 · a28101d · a28101d
2 parents 534c305 + c75f4e0
commit a28101d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/Mp3Info.php b/src/Mp3Info.php
@@ -235,7 +235,7 @@ public function __construct($filename, $parseTags = false) {
 
 
     /**
-     * @return bool|null
+     * @return bool|null|string
      */
     public function getCover()
     {
@@ -244,6 +244,9 @@ public function getCover()
         }
 
         $fp = fopen($this->_fileName, 'rb');
+        if ($fp === false) {
+            return false;
+        }
         fseek($fp, $this->coverProperties['offset']);
         $data = fread($fp, $this->coverProperties['size']);
         fclose($fp);