Page MenuHomeFreeBSD
Differential D49040

tests/netinet: add few tests for sending a broadcast packet
ClosedPublic
Actions

Authored by glebius on Feb 17 2025, 11:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 5, 11:08 PM
Unknown Object (File)
Sat, Sep 5, 3:07 PM
Unknown Object (File)
Sat, Sep 5, 12:44 PM
Unknown Object (File)
Sat, Sep 5, 10:28 AM
Unknown Object (File)
Fri, Sep 4, 8:08 AM
Unknown Object (File)
Thu, Sep 3, 7:20 PM
Unknown Object (File)
Thu, Sep 3, 5:25 PM
Unknown Object (File)
Thu, Sep 3, 3:43 PM
View All Files
Subscribers
asomers
imp
melifaro

Details

Reviewers
markj
Group Reviewers
network
Commits
rGad77d3177f3e: tests/netinet: add few tests for sending a broadcast packet
Summary
  1. Set SO_BROADCAST and send to INADDR_BROADCAST. This one is broken since at least FreeBSD 6.0, see bug 99558. Today it sends packet to a non-broadcast link level address.
  2. Set SO_BROADCAST, set IP_ONESBCAST and send to local broadcast address.
  3. Set SO_BROADCAST and send to local broadcast address.

For each test we bind a second socket to check whether the broadcast
is actually received.

PR: 99558

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

glebius created this revision.Feb 17 2025, 11:37 PM
Herald added subscribers: melifaro, asomers, imp. · View Herald TranscriptFeb 17 2025, 11:37 PM
glebius requested review of this revision.Feb 17 2025, 11:37 PM
glebius added a child revision: D49041: netinet: rename in_broadcast() to in_ifnet_broadcast().
Harbormaster completed remote builds in B62492: Diff 151132.Feb 17 2025, 11:38 PM
markj accepted this revision as: markj.Feb 18 2025, 9:27 PM
This revision is now accepted and ready to land.Feb 18 2025, 9:27 PM
glebius updated this revision to Diff 151271.Feb 20 2025, 10:08 PM

Make IP_ONESBCAST test more sophisticated: on machines with more interfaces
do more testing.

This revision now requires review to proceed.Feb 20 2025, 10:08 PM
Harbormaster completed remote builds in B62555: Diff 151271.Feb 20 2025, 10:08 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 22 2025, 2:13 AM
Closed by commit rGad77d3177f3e: tests/netinet: add few tests for sending a broadcast packet (authored by glebius). · Explain Why
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rGad77d3177f3e: tests/netinet: add few tests for sending a broadcast packet.

Revision Contents
Changeset List

PathSize
tests/
sys/
netinet/
3 lines
192 lines

Diff 151326

View Options

tests/sys/netinet/Makefile

Loading...
View Options

tests/sys/netinet/broadcast.c

Loading...