sbrl-github/Pepperminty-Wiki
sbrl-github/Pepperminty-Wiki
1
0
Fork 0
mirror of https://github.com/sbrl/Pepperminty-Wiki.git synced 2024-12-22 01:35:02 +00:00
Code Activity

Use tempnam() instead of tmpfile() when unpacking extra data

Browse source 
It seems that some people were experiencing some strange issues with
stream_get_meta_data($handle)["uri"] - hrm 🤔
This commit is contained in:
Starbeamrainbowlabs 2019-09-11 23:44:59 +01:00
parent 6e8e35748b
commit 10b94ce2ef
Signed by: sbrl
GPG key ID: 1BE5172E637709C2
1 changed files with 7 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
core/07-unpack.php
View file

@ -22,11 +22,12 @@ if(!file_exists($paths->extra_data_directory) ||
http_response_code(503); http_response_code(503);
exit(page_renderer::render_minimal("Unpacking error - $settings->sitename", "<p>Oops! It looks like $settings->sitename isn't able to change the last modified time of the extra data directory.</p>$error_message_help")); exit(page_renderer::render_minimal("Unpacking error - $settings->sitename", "<p>Oops! It looks like $settings->sitename isn't able to change the last modified time of the extra data directory.</p>$error_message_help"));
} }
$temp_file = tmpfile(); $temp_filename = tempnam(sys_get_temp_dir(), "PeppermintExtract");
$temp_file = fopen($temp_filename, "wb+");
if($temp_file === false) { if($temp_file === false) {
http_response_code(503); http_response_code(503);
exit(page_renderer::render_minimal("Unpacking error - $settings->sitename", "<p>Oops! $settings->sitename wasn't able to create a new temporary file with <code>tmpfile()</code>. Perhaps your server is mis-configured?</p>")); exit(page_renderer::render_minimal("Unpacking error - $settings->sitename", "<p>Oops! $settings->sitename wasn't able to create a new temporary file with <code>tempnam()</code>. Perhaps your server is mis-configured?</p>"));
} }
$source = fopen(__FILE__, "r"); $source = fopen(__FILE__, "r");
if($source === false) { if($source === false) {
@ -36,17 +37,14 @@ if(!file_exists($paths->extra_data_directory) ||
fseek($source, __COMPILER_HALT_OFFSET__); fseek($source, __COMPILER_HALT_OFFSET__);
stream_copy_to_stream($source, $temp_file); stream_copy_to_stream($source, $temp_file);
fclose($temp_file);
$temp_filename = stream_get_meta_data($temp_file)["uri"];
$extractor = new ZipArchive(); $extractor = new ZipArchive();
$extractor->open($temp_filename); $extractor->open($temp_filename);
$extractor->extractTo($paths->extra_data_directory); $extractor->extractTo($paths->extra_data_directory);
$extractor->close(); $extractor->close();
if(fclose($temp_file) === false) {
http_response_code(503); unlink($temp_filename);
exit(page_renderer::render_minimal("Unpacking error - $settings->sitename", "<p>Oops! $settings->sitename wasn't able to close the temporary file that it created with <code>tmpfile()</code>. Perhaps your server is mis-configured?</p>"));
}
unset($error_message_help); unset($error_message_help);
} }