Once an interrupt source is registered it's never deregistered.
However, when an MSI is released the pointer for it becomes NULLed out,
resulting in a NULL pointer dereference when attempting to disable the
now-released MSI source. Add NULL check to avoid this.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 66203 Build 63086: arc lint + arc unit
Event Timeline
| sys/x86/x86/msi.c | ||
|---|---|---|
| 222 | Multi-line comment should have a blank line before it. It also should formatted like this: /* * Interrupt sources are always registered, but never unregistered. ... | |
| sys/x86/x86/msi.c | ||
|---|---|---|
| 227–228 | Have you ever actually seen this occur? I believe this is impossible as it would mean isrc == NULL which shouldn't ever occur. | |