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

fuzz: set nMaxOutboundLimit in connman target #29172

Merged

Conversation

brunoerg
Copy link
Contributor

@brunoerg brunoerg commented Jan 3, 2024

Setting nMaxOutboundLimit (-maxuploadtarget) will make fuzz to reach more coverage in connman target. This value is used in GetMaxOutboundTimeLeftInCycle, OutboundTargetReached and GetOutboundTargetBytesLeft.

@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 3, 2024

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK dergoegge, jonatack
Stale ACK maflcko

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #28584 (Fuzz: extend CConnman tests by vasild)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@DrahtBot DrahtBot added the Tests label Jan 3, 2024
@brunoerg
Copy link
Contributor Author

brunoerg commented Jan 3, 2024

CI failure is unrelated to this PR.

@maflcko
Copy link
Member

maflcko commented Jan 4, 2024

lgtm ACK 46d7113

@brunoerg
Copy link
Contributor Author

brunoerg commented Jan 4, 2024

friendly ping: @dergoegge

src/test/fuzz/connman.cpp Outdated Show resolved Hide resolved
src/test/fuzz/connman.cpp Outdated Show resolved Hide resolved
@brunoerg brunoerg force-pushed the 2024-fuzz-connman-maxuploadtarget branch from 46d7113 to e5b9ee0 Compare January 5, 2024 15:44
@brunoerg
Copy link
Contributor Author

brunoerg commented Jan 5, 2024

Thanks, @dergoegge for your review. Force-pushed addressing: #29172 (comment) and #29172 (comment).

Copy link
Member

@dergoegge dergoegge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK e5b9ee0

@DrahtBot DrahtBot requested a review from maflcko January 5, 2024 17:24
Copy link
Contributor

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK e5b9ee0

@fanquake fanquake merged commit f921d94 into bitcoin:master Jan 9, 2024
16 checks passed
@achow101
Copy link
Member

achow101 commented Jan 9, 2024

New compiler warning, using gcc 13.2.1

test/fuzz/connman.cpp: In function ‘void connman_fuzz_target(FuzzBufferType)’:
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vSeedNodes’ [-Werror=missing-field-initializers]
   43 |     connman.Init({ .nMaxOutboundLimit = max_outbound_limit });
      |     ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vWhitelistedRange’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vWhiteBinds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vBinds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::onion_binds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::bind_on_any’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vSeedNodes’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vWhitelistedRange’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vWhiteBinds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::vBinds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::onion_binds’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::bind_on_any’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::m_specified_outgoing’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::m_added_nodes’ [-Werror=missing-field-initializers]
test/fuzz/connman.cpp:43:17: error: missing initializer for member ‘CConnman::Options::m_i2p_accept_incoming’ [-Werror=missing-field-initializers]

@brunoerg
Copy link
Contributor Author

brunoerg commented Jan 9, 2024

New compiler warning, using gcc 13.2.1

I'm checking it atm.

achow101 added a commit that referenced this pull request Jan 10, 2024
e84dc36 fuzz: fix `connman` initialization (brunoerg)

Pull request description:

  Fixes #29172 (comment)

ACKs for top commit:
  achow101:
    ACK e84dc36

Tree-SHA512: e5f3c378cfe367cc4c387fa1b13663a74d8b667a5d130d62919e21455861cfb9383b63ef4ebe56daab7b2c09e3b5031acc463065455f71607c5fb9e3c370d3ad
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants