✗ CI.checkpatch: warning for drm/xe: Fix uninitialized variable in xe_vm_bind_ioctl() (rev2)
Patchwork
patchwork at emeril.freedesktop.org
Wed Mar 12 02:43:22 UTC 2025
== Series Details ==
Series: drm/xe: Fix uninitialized variable in xe_vm_bind_ioctl() (rev2)
URL : https://patchwork.freedesktop.org/series/146076/
State : warning
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
cbb4e4a079d89106c2736adc3c7de6f9dc56da07
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 48db77aa71cd07376abec0360ce7a9564cd37947
Author: Matthew Brost <matthew.brost at intel.com>
Date: Tue Mar 11 19:38:16 2025 -0700
drm/xe: Fix uninitialized variable in xe_vm_bind_ioctl()
The error handling assumes that vm_bind_ioctl_check_args() will
initialize "bind_ops" but there are a couple early returns where that's
not true. Initialize "bind_ops" to NULL from the start.
Also if vm_bind_ioctl_check_args, fails set bind_ops to NULL after
freeing.
Fixes: b43e864af0d4 ("drm/xe/uapi: Add DRM_XE_VM_BIND_FLAG_CPU_ADDR_MIRROR")
Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
Signed-off-by: Dan Carpenter <dan.carpenter at linaro.org>
Signed-off-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch eb17816e52395a403aa0b447aa0befa9d2f86dd5 drm-intel
48db77aa71cd drm/xe: Fix uninitialized variable in xe_vm_bind_ioctl()
-:14: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report
#14:
Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
Signed-off-by: Dan Carpenter <dan.carpenter at linaro.org>
total: 0 errors, 1 warnings, 0 checks, 19 lines checked
More information about the Intel-xe
mailing list