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, Jan 27, 1:48 AM

	Unknown Object (File)
	Dec 13 2024, 3:24 AM

	Unknown Object (File)
	Oct 6 2024, 7:48 PM

	Unknown Object (File)
	Oct 3 2024, 6:17 PM

	Unknown Object (File)
	Sep 16 2024, 10:10 AM

	Unknown Object (File)
	Sep 16 2024, 10:10 AM

	Unknown Object (File)
	Sep 16 2024, 10:00 AM

	Unknown Object (File)
	Sep 5 2024, 8:52 AM

View All 40 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