commit 6c7ee4f4c406f851242a2100bc4e7a2f26d2e300
author Manojkiran Eda <manojkiran.eda@gmail.com> Tue Mar 18 09:56:39 2025 +0530
committer Manojkiran Eda <manojkiran.eda@gmail.com> Tue Mar 18 14:08:23 2025 +0530
tree 61cd77050d8359317c7a6c48368c1eb33e2c370d
parent bc40dd5a64f5b375ba86a0ab6b2ed3e7863e2f5b

requester: Add null check in pldm_instance_id_alloc

pldm_instance_id_alloc() function does not check for a valid
ctx object before dereferencing it. Hence it could cause
crash when its clients pass a null pointer by accident.

Change-Id: I340aa8171cd397f5af772a9cc6d4f80c8263a089
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

src/requester/instance-id.c

diff --git a/src/requester/instance-id.c b/src/requester/instance-id.c
index d891a48..10dc276 100644
--- a/src/requester/instance-id.c
+++ b/src/requester/instance-id.c
@@ -122,7 +122,7 @@
 {
 	uint8_t l_iid;
 
-	if (!iid) {
+	if (!ctx || !iid) {
 		return -EINVAL;
 	}