Simplify string mangling in ifmaybeload().
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Oct 1 2024, 6:04 AM

	Unknown Object (File)
	Sep 23 2024, 1:14 PM

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

	Unknown Object (File)
	Sep 23 2024, 2:06 AM

	Unknown Object (File)
	Sep 22 2024, 7:25 PM

	Unknown Object (File)
	Sep 18 2024, 5:45 PM

	Unknown Object (File)
	Sep 18 2024, 11:26 AM

	Unknown Object (File)
	Sep 16 2024, 3:37 PM

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