Hi,
On 2020/7/19 4:25, Bart Van Assche wrote:
> On 2020-07-17 01:44, Xiongfeng Wang wrote:
>> diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c
>> index 182fd25..e443dee 100644
>> --- a/drivers/scsi/scsi_transport_sas.c
>> +++ b/drivers/scsi/scsi_transport_sas.c
>> @@ -563,7 +563,7 @@ static ssize_t do_sas_phy_enable(struct device *dev,
>> {
>> struct sas_phy *phy = transport_class_to_phy(dev);
>>
>> - return snprintf(buf, 20, "%d", phy->enabled);
>> + return snprintf(buf, 20, "%d\n", phy->enabled);
>> }
>
> For future sysfs show function patches, please use PAGE_SIZE as second
> snprintf() argument or use sprintf() instead of snprintf() because in
> this case it is clear that no output buffer overflow will happen. Using
Thanks for your advice. I will follow it in the future patches.
> any other size argument than PAGE_SIZE makes patches like the above
> harder to verify than necessary. Anyway:
>
> Reviewed-by: Bart van Assche <bvanassche@xxxxxxx>
Thanks,
Xiongfeng
>
