Fix #50678: files extracted by ZipArchive class lost their original mtime

ext/zip/php_zip.c

@@ -150,6 +150,9 @@ static int php_zip_extract_file(struct zip * za, char *dest, char *file, int fil
 	size_t path_cleaned_len;
 	cwd_state new_state;
 	zend_string *file_basename;
+	struct utimbuf newtimebuf;
+	struct utimbuf *newtime = &newtimebuf;
+	php_stream_wrapper *wrapper;
 
 	new_state.cwd = CWD_STATE_ALLOC(1);
 	new_state.cwd[0] = '\0';
@@ -262,6 +265,14 @@ static int php_zip_extract_file(struct zip * za, char *dest, char *file, int fil
 	php_stream_close(stream);
 	n = zip_fclose(zf);
 
+	/* try to set mtime of extracted file; don't mind if that fails */
+	newtime->modtime = sb.mtime;
+	newtime->actime = time(NULL);
+	wrapper = php_stream_locate_url_wrapper(fullpath, NULL, 0);
+	if (wrapper && wrapper->wops) {
+		wrapper->wops->stream_metadata(wrapper, fullpath, PHP_STREAM_META_TOUCH, newtime, NULL);
+	}
+
 done:
 	efree(fullpath);
 	zend_string_release(file_basename);

ext/zip/tests/bug50678.phpt

@@ -0,0 +1,26 @@
+--TEST--
+Bug #50678 (files extracted by ZipArchive class loose their original modified time)
+--SKIPIF--
+<?php
+if (!extension_loaded('zip')) print 'skip zip extension not available';
+?>
+--FILE--
+<?php
+$dest = __DIR__ . '/bug50678';
+mkdir($dest);
+$zip = new ZipArchive();
+$zip->open(__DIR__ . '/bug50678.zip');
+$zip->extractTo($dest);
+$zip->close();
+$filename = $dest . '/bug50678.txt';
+// check that the mtime is properly set, if the extracted file is writable
+var_dump(!is_writable($filename) || filemtime($filename) == 1432163274);
+?>
+--CLEAN--
+<?php
+$dest = __DIR__ . '/bug50678';
+unlink($dest . '/bug50678.txt');
+rmdir($dest);
+?>
+--EXPECT--
+bool(true)