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
Unknown Object (File)
Sun, Aug 16, 4:40 PM
Unknown Object (File)
Fri, Aug 14, 10:10 AM
Unknown Object (File)
Sat, Aug 8, 4:54 PM
Unknown Object (File)
Sat, Aug 8, 11:33 AM
Unknown Object (File)
Fri, Aug 7, 7:57 AM
Unknown Object (File)
Wed, Aug 5, 12:47 PM
Unknown Object (File)
Tue, Aug 4, 6:12 AM
Unknown Object (File)
Sun, Jul 26, 5:30 AM
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...