[libfuzzer] Fuzzing different Transport Types for all-clusters-app #35629
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Review changes with SemanticDiff. |
Alami-Amine
force-pushed
the
AA/allclusters-fuzz
branch
from
3c259d0 to
1db065e
Compare
pullapprove
bot
requested review from
andy31415,
andyg-apple,
anush-apple,
arkq,
axelnxp,
bauerschwan,
bzbarsky-apple,
carol-apple,
cecille,
chapongatien,
chrisdecenzo,
chshu,
chulspro,
cliffamzn,
Damian-Nordic,
dhrishi,
doru91,
fessehaeve,
harimau-qirex,
harsha-rajendran,
hawk248,
hicklin,
jepenven-silabs,
jmartinez-silabs and
jmeg-sfy
Alami-Amine
force-pushed
the
AA/allclusters-fuzz
branch
from
1db065e to
04789a3
Compare
andy31415
reviewed
Sep 17, 2024
|
PR #35629: Size comparison from 85b2fd3 to 04789a3 Full report (82 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
kiel-apple
approved these changes
Sep 18, 2024
|
PR #35629: Size comparison from 85b2fd3 to f3ab98c Full report (82 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
andy31415
reviewed
Sep 18, 2024
2. using different parts of the fuzzed input data for TransportType and for Payload
Alami-Amine
force-pushed
the
AA/allclusters-fuzz
branch
from
f3ab98c to
a3d0fb7
Compare
|
PR #35629: Size comparison from 640f21a to 8f17d3b Full report (82 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Alami-Amine
force-pushed
the
AA/allclusters-fuzz
branch
from
bbb5f2f to
fec7b49
Compare
|
PR #35629: Size comparison from 884d9fa to fec7b49 Full report (82 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
bzbarsky-apple
approved these changes
Sep 19, 2024
pidarped
approved these changes
Sep 19, 2024
austina-csa
pushed a commit
to austina-csa/connectedhomeip
that referenced
this pull request
Sep 20, 2024
…roject-chip#35629) * Fuzzing different Transport Types for all-clusters-app * Adding an enum value for the number of transport types * 1. replacing magic number when fuzzing the number of transport types 2. using different parts of the fuzzed input data for TransportType and for Payload * Restyled by clang-format * avoiding out of bounds access --------- Co-authored-by: Restyled.io <[email protected]>
yyzhong-g
pushed a commit
to yyzhong-g/connectedhomeip
that referenced
this pull request
Dec 12, 2024
…roject-chip#35629) * Fuzzing different Transport Types for all-clusters-app * Adding an enum value for the number of transport types * 1. replacing magic number when fuzzing the number of transport types 2. using different parts of the fuzzed input data for TransportType and for Payload * Restyled by clang-format * avoiding out of bounds access --------- Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fuzzing different Transport Types for all-clusters-app instead of just with
Transport::Type = kUndefined