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

Receive path Initial packet Fuzzer #3896

Merged
merged 28 commits into from
Oct 25, 2023
Merged

Receive path Initial packet Fuzzer #3896

merged 28 commits into from
Oct 25, 2023

Conversation

gaurav2699
Copy link
Contributor

Description

Describe the purpose of and changes within this Pull Request.

Testing

Do any existing tests cover this change? Are new tests needed?

Documentation

Is there any documentation impact for this change?

@gaurav2699 gaurav2699 requested a review from a team as a code owner October 5, 2023 09:29
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #3896 (ea27341) into main (4425da2) will decrease coverage by 0.12%.
Report is 27 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3896      +/-   ##
==========================================
- Coverage   86.66%   86.55%   -0.12%     
==========================================
  Files          56       56              
  Lines       16854    16901      +47     
==========================================
+ Hits        14607    14628      +21     
- Misses       2247     2273      +26     

see 21 files with indirect coverage changes

src/tools/CMakeLists.txt Outdated Show resolved Hide resolved
@nibanks
Copy link
Member

nibanks commented Oct 16, 2023

Should try onboarding this to stress.yml too.

src/tools/CMakeLists.txt Outdated Show resolved Hide resolved
@gaurav2699 gaurav2699 changed the title [DRAFT] Receive path packet Fuzzer Receive path packet Fuzzer Oct 23, 2023
@gaurav2699 gaurav2699 marked this pull request as draft October 23, 2023 17:26
@gaurav2699 gaurav2699 self-assigned this Oct 23, 2023
Copy link
Member

@nibanks nibanks left a comment

Choose a reason for hiding this comment

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

Mostly style stuff, some refactoring; one fix for Linux.

src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/recvfuzz/recvfuzz.cpp Outdated Show resolved Hide resolved
src/tools/CMakeLists.txt Outdated Show resolved Hide resolved
@gaurav2699 gaurav2699 marked this pull request as ready for review October 25, 2023 09:25
@gaurav2699 gaurav2699 changed the title Receive path packet Fuzzer Receive path Initial packet Fuzzer Oct 25, 2023
@nibanks nibanks merged commit ee3da4e into main Oct 25, 2023
396 of 397 checks passed
@nibanks nibanks deleted the packet_fuzzer branch October 25, 2023 12:03
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