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

fix: add string as return for getCover and make sure to not pass false to fseek

Browse Source 
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
This commit is contained in:
Daniel Kesselberg 2023-05-28 16:33:05 +02:00
parent cbbd6b306b
commit c75f4e0a39
No known key found for this signature in database
GPG Key ID: 36E3664E099D0614
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

@ -233,7 +233,7 @@ class Mp3Info {
/** /**
* @return bool|null * @return bool|null|string
*/ */
public function getCover() public function getCover()
{ {
@ -242,6 +242,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);