-
Notifications
You must be signed in to change notification settings - Fork 13.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
crust functions passed as arguments can't be treated as *u8 #3239
Labels
Comments
Soon this will be obsolete since the type of crust functions won't be |
Visiting for triage; still relevant. |
(bug triage) Is this still relevant? @pcwalton ? |
Now you need to pass extern "C" fn, so this should be moot. |
RalfJung
pushed a commit
to RalfJung/rust
that referenced
this issue
Jan 4, 2024
…Jung Use `windows-sys` in windows tests This PR adds `windows-sys` to `test_dependencies` so that we don't have to write out windows api bindings for each test.
The job Click to see the possible cause of the failure (guessed by this bot)
|
jaisnan
pushed a commit
to jaisnan/rust-dev
that referenced
this issue
Jul 29, 2024
The current method for creating the modifies wrapper requires changing the `ensures` clause to have `_renamed` variables which are unsafe copies of the original function arguments. This causes issues with regards to some possible tests as in rust-lang#3239. This change removes the `_renamed` variables and instead simply changes the modifies clauses within the replace to unsafely dereference the pointer to modify the contents of it unsafely, condensing all instances of unsafe Rust into a single location. Resolves rust-lang#3239 Resolves rust-lang#3026 May affect rust-lang#3027. In my attempt to run this example with slight modification to fit the current implementation, I got the error `CBMC appears to have run out of memory. You may want to rerun your proof in an environment with additional memory or use stubbing to reduce the size of the code the verifier reasons about.` This suggests that the compilation is working appropriately but the test case is just too large for CBMC. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses. Co-authored-by: Matias Scharager <mscharag@amazon.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Attempts to use finalize inside a JSClass struct (which expects a *u8) fail, whereas inlining this code in the caller work fine.
The text was updated successfully, but these errors were encountered: