dma-buf/dma-resv: check if the new fence is really later
authorChristian König <ckoenig.leichtzumerken@gmail.com>
Wed, 10 Aug 2022 17:26:17 +0000 (19:26 +0200)
committerChristian König <christian.koenig@amd.com>
Thu, 25 Aug 2022 11:10:30 +0000 (13:10 +0200)
commita3f7c10a269d5b77dd5822ade822643ced3057f0
treeef21e969f869b7cb50e128264e2af4fc59db92f3
parent6b04ce966a738ecdd9294c9593e48513c0dc90aa
dma-buf/dma-resv: check if the new fence is really later

Previously when we added a fence to a dma_resv object we always
assumed the the newer than all the existing fences.

With Jason's work to add an UAPI to explicit export/import that's not
necessary the case any more. So without this check we would allow
userspace to force the kernel into an use after free error.

Since the change is very small and defensive it's probably a good
idea to backport this to stable kernels as well just in case others
are using the dma_resv object in the same way.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Jason Ekstrand <jason.ekstrand@collabora.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220810172617.140047-1-christian.koenig@amd.com
Cc: stable@vger.kernel.org # v5.19+
drivers/dma-buf/dma-resv.c