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@xxxxxxxxxx> Signed-off-by: Lixiaokeng <lixiaokeng@xxxxxxxxxx> Signed-off-by: Linfeilong <linfeilong@xxxxxxxxxx> --- 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"); -- -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel