Skip to content

Correctness and memory-safety hardening across lock, route, and config paths#82

Merged
douzzer merged 6 commits into
wolfSSL:masterfrom
JeremiahM37:fenrir-fixes-2
May 12, 2026
Merged

Correctness and memory-safety hardening across lock, route, and config paths#82
douzzer merged 6 commits into
wolfSSL:masterfrom
JeremiahM37:fenrir-fixes-2

Conversation

@JeremiahM37
Copy link
Copy Markdown
Contributor

Fixes F-2057, F-2058, F-2059, F-2063, F-2066, F-2067, F-2068, F-2440, F-2447, F-2455, F-2830, F-2850, F-2851, F-3184, F-3185, F-3192, F-3193

  • Release locks on error-return paths in event/action/lwip install functions
  • Plug leaks in JSON init, dict dump, and kv_clone; cap JSON parser size; fix KV string JSON escaping
  • Register LINK64 in ntop/render; correct last_af_wildcard_route insert and delete tracking
  • Snapshot route config before dropping parent_event to avoid route_free_1 use-after-free
  • Treat max_connection_count=0 as unlimited; reject non-decisional default_policy values
  • Add regression tests for penaltybox/greenlist order, default_policy guard, LINK64 roundtrip, unlimited connections, and JSON string escapes

@JeremiahM37 JeremiahM37 self-assigned this Apr 21, 2026
@JeremiahM37 JeremiahM37 force-pushed the fenrir-fixes-2 branch 3 times, most recently from a4857f8 to 6047c6e Compare April 21, 2026 17:48
@JeremiahM37 JeremiahM37 marked this pull request as ready for review April 21, 2026 17:48
Copy link
Copy Markdown

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #82

Scan targets checked: wolfsentry-bugs, wolfsentry-src

No new issues found in the changed files. ✅

@dgarske dgarske requested a review from douzzer May 5, 2026 17:38
Copy link
Copy Markdown
Collaborator

@douzzer douzzer left a comment

Choose a reason for hiding this comment

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

passed make check-all

@douzzer douzzer merged commit 6c020a3 into wolfSSL:master May 12, 2026
2 checks passed
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.

4 participants