dma-buff name can be set via DMA_BUF_SET_NAME ioctl, but once set it never gets freed.
Free it in dma_buf_release().
Fixes: bb2bb9030425 ("dma-buf: add DMA_BUF_SET_NAME ioctls") Reported-by: syzbot+b2098bc44728a4efb3e9@syzkaller.appspotmail.com Cc: Greg Hackmann ghackmann@google.com Cc: Chenbo Feng fengc@google.com Cc: Sumit Semwal sumit.semwal@linaro.org Signed-off-by: Cong Wang xiyou.wangcong@gmail.com --- drivers/dma-buf/dma-buf.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index ce41cd9b758a..2427398ff22a 100644 --- a/drivers/dma-buf/dma-buf.c +++ b/drivers/dma-buf/dma-buf.c @@ -108,6 +108,7 @@ static int dma_buf_release(struct inode *inode, struct file *file) dma_resv_fini(dmabuf->resv);
module_put(dmabuf->owner); + kfree(dmabuf->name); kfree(dmabuf); return 0; }
Acked-by: Chenbo Feng fengc@google.com
On Thu, Dec 26, 2019 at 10:32 PM Cong Wang xiyou.wangcong@gmail.com wrote:
dma-buff name can be set via DMA_BUF_SET_NAME ioctl, but once set it never gets freed.
Free it in dma_buf_release().
Fixes: bb2bb9030425 ("dma-buf: add DMA_BUF_SET_NAME ioctls") Reported-by: syzbot+b2098bc44728a4efb3e9@syzkaller.appspotmail.com Cc: Greg Hackmann ghackmann@google.com Cc: Chenbo Feng fengc@google.com Cc: Sumit Semwal sumit.semwal@linaro.org Signed-off-by: Cong Wang xiyou.wangcong@gmail.com
drivers/dma-buf/dma-buf.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index ce41cd9b758a..2427398ff22a 100644 --- a/drivers/dma-buf/dma-buf.c +++ b/drivers/dma-buf/dma-buf.c @@ -108,6 +108,7 @@ static int dma_buf_release(struct inode *inode, struct file *file) dma_resv_fini(dmabuf->resv);
module_put(dmabuf->owner);
kfree(dmabuf->name); kfree(dmabuf); return 0;
}
2.21.0
Acked-by: Chenbo Feng fengc@google.com
On Thu, Dec 26, 2019 at 10:32 PM Cong Wang xiyou.wangcong@gmail.com wrote:
dma-buff name can be set via DMA_BUF_SET_NAME ioctl, but once set it never gets freed.
Free it in dma_buf_release().
Fixes: bb2bb9030425 ("dma-buf: add DMA_BUF_SET_NAME ioctls") Reported-by: syzbot+b2098bc44728a4efb3e9@syzkaller.appspotmail.com Cc: Greg Hackmann ghackmann@google.com Cc: Chenbo Feng fengc@google.com Cc: Sumit Semwal sumit.semwal@linaro.org Signed-off-by: Cong Wang xiyou.wangcong@gmail.com
drivers/dma-buf/dma-buf.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index ce41cd9b758a..2427398ff22a 100644 --- a/drivers/dma-buf/dma-buf.c +++ b/drivers/dma-buf/dma-buf.c @@ -108,6 +108,7 @@ static int dma_buf_release(struct inode *inode, struct file *file) dma_resv_fini(dmabuf->resv);
module_put(dmabuf->owner);
kfree(dmabuf->name); kfree(dmabuf); return 0;
}
2.21.0
dri-devel@lists.freedesktop.org