Page MenuHomeFreeBSD
Differential D10940

Static analysis: Remove unused variables in IPv6 code
ClosedPublic
Actions

Authored by jtl on May 26 2017, 4:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 5, 11:36 PM
Unknown Object (File)
Sat, Dec 21, 10:34 AM
Unknown Object (File)
Fri, Dec 20, 11:47 AM
Unknown Object (File)
Fri, Dec 20, 10:37 AM
Unknown Object (File)
Nov 11 2024, 6:52 AM
Unknown Object (File)
Oct 23 2024, 3:33 AM
Unknown Object (File)
Sep 23 2024, 11:15 AM
Unknown Object (File)
Sep 16 2024, 11:16 PM
View All 48 Files
Subscribers
ae
bz
imp
vangyzen

Details

Reviewers
bz
Group Reviewers
network
Commits
rS331484: Remove some unneccessary variable sets in IPv6 code, as detected by
Summary

I ran clang's static analyzer over the kernel sources. It identified a number of places where variables are set, but not used. This change cleans up a number of these places.

Sponsored by: Netflix
MFC after: 2 weeks

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15713
Build 15739: arc lint + arc unit

Event Timeline

jtl created this revision.May 26 2017, 4:43 PM
Herald added subscribers: ae, imp. · View Herald TranscriptMay 26 2017, 4:43 PM
jtl mentioned this in D10936: Fix static analysis errors in IPv6 code.May 26 2017, 5:08 PM
jtl updated this revision to Diff 40593.Mar 22 2018, 10:54 AM

Most of these changes have been committed by now. Updating diff with a few remaining bits.

Herald added a subscriber: vangyzen. · View Herald TranscriptMar 22 2018, 10:54 AM
Harbormaster completed remote builds in B15713: Diff 40593.Mar 22 2018, 10:54 AM
jtl added a comment.Mar 22 2018, 10:56 AM

Starting the (approx.) 48-hour countdown timer until commit.

bz accepted this revision.Mar 22 2018, 2:14 PM
bz added a subscriber: bz.

Seems good to me.

This revision is now accepted and ready to land.Mar 22 2018, 2:14 PM
Closed by commit rS331484: Remove some unneccessary variable sets in IPv6 code, as detected by (authored by jtl). · Explain WhyMar 24 2018, 12:43 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
netinet6/
2 lines
1 line
2 lines
2 lines

Diff 40593

View Options

sys/netinet6/dest6.c

Loading...
View Options

sys/netinet6/icmp6.c

Loading...
View Options

sys/netinet6/ip6_output.c

Loading...
View Options

sys/netinet6/udp6_usrreq.c

Loading...