james/fembooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #747 from sanmadjack/ffmpeg_thumb_temp_file

Browse Source 
Added handling to ensure that video thumbnails don't leave temp files…
This commit is contained in:
Shish 2020-09-16 11:20:51 +01:00 committed by GitHub
commit 111c4e3fb5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 35 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ext/media/main.php
View File

@ -328,6 +328,7 @@ class Media extends Extension
$inname = warehouse_path(Image::IMAGE_DIR, $hash);
$tmpname = tempnam(sys_get_temp_dir(), "shimmie_ffmpeg_thumb");
try {
$outname = warehouse_path(Image::THUMBNAIL_DIR, $hash);
$orig_size = self::video_size($inname);
@ -359,7 +360,7 @@ class Media extends Extension
exec($cmd, $output, $ret);
if ((int)$ret == (int)0) {
if ((int)$ret === (int)0) {
log_debug('media', "Generating thumbnail with command `$cmd`, returns $ret");
create_scaled_image($tmpname, $outname, $scaled_size, MimeType::PNG);
@ -370,6 +371,9 @@ class Media extends Extension
log_error('media', "Generating thumbnail with command `$cmd`, returns $ret");
return false;
}
} finally {
@unlink($tmpname);
}
}