Skip to content

Add multicast UDP sockets#103

Merged
gasbytes merged 6 commits intowolfSSL:masterfrom
danielinux:multicast
Apr 22, 2026
Merged

Add multicast UDP sockets#103
gasbytes merged 6 commits intowolfSSL:masterfrom
danielinux:multicast

Conversation

@danielinux
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 22, 2026 10:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds optional IPv4 UDP multicast support to wolfIP (gated by IP_MULTICAST), including IGMPv3 ASM membership reports and BSD-style multicast socket options, plus unit/interop tests and build targets.

Changes:

  • Implement IGMPv3 membership report send/receive and multicast membership tracking for UDP sockets.
  • Add multicast-related setsockopt/getsockopt options (*_ADD/DROP_MEMBERSHIP, *_MULTICAST_{IF,TTL,LOOP}) and multicast MAC/TTL/loop handling.
  • Add unit tests, a Linux TAP interop smoke test, and document/build support for multicast.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
wolfip.h Adds multicast socket option constants and mreq structs under IP_MULTICAST.
src/wolfip.c Core multicast implementation (IGMP, membership tracking, UDP send/recv updates).
src/test/unit/unit_tests_multicast.c New unit tests covering join/drop, receive gating, loopback send, IGMP query refresh.
src/test/unit/unit.c Registers multicast unit tests when IP_MULTICAST is enabled.
src/test/test_multicast_interop.c New Linux TAP interop test for multicast send/receive.
docs/API.md Updates feature list to mention optional multicast UDP.
README.md Documents multicast feature, options, and new test targets.
Makefile Adds unit-multicast, interop build target, and multicast coverage targets.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/wolfip.c Outdated
Comment thread src/wolfip.c
Comment thread src/wolfip.c Outdated
Comment thread src/wolfip.c
Comment thread src/wolfip.c
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 #103

Scan targets checked: wolfip-bugs, wolfip-src

Findings: 2
2 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread src/wolfip.c Outdated
Comment thread src/wolfip.c Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/wolfip.c Outdated
Comment thread src/wolfip.c Outdated
@gasbytes gasbytes merged commit eb6e86f into wolfSSL:master Apr 22, 2026
30 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