[dm-devel] [PATCH V2 05/14] multipathd: use MALLOC and check return value in cli_getprkey func

lixiaokeng lixiaokeng at huawei.com
Tue Sep 8 06:57:59 UTC 2020


In cli_getprkey func, we use MALLOC instead of malloc, and check
the return value of MALLOC.

V1->V2: turn back MALLOC to malloc and reomve error message

Signed-off-by: Zhiqiang Liu <liuzhiqiang26 at huawei.com>
Signed-off-by: Lixiaokeng <lixiaokeng at huawei.com>
Signed-off-by: Linfeilong <linfeilong at huawei.com>
---
 multipathd/cli_handlers.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/multipathd/cli_handlers.c b/multipathd/cli_handlers.c
index e9698704..235e2a2e 100644
--- a/multipathd/cli_handlers.c
+++ b/multipathd/cli_handlers.c
@@ -1531,6 +1531,8 @@ cli_getprkey(void * v, char ** reply, int * len, void * data)
                return 1;

        *reply = malloc(26);
+       if (!*reply)
+               return 1;

        if (!get_be64(mpp->reservation_key)) {
                sprintf(*reply, "none\n");
--





More information about the dm-devel mailing list