mbirth/php-mp3info
1
0
Fork 0
Code Issues Pull Requests Activity

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

Browse Source 
fix: add string as return for getCover and make sure to not pass false to fseek
This commit is contained in:
Sergey 2023-05-28 21:05:37 +03:00 committed by GitHub
commit a28101d523
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Mp3Info.php
View File

@ -235,7 +235,7 @@ class Mp3Info {
/** /**
* @return bool|null * @return bool|null|string
*/ */
public function getCover() public function getCover()
{ {
@ -244,6 +244,9 @@ class Mp3Info {
} }
$fp = fopen($this->_fileName, 'rb'); $fp = fopen($this->_fileName, 'rb');
if ($fp === false) {
return false;
}
fseek($fp, $this->coverProperties['offset']); fseek($fp, $this->coverProperties['offset']);
$data = fread($fp, $this->coverProperties['size']); $data = fread($fp, $this->coverProperties['size']);
fclose($fp); fclose($fp);