diff options
| author |   Mathias Krause <minipli@googlemail.com> | 2016-06-19 14:31:29 +0200 |
|---|---|---|
| committer |   Sumit Semwal <sumit.semwal@linaro.org> | 2016-06-20 22:26:36 +0530 |
| commit | eb0b947e3bf59d263f5bc7c080ce40333b515004 (patch) | |
| tree | 826a3e829563dcae5e9a10420fee3718fb521231 /drivers/gpu/drm/drm_irq.c | |
| parent | drm/i915: Avoid use-after-free of intel_encoder in intel_dp_connector_destrpy (diff) | |
| download | wireguard-linux-eb0b947e3bf59d263f5bc7c080ce40333b515004.tar.xz wireguard-linux-eb0b947e3bf59d263f5bc7c080ce40333b515004.zip | |
dma-buf: propagate errors from dma_buf_describe() on debugfs read
The callback function dma_buf_describe() returns an int not void so the
function pointer cast in dma_buf_show() is wrong. dma_buf_describe() can
also fail when acquiring the mutex gets interrupted so always returning
0 in dma_buf_show() is wrong, too.
Fix both issues by avoiding the indirection via dma_buf_show() and call
dma_buf_describe() directly. Rename it to dma_buf_debug_show() to get it
in line with the other functions.
This type mismatch was caught by the PaX RAP plugin.
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Cc: Sumit Semwal <sumit.semwal@linaro.org>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Brad Spengler <spender@grsecurity.net>
Cc: PaX Team <pageexec@freemail.hu>
Signed-off-by: Sumit Semwal <sumit.semwal@linaro.org>
Diffstat (limited to 'drivers/gpu/drm/drm_irq.c')
0 files changed, 0 insertions, 0 deletions