Merge pull request #2801 from peterhuene/fix-initialize-instance-range
Fix incorrect range in `ininitialize_instance`.
This commit is contained in:
@@ -379,7 +379,9 @@ fn initialize_instance(
|
|||||||
for (page_index, page) in pages.iter().enumerate() {
|
for (page_index, page) in pages.iter().enumerate() {
|
||||||
if let Some(data) = page {
|
if let Some(data) = page {
|
||||||
debug_assert_eq!(data.len(), WASM_PAGE_SIZE as usize);
|
debug_assert_eq!(data.len(), WASM_PAGE_SIZE as usize);
|
||||||
slice[page_index * WASM_PAGE_SIZE as usize..].copy_from_slice(data);
|
let start = page_index * WASM_PAGE_SIZE as usize;
|
||||||
|
let end = start + WASM_PAGE_SIZE as usize;
|
||||||
|
slice[start..end].copy_from_slice(data);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -11,3 +11,23 @@ fn wrong_import_numbers() -> Result<()> {
|
|||||||
assert!(Instance::new(&store, &module, &[func.clone().into(), func.into()]).is_err());
|
assert!(Instance::new(&store, &module, &[func.clone().into(), func.into()]).is_err());
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn initializes_linear_memory() -> Result<()> {
|
||||||
|
// Test for https://github.com/bytecodealliance/wasmtime/issues/2784
|
||||||
|
let wat = r#"
|
||||||
|
(module
|
||||||
|
(memory (export "memory") 2)
|
||||||
|
(data (i32.const 0) "Hello World!")
|
||||||
|
)"#;
|
||||||
|
let module = Module::new(&Engine::default(), wat)?;
|
||||||
|
|
||||||
|
let store = Store::new(module.engine());
|
||||||
|
let instance = Instance::new(&store, &module, &[])?;
|
||||||
|
let memory = instance.get_memory("memory").unwrap();
|
||||||
|
|
||||||
|
let mut bytes = [0; 12];
|
||||||
|
memory.read(0, &mut bytes)?;
|
||||||
|
assert_eq!(bytes, "Hello World!".as_bytes());
|
||||||
|
Ok(())
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user