Re: [PATCH] zonefs: Fix the wrong format specifier

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Zhu Jun <zhujun2@xxxxxxxxxxxxxxxxxxxx>, "dlemoal@xxxxxxxxxx" <dlemoal@xxxxxxxxxx>
Subject: Re: [PATCH] zonefs: Fix the wrong format specifier
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Wed, 13 Nov 2024 17:55:56 +0000
Accept-language: en-US
Cc: "naohiro.aota@xxxxxxx" <naohiro.aota@xxxxxxx>, "jth@xxxxxxxxxx" <jth@xxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <20241111054126.2929-1-zhujun2@cmss.chinamobile.com>
Thread-index: AQHbM/xmpvV+OGVUUUStVlB7TttGT7K1gpmA
Thread-topic: [PATCH] zonefs: Fix the wrong format specifier
User-agent: Mozilla Thunderbird

On 11/10/24 21:41, Zhu Jun wrote:
> The format specifier of "signed int" in snprintf() should be "%d", not
> "%u".
>
> Signed-off-by: Zhu Jun<zhujun2@xxxxxxxxxxxxxxxxxxxx>

Indeed :-
794         int f;

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>

-ck

References:
- [PATCH] zonefs: Fix the wrong format specifier
  - From: Zhu Jun

Prev by Date: [PATCH v2 1/3] pseudofs: add support for export_ops
Next by Date: [PATCH v2 0/3] pidfs: implement file handle support
Previous by thread: [PATCH] zonefs: Fix the wrong format specifier
Next by thread: Re: [PATCH] zonefs: Fix the wrong format specifier
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]

Powered by Linux