✓ CI.checkpatch: success for drm/xe/display: Fix memory leak in parse_lfp_panel_dtd()

Patchwork patchwork at emeril.freedesktop.org
Thu Oct 10 00:06:39 UTC 2024


== Series Details ==

Series: drm/xe/display: Fix memory leak in parse_lfp_panel_dtd()
URL   : https://patchwork.freedesktop.org/series/139818/
State : success

== 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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 717e7d32495b57fda31391c8611a7a6043069c12
Author: Shuicheng Lin <shuicheng.lin at intel.com>
Date:   Wed Oct 9 23:27:09 2024 +0000

    drm/xe/display: Fix memory leak in parse_lfp_panel_dtd()
    
    The function parse_lfp_panel_dtd() is called when the driver
    attempts to initialize the eDP connector, and it allocates memory,
    which is recorded in panel->vbt.lfp_vbt_mode. However, since no
    eDP panel is connected, the driver fails at intel_edp_init_dpcd()
    and follows the failure path. Unfortunately, the allocated memory
    is not freed in this case.
    
    To fix this issue, free the memory in the failure path.
    
    leak info from kmemleak:
    "
    unreferenced object 0xffff8881252f8800 (size 128):
      comm "systemd-udevd", pid 192, jiffies 4294896880
      hex dump (first 32 bytes):
        e8 fd 00 00 00 04 18 04 a0 04 40 05 00 00 00 03  .......... at .....
        03 03 09 03 26 03 00 00 0a 00 00 00 00 00 00 00  ....&...........
      backtrace (crc 7448f6b4):
        [<ffffffff82475c9b>] kmemleak_alloc+0x4b/0x80
        [<ffffffff814bb50e>] __kmalloc_cache_noprof+0x2be/0x390
        [<ffffffffa069862c>] intel_bios_init_panel+0x1c4c/0x2720 [xe]
        [<ffffffffa0699123>] intel_bios_init_panel_early+0x13/0x20 [xe]
        [<ffffffffa06fceb9>] intel_dp_init_connector+0x2f9/0x1080 [xe]
        [<ffffffffa06c370a>] intel_ddi_init+0xbba/0xf50 [xe]
        [<ffffffffa069b906>] intel_bios_for_each_encoder+0x36/0x60 [xe]
        [<ffffffffa06d7bd6>] intel_setup_outputs+0x206/0x450 [xe]
        [<ffffffffa06dad33>] intel_display_driver_probe_nogem+0x163/0x1f0 [xe]
        [<ffffffffa0680fc7>] xe_display_init_noaccel+0x27/0x70 [xe]
        [<ffffffffa05b30d6>] xe_device_probe+0x806/0x9a0 [xe]
        [<ffffffffa0612f0f>] xe_pci_probe+0x31f/0x590 [xe]
        [<ffffffff81b41718>] local_pci_probe+0x48/0xb0
        [<ffffffff81b432c8>] pci_device_probe+0xc8/0x280
        [<ffffffff81d5dde8>] really_probe+0xf8/0x390
        [<ffffffff81d5e11a>] __driver_probe_device+0x8a/0x170
    "
    
    Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>
+ /mt/dim checkpatch e1aba2bf4f79f2f8ae7ce538124d445fc91df852 drm-intel
717e7d32495b drm/xe/display: Fix memory leak in parse_lfp_panel_dtd()




More information about the Intel-xe mailing list