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

	F167851085: D49040.id151326.diff
	Tue, Aug 25, 12:19 AM

	F167848993: D49040.diff
	Mon, Aug 24, 11:57 PM

	F167826287: D49040.id151271.diff
	Mon, Aug 24, 7:44 PM

	F167805363: D49040.diff
	Mon, Aug 24, 4:42 PM

	Unknown Object (File)
	Mon, Aug 24, 12:32 PM

	Unknown Object (File)
	Sat, Aug 22, 3:36 PM

	Unknown Object (File)
	Sat, Aug 22, 3:08 PM

	Unknown Object (File)
	Sat, Aug 22, 12:30 PM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGad77d3177f3e: tests/netinet: add few tests for sending a broadcast packet

Summary

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.
Set SO_BROADCAST, set IP_ONESBCAST and send to local broadcast address.
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

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

Path

Size

tests/

sys/

netinet/

3 lines

192 lines

Diff 151326

tests/sys/netinet/Makefile

Loading...

tests/sys/netinet/broadcast.c

Loading...