axgbe: Don't dereference NULL pointers
ClosedPublic
Actions

Authored by imp on May 21 2021, 4:51 AM.

Tags

None

Referenced Files

	F151674016: D30373.id90371.diff
	Thu, Apr 9, 10:42 PM

	Unknown Object (File)
	Thu, Apr 9, 9:11 AM

	Unknown Object (File)
	Thu, Apr 9, 7:22 AM

	Unknown Object (File)
	Thu, Apr 9, 12:10 AM

	Unknown Object (File)
	Sat, Mar 28, 8:10 PM

	Unknown Object (File)
	Sat, Mar 28, 6:34 AM

	Unknown Object (File)
	Fri, Mar 27, 10:59 PM

	Unknown Object (File)
	Sun, Mar 22, 4:17 PM

View All 98 Files

Subscribers

allanjude

Details

Reviewers

manu
rajeshasp
allanjude

Commits

rGce0a9d7c126f: axgbe: Don't dereference NULL pointers

Summary

if (sb == NULL) { ... sb->s_error } is going to be a bad time. Return
ENOMEM when we cannot allocate an sbuf for the sysctl rather than
dereferencing the NULL pointer just returned.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 39332
Build 36221: arc lint + arc unit

Event Timeline

imp requested review of this revision.May 21 2021, 4:51 AM

imp created this revision.

Harbormaster completed remote builds in B39332: Diff 89568.May 21 2021, 4:51 AM

imp added a reviewer: manu.May 21 2021, 4:05 PM

imp added a reviewer: rajeshasp.May 21 2021, 4:09 PM

manu accepted this revision.May 21 2021, 4:09 PM

This revision is now accepted and ready to land.May 21 2021, 4:09 PM

Reviewed By: allanjude

Closed by commit rGce0a9d7c126f: axgbe: Don't dereference NULL pointers (authored by imp). · Explain WhyJun 3 2021, 6:04 PM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rGce0a9d7c126f: axgbe: Don't dereference NULL pointers.

Revision Contents
Changeset List

Path

Size

sys/

dev/

axgbe/

xgbe-sysctl.c

38 lines

Diff 89568

View Options

sys/dev/axgbe/xgbe-sysctl.c

axgbe: Don't dereference NULL pointersClosedPublicActions