Skip to content

memanalyze.pl: fix getaddrinfo/freeaddrinfo checks#17503

Closed
bagder wants to merge 1 commit intomasterfrom
bagder/memanalyze-addr
Closed

memanalyze.pl: fix getaddrinfo/freeaddrinfo checks#17503
bagder wants to merge 1 commit intomasterfrom
bagder/memanalyze-addr

Conversation

@bagder
Copy link
Member

@bagder bagder commented May 31, 2025

It did not parse them correctly. Problem never shows because we don't run torture tests with these functions - yet.

Seen when enabling torture tests with the threaded resolver.

It did not parse them correctly. Problem never shows before we don't run
torture tests with these functions - yet.

Seen when enabling torture tests with the threaded resolver.
@bagder bagder added the tests label May 31, 2025
@bagder bagder marked this pull request as ready for review May 31, 2025 21:42
@testclutch
Copy link

Analysis of PR #17503 at 58285a21:

Test 1635 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 6 different CI jobs (the link just goes to one of them).

Generated by Testclutch

@bagder bagder closed this in 7011576 Jun 1, 2025
@bagder bagder deleted the bagder/memanalyze-addr branch June 1, 2025 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants

Comments