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

gnrc_ipv6_ext_frag: _completed: Add comment why list head is not checked for NULL pointer dereference #19974

Merged

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Oct 12, 2023

Contribution description

Should avoid some confusion in security reviews.

Testing procedure

Issues/PRs references

@miri64 miri64 requested a review from benpicco October 12, 2023 14:25
@github-actions github-actions bot added Area: network Area: Networking Area: sys Area: System labels Oct 12, 2023
@benpicco benpicco added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 15, 2023
@riot-ci
Copy link

riot-ci commented Oct 15, 2023

Murdock results

✔️ PASSED

988db69 gnrc_ipv6_ext_frag: _completed: Add comment why list head is not checked

Success Failures Total Runtime
1 0 1 01m:02s

Artifacts

@benpicco
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 16, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 910e0e2 into RIOT-OS:master Oct 16, 2023
26 checks passed
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.01 milestone Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants