Paths

Table of Contentst

Differential D11118

usr.sbin/ndp: small fixes
ClosedPublic
Actions

Authored by garga on Jun 9 2017, 6:00 PM.

Tags

None

Referenced Files

	F82873896: D11118.id46614.diff
	Fri, May 3, 10:48 AM

	Unknown Object (File)
	Wed, May 1, 9:08 AM

	Unknown Object (File)
	Tue, Apr 30, 11:19 PM

	Unknown Object (File)
	Sun, Apr 28, 12:58 AM

	Unknown Object (File)
	Sat, Apr 27, 3:16 AM

	Unknown Object (File)
	Sat, Apr 27, 2:41 AM

	Unknown Object (File)
	Sat, Apr 27, 2:40 AM

	Unknown Object (File)
	Sat, Apr 27, 2:40 AM

View All 52 Files

Subscribers

Details

Reviewers

loos
allanjude
kevans

Commits

rS338243: usr.sbin/ndp: Cleanup in preparation to add libxo support

Summary

usr.sbin/ndp:

Constify rtpref_str declaration
Remove unused h_errno declaration
Use time_t type for expire
Use strlcpy to set static "?" value to ifname
Rename local variable 's' to stop shadowing global definition
Close socket used in pfx_flush()
Use local variables for sock() in setdefif() and getdefif()
Increase WARNS to 3

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

garga created this revision.Jun 9 2017, 6:00 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 9 2017, 6:00 PM

Harbormaster completed remote builds in B9758: Diff 29388.Jun 9 2017, 6:00 PM

garga retitled this revision from usr.sbin/ntp: small fixes to usr.sbin/ndp: small fixes.Jun 9 2017, 6:19 PM

allanjude added a reviewer: allanjude.Jul 7 2018, 3:16 AM

garga added a reviewer: kevans.Aug 9 2018, 11:35 AM

kevans added inline comments.Aug 9 2018, 8:19 PM

usr.sbin/ndp/ndp.c
670 ↗	(On Diff #29388)	You might as well change this to `ifname == NULL` while you're touching the area, since ifname is actually a pointer.
671 ↗	(On Diff #29388)	ifname is actually NULL here, so the strlcpy dest and size should be of `ifix_buf` (and set `ifname = ifix_buf`)

Address @kevans comments

Harbormaster completed remote builds in B18758: Diff 46613.Aug 13 2018, 11:34 AM

Fixed reversed check

Harbormaster completed remote builds in B18759: Diff 46614.Aug 13 2018, 11:35 AM

Approved By: allanjude

This revision is now accepted and ready to land.Aug 21 2018, 4:59 PM

Closed by commit rS338243: usr.sbin/ndp: Cleanup in preparation to add libxo support (authored by garga). · Explain WhyAug 23 2018, 10:39 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

ndp/

2 lines

77 lines

Diff 47158

head/usr.sbin/ndp/Makefile

Loading...

head/usr.sbin/ndp/ndp.c

Loading...