FFL: (cleanup) Change cast when comparing tlen to tp->th_urp
ClosedPublic
Actions

Authored by jtl on Jul 2 2016, 8:28 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 27, 6:21 PM

	Unknown Object (File)
	Apr 21 2026, 1:05 AM

	Unknown Object (File)
	Apr 14 2026, 10:31 PM

	Unknown Object (File)
	Apr 12 2026, 4:23 AM

	Unknown Object (File)
	Apr 5 2026, 8:04 AM

	Unknown Object (File)
	Apr 4 2026, 5:23 AM

	Unknown Object (File)
	Apr 3 2026, 12:36 PM

	Unknown Object (File)
	Apr 3 2026, 10:00 AM

View All 75 Files

Subscribers

imp

Details

Reviewers

gnn

Group Reviewers

transport

Summary

Cast tlen to uint32_t (instead of u_long) when comparing it to the u_short tp->th_urp. This should be a no-op change, but there is no need to over-promote.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4414
Build 4464: arc lint + arc unit

Event Timeline

jtl updated this revision to Diff 18094.Jul 2 2016, 8:28 PM

jtl retitled this revision from to FFL: (cleanup) Change cast when comparing tlen to tp->th_urp.

jtl updated this object.

jtl edited the test plan for this revision. (Show Details)

jtl added a reviewer: transport.

Herald edited edge metadata. · View Herald TranscriptJul 2 2016, 8:28 PM

Herald added a subscriber: imp. · View Herald Transcript

gnn accepted this revision.Jul 4 2016, 12:27 AM

gnn added a reviewer: gnn.

This revision is now accepted and ready to land.Jul 4 2016, 12:27 AM

Committed in rS306769.

Revision Contents
Changeset List

Path

Size

sys/

netinet/

tcp_input.c

2 lines

tcp_stacks/

fastpath.c

2 lines

Diff 18094

View Options

sys/netinet/tcp_input.c