* [PATCH] usb: ffs-test: add NULL check for malloc in descs_to_legacy
@ 2026-08-17 6:48 longlong yan
0 siblings, 0 replies; only message in thread
From: longlong yan @ 2026-08-17 6:48 UTC (permalink / raw)
To: gregkh; +Cc: linux-usb, linux-kernel, longlong yan
In descs_to_legacy(), the return value of malloc() is used without a
NULL check. If malloc() fails, dereferencing the resulting pointer
leads to a NULL pointer dereference. Add a NULL check and return 0 on
failure, consistent with the existing error handling at the start of
the function.
Signed-off-by: longlong yan <yanlonglong@kylinos.cn>
---
tools/usb/ffs-test.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/usb/ffs-test.c b/tools/usb/ffs-test.c
index 22b938fbdfb7..5c9281cbbff1 100644
--- a/tools/usb/ffs-test.c
+++ b/tools/usb/ffs-test.c
@@ -293,6 +293,8 @@ static size_t descs_to_legacy(void **legacy, const void *descriptors_v2)
length = sizeof out->header + (descs_end - descs_start);
out = malloc(length);
+ if (!out)
+ return 0;
out->header.magic = cpu_to_le32(FUNCTIONFS_DESCRIPTORS_MAGIC);
out->header.length = cpu_to_le32(length);
out->header.fs_count = cpu_to_le32(fs_count);
--
2.43.0
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2026-08-17 6:48 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-17 6:48 [PATCH] usb: ffs-test: add NULL check for malloc in descs_to_legacy longlong yan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox