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

CFE-4454: Fixed ODR violations in ASAN Unit Tests #5656

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

larsewi
Copy link
Contributor

@larsewi larsewi commented Dec 9, 2024

  • Fixed ODR violations in ASAN Unit Tests
  • Revert "Disabled ODR violation from ASAN Unit Tests"

@larsewi larsewi added the WIP Work in Progress label Dec 9, 2024
@larsewi larsewi changed the title CFE-4454: Fix ODR violations in ASAN Unit Tests CFE-4454: Fixed ODR violations in ASAN Unit Tests Dec 9, 2024
@larsewi larsewi force-pushed the violation branch 9 times, most recently from cdd32a8 to 3ffe1ac Compare December 12, 2024 16:37
Ticket: CFE-4454
Changelog: None
Signed-off-by: Lars Erik Wik <[email protected]>
This reverts commit 1589aa3.

Ticket: CFE-4454
Changelog: None
Signed-off-by: Lars Erik Wik <[email protected]>
For some reason rlist_test fails in macOS unit tests. The test seems to
`abort()` while it should trigger an assert so that
`expect_assert_failure()` can catch it. For some reason this is not
happening anymore.

```
rlist.c:135: Programming Error: Internal error: Rval contains type f instead of expected scalar
```

I created a ticket (CFE-4473) to follow-up on this.

Ticket: None
Changelog: None
Signed-off-by: Lars Erik Wik <[email protected]>
@larsewi larsewi marked this pull request as ready for review December 13, 2024 11:48
@larsewi larsewi removed the WIP Work in Progress label Dec 13, 2024
@larsewi
Copy link
Contributor Author

larsewi commented Dec 13, 2024

@cf-bottom Jenkins please :)

@cf-bottom
Copy link

@larsewi
Copy link
Contributor Author

larsewi commented Dec 13, 2024

Debian 11 failure is unrelated

@larsewi larsewi merged commit 04e8624 into cfengine:master Dec 16, 2024
42 of 43 checks passed
@larsewi larsewi deleted the violation branch December 18, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants