Make sure not to pass NULL to strtoul(3)
ClosedPublic
Actions

Authored by trasz on Aug 31 2020, 1:48 PM.

Tags

None

Referenced Files

	F172767503: D26246.diff
	Sun, Sep 20, 8:45 PM

	Unknown Object (File)
	Tue, Sep 15, 8:38 AM

	Unknown Object (File)
	Tue, Sep 15, 8:38 AM

	Unknown Object (File)
	Tue, Sep 15, 8:05 AM

	Unknown Object (File)
	Fri, Sep 11, 3:06 AM

	Unknown Object (File)
	Mon, Sep 7, 1:13 PM

	Unknown Object (File)
	Sun, Sep 6, 3:11 AM

	Unknown Object (File)
	Sat, Sep 5, 9:10 PM

View All Files

Subscribers

imp

siderop1_netapp.com

Details

Reviewers

mav

Commits

rS365042: Make sure not to pass NULL to strtoul(3). The values come

Summary

Make sure not to pass NULL to strtoul(3). The values come
from the kernel, but let's try to be on the safe side.

Sponsored by: NetApp, Inc.
Sponsored by: Klara, Inc.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

trasz created this revision.Aug 31 2020, 1:48 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 31 2020, 1:48 PM

Harbormaster completed remote builds in B33250: Diff 76395.Aug 31 2020, 1:48 PM

trasz requested review of this revision.Aug 31 2020, 1:48 PM

trasz added a reviewer: mav.Aug 31 2020, 2:09 PM

trasz edited the summary of this revision. (Show Details)Aug 31 2020, 2:23 PM

If you wish so. I don't see a big deal here.

This revision is now accepted and ready to land.Aug 31 2020, 2:36 PM

allanjude added a subscriber: siderop1_netapp.com.Aug 31 2020, 3:13 PM

Closed by commit rS365042: Make sure not to pass NULL to strtoul(3). The values come (authored by trasz). · Explain WhySep 1 2020, 2:59 PM

This revision was automatically updated to reflect the committed changes.

trasz added a commit: rS365042: Make sure not to pass NULL to strtoul(3). The values come.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

ctld/

kernel.c

14 lines

Diff 76488

View Options

head/usr.sbin/ctld/kernel.c

Make sure not to pass NULL to strtoul(3)ClosedPublicActions