Page MenuHomeFreeBSD
Differential D39746

Delete unreachable code
ClosedPublic
Actions

Authored by zlei on Apr 21 2023, 10:03 AM.
Tags
None
Referenced Files
F125643081: D39746.id120797.diff
Sun, Aug 10, 9:12 AM
Unknown Object (File)
Thu, Jul 31, 6:06 AM
Unknown Object (File)
Sun, Jul 27, 10:45 PM
Unknown Object (File)
Fri, Jul 25, 4:20 PM
Unknown Object (File)
Tue, Jul 22, 1:11 PM
Unknown Object (File)
Jun 24 2025, 8:15 AM
Unknown Object (File)
Jun 21 2025, 1:06 PM
Unknown Object (File)
Jun 21 2025, 12:55 AM
View All 62 Files
Subscribers
glebius
imp
melifaro

Details

Reviewers
ae
kp
Group Reviewers
network
Commits
rG2a3147f96241: ip_mroute: Delete unreachable code
rG1ff44a05d6bc: if_stf: Delete unreachable code
rGece63e96c364: ip_mroute: Delete unreachable code
rGf70caf1b4524: if_stf: Delete unreachable code
rGb658c0fce10a: ip_mroute: Delete unreachable code
rGc373e1d6adbc: if_stf: Delete unreachable code
Summary

As the flag M_WAITOK is passed to ip_encap_attach(), then the function will never return NULL, and the following code within NULL check branch will be unreachable.

No functional change intended.

Fixes: 6d8fdfa9d5e7d Rework IP encapsulation handling code
MFC after: 1 week

Diff Detail

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

Event Timeline

zlei created this revision.Apr 21 2023, 10:03 AM
Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptApr 21 2023, 10:03 AM
zlei requested review of this revision.Apr 21 2023, 10:03 AM
zlei edited the summary of this revision. (Show Details)
zlei updated this revision to Diff 120806.Apr 21 2023, 10:22 AM
zlei retitled this revision from if_stf: Delete unreachable code to Delete unreachable code.

I'd like to commit them separately.

kp accepted this revision as: kp.Apr 21 2023, 11:11 AM
This revision is now accepted and ready to land.Apr 21 2023, 11:11 AM
zlei edited the summary of this revision. (Show Details)Apr 23 2023, 1:25 AM
Closed by commit rGc373e1d6adbc: if_stf: Delete unreachable code (authored by zlei). · Explain WhyApr 23 2023, 4:49 AM
This revision was automatically updated to reflect the committed changes.
zlei added a commit: rGc373e1d6adbc: if_stf: Delete unreachable code.
zlei added a commit: rGb658c0fce10a: ip_mroute: Delete unreachable code.
zlei added a commit: rGf70caf1b4524: if_stf: Delete unreachable code.May 17 2023, 9:59 AM
zlei added a commit: rGece63e96c364: ip_mroute: Delete unreachable code.
zlei added a commit: rG1ff44a05d6bc: if_stf: Delete unreachable code.May 17 2023, 10:05 AM
zlei added a commit: rG2a3147f96241: ip_mroute: Delete unreachable code.

Revision Contents
Changeset List

PathSize
sys/
net/
6 lines

Diff 120884

View Options

sys/net/if_stf.c

Loading...