Page MenuHomeFreeBSD
Differential D5921

altq: remove NULL check after malloc with M_WAITOK
Needs ReviewPublic
Actions

Authored by howard0su_gmail.com on Apr 12 2016, 12:05 AM.
Tags
None
Referenced Files
F159068208: D5921.id15092.diff
Tue, Jun 9, 6:07 PM
Unknown Object (File)
Sat, May 30, 5:05 AM
Unknown Object (File)
Thu, May 28, 6:29 AM
Unknown Object (File)
Thu, May 28, 5:29 AM
Unknown Object (File)
Wed, May 27, 11:44 PM
Unknown Object (File)
Tue, May 19, 11:11 PM
Unknown Object (File)
Mon, May 18, 8:24 PM
Unknown Object (File)
Sun, May 17, 6:22 PM
View All Files
Subscribers
cem
imp
Contributor Reviews (src)

Details

Reviewers
bz

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3224
Build 3257: arc lint + arc unit

Event Timeline

howard0su_gmail.com updated this revision to Diff 15092.Apr 12 2016, 12:05 AM
howard0su_gmail.com retitled this revision from to altq: remove NULL check after malloc with M_WAITOK.
howard0su_gmail.com updated this object.
howard0su_gmail.com edited the test plan for this revision. (Show Details)
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptApr 12 2016, 12:05 AM
howard0su_gmail.com added a reviewer: bz.Apr 12 2016, 2:24 PM
cem added a subscriber: cem.Apr 12 2016, 6:39 PM
cem added inline comments.
sys/net/altq/altq_hfsc.c
1768–1771

Why didn't you remove this one too?

howard0su_gmail.com added inline comments.Apr 15 2016, 10:40 AM
sys/net/altq/altq_hfsc.c
1768–1771

Let me refine my script. my script missed the case without {}.

Revision Contents
Changeset List

PathSize
sys/
net/
altq/
9 lines

Diff 15092

View Options

sys/net/altq/altq_hfsc.c

Loading...