| Message ID | 20231109154742.271947-1-zangyangyang1@xiaomi.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [f2fs-dev,V2] libf2fs: Fix using uninitialized variables error in get_device_info() | expand |
On 2023/11/9 23:47, zangyangyang1 wrote: > This issue comes from a static code scanning tool. > When c.sparse_mode is 1, stat_buf will not be initialized, > but it will be used next. > If this issue does not require modification, please ignore this commit. > > Signed-off-by: zangyangyang1 <zangyangyang1@xiaomi.com> Reviewed-by: Chao Yu <chao@kernel.org> Thanks,
diff --git a/lib/libf2fs.c b/lib/libf2fs.c index 995e42d..47d4e49 100644 --- a/lib/libf2fs.c +++ b/lib/libf2fs.c @@ -931,7 +931,7 @@ int get_device_info(int i) } } - stat_buf = malloc(sizeof(struct stat)); + stat_buf = calloc(1, sizeof(struct stat)); ASSERT(stat_buf); if (!c.sparse_mode) {
This issue comes from a static code scanning tool. When c.sparse_mode is 1, stat_buf will not be initialized, but it will be used next. If this issue does not require modification, please ignore this commit. Signed-off-by: zangyangyang1 <zangyangyang1@xiaomi.com> --- Changes since v1: - Use calloc() instead of malloc() --- lib/libf2fs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.40.1 #/******±¾Óʼþ¼°Æ丽¼þº¬ÓÐСÃ×¹«Ë¾µÄ±£ÃÜÐÅÏ¢£¬½öÏÞÓÚ·¢Ë͸øÉÏÃæµØÖ·ÖÐÁгöµÄ¸öÈË»òȺ×é¡£½ûÖ¹ÈκÎÆäËûÈËÒÔÈκÎÐÎʽʹÓ㨰üÀ¨µ«²»ÏÞÓÚÈ«²¿»ò²¿·ÖµØй¶¡¢¸´ÖÆ¡¢»òÉ¢·¢£©±¾ÓʼþÖеÄÐÅÏ¢¡£Èç¹ûÄú´íÊÕÁ˱¾Óʼþ£¬ÇëÄúÁ¢¼´µç»°»òÓʼþ֪ͨ·¢¼þÈ˲¢É¾³ý±¾Óʼþ£¡ This e-mail and its attachments contain confidential information from XIAOMI, which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender by phone or email immediately and delete it!******/#