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
F153445272: D5921.id.diff
Tue, Apr 21, 5:28 AM
Unknown Object (File)
Mon, Apr 20, 10:21 PM
Unknown Object (File)
Tue, Apr 7, 5:12 PM
Unknown Object (File)
Sat, Apr 4, 3:12 AM
Unknown Object (File)
Thu, Mar 26, 8:52 AM
Unknown Object (File)
Mar 21 2026, 7:03 AM
Unknown Object (File)
Mar 19 2026, 9:03 AM
Unknown Object (File)
Nov 23 2025, 9:46 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...