CC: [email protected] In-Reply-To: <[email protected]> References: <[email protected]> TO: Ian Pilcher <[email protected]> TO: [email protected] TO: [email protected] CC: [email protected] CC: [email protected] CC: [email protected] CC: [email protected] CC: [email protected]
Hi Ian, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on a3fa7a101dcff93791d1b1bdb3affcad1410c8c1] url: https://github.com/0day-ci/linux/commits/Ian-Pilcher/Introduce-block-device-LED-trigger/20210910-062756 base: a3fa7a101dcff93791d1b1bdb3affcad1410c8c1 :::::: branch date: 2 days ago :::::: commit date: 2 days ago config: i386-randconfig-m021-20210912 (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> smatch warnings: drivers/leds/trigger/ledtrig-blkdev.c:410 blkdev_disk_unlink_locked() error: dereferencing freed memory 'disk' vim +/disk +410 drivers/leds/trigger/ledtrig-blkdev.c 4a00ef60e59b58 Ian Pilcher 2021-09-09 380 4a00ef60e59b58 Ian Pilcher 2021-09-09 381 66cb682de7e8bd Ian Pilcher 2021-09-09 382 /* 66cb682de7e8bd Ian Pilcher 2021-09-09 383 * 66cb682de7e8bd Ian Pilcher 2021-09-09 384 * unlink_device sysfs attribute - disassociate a device from this LED 66cb682de7e8bd Ian Pilcher 2021-09-09 385 * 66cb682de7e8bd Ian Pilcher 2021-09-09 386 */ 66cb682de7e8bd Ian Pilcher 2021-09-09 387 66cb682de7e8bd Ian Pilcher 2021-09-09 388 static void blkdev_disk_unlink_locked(struct ledtrig_blkdev_led *const led, 66cb682de7e8bd Ian Pilcher 2021-09-09 389 struct ledtrig_blkdev_link *const link, 66cb682de7e8bd Ian Pilcher 2021-09-09 390 struct ledtrig_blkdev_disk *const disk) 66cb682de7e8bd Ian Pilcher 2021-09-09 391 { 66cb682de7e8bd Ian Pilcher 2021-09-09 392 --ledtrig_blkdev_count; 66cb682de7e8bd Ian Pilcher 2021-09-09 393 66cb682de7e8bd Ian Pilcher 2021-09-09 394 if (ledtrig_blkdev_count == 0) 66cb682de7e8bd Ian Pilcher 2021-09-09 395 WARN_ON(!cancel_delayed_work_sync(&ledtrig_blkdev_work)); 66cb682de7e8bd Ian Pilcher 2021-09-09 396 66cb682de7e8bd Ian Pilcher 2021-09-09 397 sysfs_remove_link(led->dir, disk->gd->disk_name); 66cb682de7e8bd Ian Pilcher 2021-09-09 398 sysfs_remove_link(disk->dir, led->led_dev->name); 66cb682de7e8bd Ian Pilcher 2021-09-09 399 kobject_put(disk->dir); 66cb682de7e8bd Ian Pilcher 2021-09-09 400 66cb682de7e8bd Ian Pilcher 2021-09-09 401 hlist_del(&link->led_disks_node); 66cb682de7e8bd Ian Pilcher 2021-09-09 402 hlist_del(&link->disk_leds_node); 66cb682de7e8bd Ian Pilcher 2021-09-09 403 kfree(link); 66cb682de7e8bd Ian Pilcher 2021-09-09 404 66cb682de7e8bd Ian Pilcher 2021-09-09 405 if (hlist_empty(&disk->leds)) { 66cb682de7e8bd Ian Pilcher 2021-09-09 406 disk->gd->ledtrig = NULL; 66cb682de7e8bd Ian Pilcher 2021-09-09 407 kfree(disk); 66cb682de7e8bd Ian Pilcher 2021-09-09 408 } 66cb682de7e8bd Ian Pilcher 2021-09-09 409 66cb682de7e8bd Ian Pilcher 2021-09-09 @410 put_disk(disk->gd); 66cb682de7e8bd Ian Pilcher 2021-09-09 411 } 66cb682de7e8bd Ian Pilcher 2021-09-09 412 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/[email protected]
.config.gz
Description: application/gzip
_______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
