Simplify string mangling in ifmaybeload().
ClosedPublic
Actions

Authored by jhb on Feb 9 2015, 10:20 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 8 2024, 2:19 PM

	Unknown Object (File)
	Dec 22 2023, 9:17 PM

	Unknown Object (File)
	Nov 22 2023, 1:00 PM

	Unknown Object (File)
	Nov 14 2023, 6:48 AM

	Unknown Object (File)
	Nov 12 2023, 6:33 AM

	Unknown Object (File)
	Oct 19 2023, 6:17 AM

	Unknown Object (File)
	Jul 1 2023, 12:25 PM

	Unknown Object (File)
	Jul 1 2023, 12:24 PM

View All 39 Files

Subscribers

imp

Details

Reviewers

emaste
rpaulo

Commits

rS279951: Simplify string mangling in ifmaybeload().

Summary

Use strlcpy() instead of a strcpy() and use strlcat() instead of
a strlcpy() with a magic number subtracted from the length.
Also, replace strncmp(..., strlen(foo) + 1) with strcmp(...).

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

jhb updated this revision to Diff 3712.Feb 9 2015, 10:20 PM

jhb retitled this revision from to Simplify string mangling in ifmaybeload()..

jhb updated this object.

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

jhb added a reviewer: emaste.

Compile.

Herald added a subscriber: imp. · View Herald TranscriptMar 12 2015, 10:13 PM

I've runtested this now and it works fine.

rpaulo accepted this revision.Mar 13 2015, 9:36 AM

rpaulo added a reviewer: rpaulo.

This revision is now accepted and ready to land.Mar 13 2015, 9:36 AM

Closed by commit rS279951 (authored by @jhb).

Revision Contents
Changeset List

Path

Size

head/

sbin/

ifconfig/

ifconfig.c

9 lines

Diff 4209

View Options

head/sbin/ifconfig/ifconfig.c

Simplify string mangling in ifmaybeload().ClosedPublicActions