Stream: git-wasmtime

Topic: wasmtime / PR #2517 cranelift-object: reject symbol names...


view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 02:54):

cfallin opened PR #2517 from fix-null-byte-symbols to main:

Avoid a panic!(), and return a proper error, on a NUL byte. We hit a
null-byte check inside the object crate otherwise; this blocks fuzzing
when testing via a write-object-file-and-dlopen flow.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 02:54):

cfallin requested alexcrichton and abrown for a review on PR #2517.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 02:54):

cfallin requested alexcrichton and abrown for a review on PR #2517.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 15:06):

alexcrichton submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 15:06):

alexcrichton submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 15:06):

alexcrichton created PR Review Comment:

It might be better to use :? here which will show the nul byte perhaps?

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 17:39):

cfallin updated PR #2517 from fix-null-byte-symbols to main:

Avoid a panic!(), and return a proper error, on a NUL byte. We hit a
null-byte check inside the object crate otherwise; this blocks fuzzing
when testing via a write-object-file-and-dlopen flow.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 17:41):

cfallin submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 17:41):

cfallin created PR Review Comment:

Done.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 17:54):

cfallin updated PR #2517 from fix-null-byte-symbols to main:

Avoid a panic!(), and return a proper error, on a NUL byte. We hit a
null-byte check inside the object crate otherwise; this blocks fuzzing
when testing via a write-object-file-and-dlopen flow.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Dec 17 2020 at 18:33):

cfallin merged PR #2517.


Last updated: Dec 23 2024 at 12:05 UTC