Commit 7296f62f authored by Nicholas Krause's avatar Nicholas Krause Committed by Martin K. Petersen

megaraid: Fix possible NULL pointer deference in mraid_mm_ioctl

This adds the needed check after the call to the function
mraid_mm_alloc_kioc in order to make sure that this function has not
returned NULL and therefore makes sure we do not deference a NULL
pointer if one is returned by mraid_mm_alloc_kioc.  Further more add
needed comments explaining that this function call can return NULL if
the list head is empty for the pointer passed in order to allow furture
users to understand this required pointer check.
Signed-off-by: default avatarNicholas Krause <xerofoify@gmail.com>
Acked-by: default avatarSumit Saxena <sumit.saxena@avagotech.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 74e26784
...@@ -179,8 +179,12 @@ mraid_mm_ioctl(struct file *filep, unsigned int cmd, unsigned long arg) ...@@ -179,8 +179,12 @@ mraid_mm_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
/* /*
* The following call will block till a kioc is available * The following call will block till a kioc is available
* or return NULL if the list head is empty for the pointer
* of type mraid_mmapt passed to mraid_mm_alloc_kioc
*/ */
kioc = mraid_mm_alloc_kioc(adp); kioc = mraid_mm_alloc_kioc(adp);
if (!kioc)
return -ENXIO;
/* /*
* User sent the old mimd_t ioctl packet. Convert it to uioc_t. * User sent the old mimd_t ioctl packet. Convert it to uioc_t.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment