upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-09 08:44:07 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Merge pull request #38496 from nextcloud/preview-mp3-catch-errors

Browse source 
fix: catch errors in id3parser library
This commit is contained in:
Simon L 2023-05-30 10:51:33 +02:00 committed by GitHub
commit 1e1e2fec64
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
lib/private/Preview/MP3.php
View file

@ -32,6 +32,7 @@ use ID3Parser\ID3Parser;
use OCP\Files\File;
use OCP\IImage;
use Psr\Log\LoggerInterface;
class MP3 extends ProviderV2 {
/**
@ -48,8 +49,18 @@ class MP3 extends ProviderV2 {
$getID3 = new ID3Parser();
$tmpPath = $this->getLocalFile($file);
$tags = $getID3->analyze($tmpPath);
$this->cleanTmpFiles();
try {
$tags = $getID3->analyze($tmpPath);
} catch (\Throwable $e) {
\OC::$server->get(LoggerInterface::class)->info($e->getMessage(), [
'exception' => $e,
'app' => 'core',
]);
return null;
} finally {
$this->cleanTmpFiles();
}
$picture = isset($tags['id3v2']['APIC'][0]['data']) ? $tags['id3v2']['APIC'][0]['data'] : null;
if (is_null($picture) && isset($tags['id3v2']['PIC'][0]['data'])) {
$picture = $tags['id3v2']['PIC'][0]['data'];