-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[samples/ffi] use-of-uninitialized-value in SQLite sample #44312
Labels
area-vm
Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
gardening
library-ffi
Comments
Thanks for reporting @sstrickl. Simple samples
This is indeed expected to fail on MSAN. SQLite
That one I have not investigated before. |
copybara-service bot
pushed a commit
that referenced
this issue
May 22, 2023
This allows it to be instrumented by the sanitizers. Enabled only for MSAN and for Android. TEST=ci Bug: #44312 Bug: #44377 Bug: #43075 Bug: #50248 Bug: #50271 Bug: #52441 Change-Id: I96241e6ee28fb2a853d4a113aac268bc415a5fd5 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/304147 Commit-Queue: Ryan Macnak <[email protected]> Reviewed-by: Daco Harkes <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
May 22, 2023
This reverts commit bd589d4. Reason for revert: breaks dart-sdk-linux-riscv64-main Original change's description: > [build] Build the standard c++ library from source. > > This allows it to be instrumented by the sanitizers. > > Enabled only for MSAN and for Android. > > TEST=ci > Bug: #44312 > Bug: #44377 > Bug: #43075 > Bug: #50248 > Bug: #50271 > Bug: #52441 > Change-Id: I96241e6ee28fb2a853d4a113aac268bc415a5fd5 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/304147 > Commit-Queue: Ryan Macnak <[email protected]> > Reviewed-by: Daco Harkes <[email protected]> Bug: #44312 Bug: #44377 Bug: #43075 Bug: #50248 Bug: #50271 Bug: #52441 Change-Id: I0b1d0c0da1cd77e0f9645facfc58397cc216c584 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/304823 Bot-Commit: Rubber Stamper <[email protected]> Auto-Submit: Ryan Macnak <[email protected]> Commit-Queue: Rubber Stamper <[email protected]>
copybara-service bot
pushed a commit
that referenced
this issue
May 22, 2023
This allows it to be instrumented by the sanitizers. Enabled only for MSAN and for Android. Don't pick up Flutter's including no_exceptions in the default config set. TEST=ci Bug: #44312 Bug: #44377 Bug: #43075 Bug: #50248 Bug: #50271 Bug: #52441 Change-Id: If01704ff29569fba8f8181ed31d52faba8d8370f Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/304824 Reviewed-by: Alexander Aprelev <[email protected]> Commit-Queue: Ryan Macnak <[email protected]>
Still failing:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area-vm
Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
gardening
library-ffi
There are new test failures on Migrator: Implement check boxes on directories...[test] Fixes for running tests on Fuchsia emulator.
The tests
are failing on configurations
TBH, I've forgotten how to check now if these were approved in the old location, but creating a bug and approving for now in case they were not. Feel free to close if they were approved there and/or an issue for that already exists. (Searched for ffi and MSAN and none of the issues that came up were directly related.)
/cc @dcharkes
The text was updated successfully, but these errors were encountered: