The Static Analysis tool Scan-Build added a null pointer dereference in function m_dup_pkthdr for pointers to and from.
Comment on attachment 8690719 [details] [diff] [review] Bug 1227055.diff Hello Patrick, Could you please take a look other this patch? THX
Attachment #8690719 - Flags: review?(mcmanus)
Comment on attachment 8690719 [details] [diff] [review] Bug 1227055.diff we are actually a downstream consumer of this code - :jesup can shepherd
Attachment #8690719 - Flags: review?(mcmanus) → review?(rjesup)
Michael - for upstreaming
KASSERT() needs two arguments. I have added the suggested protection code with correct syntax in https://github.com/sctplab/usrsctp/commit/12775c1e8104116df9916b2114aa93a4d65519e3 Please note that this is a no-op unless you compile with INVARIANTS, which is not done by Firefox (which is good). If you compile with INVARIANTS, you will see an error message. In any case, if one of the pointers is NULL, you will die... Please use the patch committed upstream.
dealt with upstream
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.