fixes

substrate/frame/contracts/src/wasm/mod.rs

@@ -210,7 +210,6 @@ impl<T: Config> WasmBlob<T> {
 		host_state: H,
 		schedule: &Schedule<T>,
 		allow_deprecated: AllowDeprecatedInterface,
-		compilation_mode: CompilationMode,
 	) -> Result<(Store<H>, Memory, InstancePre), &'static str>
 	where
 		E: Environment<H>,
@@ -366,6 +365,7 @@ impl<T: Config> Executable<T> for WasmBlob<T> {
 			&code,
 			self.code_info.determinism,
 			Some(StackLimits::default()),
+			CompilationMode::Lazy,
 			false,
 		)
 		.map_err(|err| {
@@ -381,7 +381,6 @@ impl<T: Config> Executable<T> for WasmBlob<T> {
 				ExportedFunction::Call => AllowDeprecatedInterface::Yes,
 				ExportedFunction::Constructor => AllowDeprecatedInterface::No,
 			},
-			CompilationMode::Lazy,
 		)
 		.map_err(|msg| {
 			log::debug!(target: LOG_TARGET, "failed to instantiate code to wasmi: {}", msg);

substrate/frame/contracts/src/wasm/prepare.rs

@@ -94,15 +94,16 @@ impl LoadedModule {
 		let module = if validate_module {
 			Module::new(&engine, code)
 		} else {
-			unsafe { Module::new_unsafe(&engine, code) }
+			// Safety: The code has been validated, Therefore we know that it's a valid binary.
+			unsafe { Module::new_unchecked(&engine, code) }
 		}
 		.map_err(|err| {
 			log::debug!(target: LOG_TARGET, "Module creation failed: {:?}", err);
 			"Can't load the module into wasmi!"
 		})?;
 
 		#[cfg(test)]
-		LOADED_MODULE_TRACKER.with(|t| t.borrow_mut().push(_validate_module));
+		LOADED_MODULE_TRACKER.with(|t| t.borrow_mut().push(validate_module));
 
 		// Return a `LoadedModule` instance with
 		// __valid__ module.