Skip to content
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

UB-free analysis/test WIP #650

Closed

Conversation

kkysen
Copy link
Contributor

@kkysen kkysen commented Aug 26, 2022

…an't reproduce the failures locally. Let's see about CI.
@kkysen kkysen marked this pull request as draft August 26, 2022 06:39
@kkysen
Copy link
Contributor Author

kkysen commented Aug 26, 2022

Well, this is still failing CI, welp.

Some ideas for investigating further that we discussed:

  • Trying to replicate the CI environment locally as much as possible, like running with reduced core count and under heavy load.
  • Check to see if rustc and our instrumentation is being deterministic.
  • Check to see if any events are being lost. Perhaps try processing the events on the main thread to keep things single-threaded, as perhaps there's a bug in there, especially around finalization when the main thread finishes first.
  • See if rr could be used to debug post-mortem.

kkysen added a commit that referenced this pull request Aug 29, 2022
…-args: Fix leaking `args` in `analysis/test`'s `main`

This fixes the memory leakage of `args` in `main` in `analysis/test` by not `.into_raw()`ing the `CString`s, but storing them in a `Vec` and using `.as_ptr()`.

Checked by `miri` (in #650) to remove the leaks created in `main`.
@kkysen kkysen mentioned this pull request Sep 25, 2022
@aneksteind
Copy link
Contributor

What's the high level difference between this and #683 ?

@kkysen
Copy link
Contributor Author

kkysen commented Sep 27, 2022

What's the high level difference between this and #683 ?

This PR is just everything thrown together in 1 commit. There might be some other changes added in here, too.

Also, #683 only fixes the UB. Other PRs fix the rest of the things in here.

@kkysen kkysen closed this Oct 10, 2022
@kkysen kkysen deleted the kkysen/ub-free-analysis-test-wip branch October 10, 2022 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants