fluencelabs/parity-wasm
1
0
Fork 0
mirror of https://github.com/fluencelabs/parity-wasm synced 2025-07-30 22:01:58 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #197 from paritytech/fix-peek

Browse Source 
Fix peek size on edge case
This commit is contained in:
Nikolay Volf
2018-03-08 06:00:56 +03:00
committed by GitHub
parent 6dc7f54ab8 9bb84a9111
commit f252c92d02
2 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

BIN
res/cases/v1/peek_sample.wasm Normal file
View File

Binary file not shown.

15
src/elements/module.rs
View File

@@ -342,7 +342,10 @@ pub fn peek_size(source: &[u8]) -> usize {
if section_id <= 11 && section_len > 0 {
let next_cursor = cursor + new_cursor + section_len as usize;
if next_cursor >= source.len() {
if next_cursor > source.len() {
break;
} else if next_cursor == source.len() {
cursor = next_cursor;
break;
}
cursor = next_cursor;
@@ -498,6 +501,16 @@ mod integration_tests {
assert_eq!(peek_size(&buf), buf.len() - 9);
}
#[test]
fn peek_3() {
use super::peek_size;
let module = deserialize_file("./res/cases/v1/peek_sample.wasm").expect("Should be deserialized");
let buf = serialize(module).expect("serialization to succeed");
assert_eq!(peek_size(&buf), buf.len());
}
#[test]
fn module_default_round_trip() {
let module1 = Module::default();