✗ 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