From 2c99bf69ec4398f4281d7e670837f8a2395de57f Mon Sep 17 00:00:00 2001 From: Zdenek Styblik Date: Sun, 31 Jul 2016 08:56:38 +0200 Subject: [PATCH] Fix missing `goto out_free;` when ipmi_parse_hex() returns (-1) --- lib/ipmi_main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/ipmi_main.c b/lib/ipmi_main.c index 2ea3c26..811c80b 100644 --- a/lib/ipmi_main.c +++ b/lib/ipmi_main.c @@ -499,10 +499,11 @@ ipmi_main(int argc, char ** argv, break; case 'y': memset(kgkey, 0, sizeof(kgkey)); - rc = ipmi_parse_hex(optarg, kgkey, sizeof(kgkey) - 1); + rc = ipmi_parse_hex(optarg, kgkey, sizeof(kgkey) - 1); if (rc == -1) { lprintf(LOG_ERR, "Number of Kg key characters is not even"); + goto out_free; } else if (rc == -3) { lprintf(LOG_ERR, "Kg key is not hexadecimal number"); goto out_free;