Grouped Backports to the 5.0 branch.

- Install: When populating options, maybe_serialize instead of always serialize.
- Uploads: Check for and verify ZIP archives.

Merges [57388] and [57389] to the 5.0 branch.

Props costdev, peterwilsoncc, azaozz, tykoted, johnbillion, desrosj, afragen, jorbin, xknown.

Built from https://develop.svn.wordpress.org/branches/5.0@57405


git-svn-id: http://core.svn.wordpress.org/branches/5.0@56911 1a063a9b-81f0-0310-95a4-ce76da25c4cd

wp-admin/includes/class-file-upload-upgrader.php

@@ -63,6 +63,30 @@ class File_Upload_Upgrader {
 			if ( isset( $file['error'] ) )
 				wp_die( $file['error'] );
 
+			if ( 'pluginzip' === $form || 'themezip' === $form ) {
+				$archive_is_valid = false;
+
+				/** This filter is documented in wp-admin/includes/file.php */
+				if ( class_exists( 'ZipArchive', false ) && apply_filters( 'unzip_file_use_ziparchive', true ) ) {
+					$archive          = new ZipArchive();
+					$archive_is_valid = $archive->open( $file['file'], ZIPARCHIVE::CHECKCONS );
+
+					if ( true === $archive_is_valid ) {
+						$archive->close();
+					}
+				} else {
+					require_once ABSPATH . 'wp-admin/includes/class-pclzip.php';
+
+					$archive          = new PclZip( $file['file'] );
+					$archive_is_valid = is_array( $archive->properties() );
+				}
+
+				if ( true !== $archive_is_valid ) {
+					wp_delete_file( $file['file'] );
+					wp_die( __( 'Incompatible Archive.' ) );
+				}
+			}
+
 			$this->filename = $_FILES[$form]['name'];
 			$this->package = $file['file'];
 

wp-admin/includes/schema.php

@@ -552,10 +552,11 @@ function populate_options() {
 		else
 			$autoload = 'yes';
 
-		if ( is_array($value) )
-			$value = serialize($value);
 		if ( !empty($insert) )
 			$insert .= ', ';
+
+		$value = maybe_serialize( sanitize_option( $option, $value ) );
+
 		$insert .= $wpdb->prepare( "(%s, %s, %s)", $option, $value, $autoload );
 	}
 

wp-admin/update.php

@@ -146,6 +146,10 @@ if ( isset($_GET['action']) ) {
 
 		check_admin_referer('plugin-upload');
 
+		if ( isset( $_FILES['pluginzip']['name'] ) && ! str_ends_with( strtolower( $_FILES['pluginzip']['name'] ), '.zip' ) ) {
+			wp_die( __( 'Only .zip archives may be uploaded.' ) );
+		}
+
 		$file_upload = new File_Upload_Upgrader('pluginzip', 'package');
 
 		$title = __('Upload Plugin');
@@ -249,6 +253,10 @@ if ( isset($_GET['action']) ) {
 
 		check_admin_referer('theme-upload');
 
+		if ( isset( $_FILES['themezip']['name'] ) && ! str_ends_with( strtolower( $_FILES['themezip']['name'] ), '.zip' ) ) {
+			wp_die( __( 'Only .zip archives may be uploaded.' ) );
+		}
+
 		$file_upload = new File_Upload_Upgrader('themezip', 'package');
 
 		$title = __('Upload Theme');