Re: [PATCH 3/3] dma-buf: remove dma_buf_debugfs_create_file()

2016-06-20 Thread Mathias Krause
On 20 June 2016 at 15:31, Daniel Vetter <dan...@ffwll.ch> wrote: > On Sun, Jun 19, 2016 at 02:31:31PM +0200, Mathias Krause wrote: >> [...] >> With no users left, we can remove dma_buf_debugfs_create_file(). >> >> While at it, simplify the error handling in d

[PATCH 1/3] dma-buf: propagate errors from dma_buf_describe() on debugfs read

2016-06-19 Thread Mathias Krause
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 <mini...@googlemail.com> Cc: Sumit Semwal <sumit.sem...@linaro.org>

[PATCH 3/3] dma-buf: remove dma_buf_debugfs_create_file()

2016-06-19 Thread Mathias Krause
at it, simplify the error handling in dma_buf_init_debugfs() slightly. Signed-off-by: Mathias Krause <mini...@googlemail.com> Cc: Sumit Semwal <sumit.sem...@linaro.org> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> --- drivers/dma-buf/dma-buf.c | 29 + inclu

[PATCH 2/3] dma-buf: remove dma_buf directory on bufinfo file creation errors

2016-06-19 Thread Mathias Krause
Change the error handling in dma_buf_init_debugfs() to remove the "dma_buf" directory if creating the "bufinfo" file fails. No need to have an empty debugfs directory around. Signed-off-by: Mathias Krause <mini...@googlemail.com> Cc: Sumit Semwal <sumit.sem...@

[PATCH 0/3] dma-buf: debugfs fixes

2016-06-19 Thread Mathias Krause
the then unneeded function dma_buf_debugfs_create_file() (patch 3). Please apply! Mathias Krause (3): dma-buf: propagate errors from dma_buf_describe() on debugfs read dma-buf: remove dma_buf directory on bufinfo file creation errors dma-buf: remove dma_buf_debugfs_create_file() drivers/dma-buf/dma

Re: [PATCH] dma-buf: propagate errors from dma_buf_describe() on debugfs read

2016-06-19 Thread Mathias Krause
On 19 June 2016 at 10:45, Daniel Vetter <dan...@ffwll.ch> wrote: > On Fri, Jun 17, 2016 at 08:57:03PM +0200, Mathias Krause wrote: >> diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c >> index 6355ab38d630..0f2a4592fdd2 100644 >> --- a/drivers/dma-buf

[PATCH] dma-buf: propagate errors from dma_buf_describe() on debugfs read

2016-06-17 Thread Mathias Krause
pointer to the correct type and propagate its return value. This type mismatch was caught by the PaX RAP plugin. Signed-off-by: Mathias Krause <mini...@googlemail.com> Cc: Sumit Semwal <sumit.sem...@linaro.org> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: PaX Team <pagee...@free

[PATCH] [media] OMAP1: fix use after free

2011-01-30 Thread Mathias Krause
/kfree.cocci. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Mathias Krause mini...@googlemail.com --- drivers/media/video/omap1_camera.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/omap1_camera.c b