Re: (subset) [PATCH 00/42] Fix coccicheck warnings
On Tue, 16 Jan 2024 12:10:47 +0800, Li Zhijian wrote: > make coccicheck COCCI=$PWD/scripts/coccinelle/api/device_attr_show.cocci` > complians some warnnings as following[1]: > > Not sure if someone had tried these fixes, feel free to ignore this > patch set if we have come to a *NOT-FIX* conclusion before :) > > This patch set also fix a few snprintf() beside coccicheck reported. > For example, some thing like > xxx_show() { > rc = snprintf(); > ... > return rc; > } > > [...] Applied to 6.9/scsi-queue, thanks! [22/42] drivers/scsi/fnic: Convert snprintf to sysfs_emit https://git.kernel.org/mkp/scsi/c/1ad717c92925 [25/42] drivers/scsi/ibmvscsi: Convert snprintf to sysfs_emit https://git.kernel.org/mkp/scsi/c/29ff822f466e [26/42] drivers/scsi/ibmvscsi_tgt: Convert snprintf to sysfs_emit https://git.kernel.org/mkp/scsi/c/01105c23de42 [27/42] drivers/scsi/isci: Convert snprintf to sysfs_emit https://git.kernel.org/mkp/scsi/c/5fbf37e53091 [34/42] drivers/scsi/pm8001: Convert snprintf to sysfs_emit https://git.kernel.org/mkp/scsi/c/8179041f801d -- Martin K. Petersen Oracle Linux Engineering
[PATCH 00/42] Fix coccicheck warnings
make coccicheck COCCI=$PWD/scripts/coccinelle/api/device_attr_show.cocci` complians some warnnings as following[1]: Not sure if someone had tried these fixes, feel free to ignore this patch set if we have come to a *NOT-FIX* conclusion before :) This patch set also fix a few snprintf() beside coccicheck reported. For example, some thing like xxx_show() { rc = snprintf(); ... return rc; } TODOs: 1. Fix other abused cases that coccicheck has not detected. For example, ./drivers/scsi/scsi_transport_fc.c has other places using snprint inside the macro. 2. Improve device_attr_show.cocci to detect sprintf() and fix them [1] $ make coccicheck COCCI=$PWD/scripts/coccinelle/api/device_attr_show.cocci ... ./arch/arm/mm/cache-l2x0-pmu.c:346:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/amd/core.c:1282:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/core.c:1895:11-19: WARNING: use scnprintf or sprintf ./arch/x86/events/core.c:2542:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/core.c:2600:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/intel/core.c:5496:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/intel/core.c:5530:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/intel/core.c:5546:8-16: WARNING: use scnprintf or sprintf ./arch/x86/events/intel/pt.c:99:8-16: WARNING: use scnprintf or sprintf ./drivers/bus/fsl-mc/fsl-mc-bus.c:205:8-16: WARNING: use scnprintf or sprintf ./drivers/edac/edac_mc_sysfs.c:210:8-16: WARNING: use scnprintf or sprintf ./drivers/edac/edac_mc_sysfs.c:518:8-16: WARNING: use scnprintf or sprintf ./drivers/fsi/fsi-master-ast-cf.c:1086:8-16: WARNING: use scnprintf or sprintf ./drivers/fsi/fsi-master-gpio.c:721:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:558:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:602:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:792:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:822:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:852:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:882:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:912:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-lenovo.c:941:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-picolcd_core.c:259:9-17: WARNING: use scnprintf or sprintf ./drivers/hid/hid-picolcd_core.c:304:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-isku.c:64:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:403:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:412:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:435:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:444:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:454:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kone.c:556:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-koneplus.c:245:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-koneplus.c:312:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kovaplus.c:275:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kovaplus.c:328:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kovaplus.c:337:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kovaplus.c:347:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-kovaplus.c:368:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-pyra.c:286:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-pyra.c:303:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-roccat-pyra.c:324:8-16: WARNING: use scnprintf or sprintf ./drivers/hid/hid-sensor-custom.c:375:10-18: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3100:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3116:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3132:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3145:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3159:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3173:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3186:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3198:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3211:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3224:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3237:8-16: WARNING: use scnprintf or sprintf ./drivers/message/fusion/mptscsih.c:3250:8-16: WARNING: use scnprintf or sprintf ./drivers/most/core.c:211:8-16: WARNING: use scnprintf or sprintf ./drivers/most/core.c:222:8-16: WARNING: use scnprintf or sprintf