]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
drm/i915: Sanity check DP AUX message buffer and size
authorImre Deak <imre.deak@intel.com>
Fri, 29 Jan 2016 12:52:26 +0000 (14:52 +0200)
committerImre Deak <imre.deak@intel.com>
Tue, 2 Feb 2016 16:28:20 +0000 (18:28 +0200)
commitd81a67cc1bfdc3efed8590ee4e07a40b198908ba
tree77fb04360979d9ab1cc132bc369d4c494a64c55c
parent985dd4360fdf2533fe48a33a4a2094f2e4718dc0
drm/i915: Sanity check DP AUX message buffer and size

While we are calling intel_dp_aux_transfer() with msg->size=0 whenever
msg->buffer is NULL, passing NULL to memcpy() is undefined according to
the ISO C standard. I haven't found any notes about this in the GNU C's
or the kernel's documentation of the function and can't imagine what it
would do with the NULL ptr. To better document this use of the
parameters it still make sense to add an explicit check for this to the
code.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: David Weinehall <david.weinehall@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1454071949-24677-1-git-send-email-imre.deak@intel.com
drivers/gpu/drm/i915/intel_dp.c