Public bug reported:

[Impact]
Coverity scanning discovered a potential NULL dereference in this driver. 
Though I'm unaware of anyone hitting it in practice, it is an obvious bug.

[Test Case]
There is no known reproducer for this, so it can only be regression tested.

[Regression Risk]
The fix is localized to a driver and is a clean cherry pick from upstream.
The bug fix looks straightforward - don't dereference the pointer if it is 
NULL. The only regression theory I have would be if this means that, instead of 
crashing, we now execute code that is *worse* than crashing (e.g. resulting in 
corruption, etc).

** Affects: linux (Ubuntu)
     Importance: Medium
     Assignee: dann frazier (dannf)
         Status: In Progress

** Affects: linux (Ubuntu Zesty)
     Importance: Medium
     Assignee: dann frazier (dannf)
         Status: In Progress

** Also affects: linux (Ubuntu Zesty)
   Importance: Undecided
       Status: New

** Changed in: linux (Ubuntu Zesty)
       Status: New => Confirmed

** Changed in: linux (Ubuntu Zesty)
   Importance: Undecided => Medium

** Changed in: linux (Ubuntu Zesty)
     Assignee: (unassigned) => dann frazier (dannf)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1708714

Title:
  scsi: hisi_sas: add null check before indirect pointer dereference

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1708714/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to