Skip to content

Commit c22e7db

Browse files
committed
in_forward: fix leaks on exception (CID 507786 507954 508064)
Signed-off-by: Eduardo Silva <[email protected]>
1 parent cad2476 commit c22e7db

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

plugins/in_forward/fw_prot.c

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -599,6 +599,7 @@ static int check_ping(struct flb_input_instance *ins,
599599
if (o.type != MSGPACK_OBJECT_STR) {
600600
flb_plg_error(ins, "Invalid shared_key_salt type message");
601601
flb_free(serverside);
602+
flb_free(hostname);
602603
msgpack_unpacked_destroy(&result);
603604
return -1;
604605
}
@@ -609,7 +610,9 @@ static int check_ping(struct flb_input_instance *ins,
609610
if (o.type != MSGPACK_OBJECT_STR) {
610611
flb_plg_error(ins, "Invalid shared_key_digest type message");
611612
flb_free(serverside);
613+
flb_free(hostname);
612614
msgpack_unpacked_destroy(&result);
615+
613616
return -1;
614617
}
615618
shared_key_digest = flb_sds_create_len(o.via.str.ptr, o.via.str.size);
@@ -635,6 +638,7 @@ static int check_ping(struct flb_input_instance *ins,
635638
flb_free(hostname);
636639
flb_free(shared_key_salt);
637640
flb_free(shared_key_digest);
641+
flb_free(username);
638642
msgpack_unpacked_destroy(&result);
639643
return -1;
640644
}
@@ -649,7 +653,10 @@ static int check_ping(struct flb_input_instance *ins,
649653
flb_free(serverside);
650654
flb_free(username);
651655
flb_free(password_digest);
652-
flb_plg_error(ctx->ins, "failed to hash shard_key");
656+
flb_free(shared_key_salt);
657+
flb_free(shared_key_digest);
658+
flb_free(hostname);
659+
flb_plg_error(ctx->ins, "failed to hash shared_key");
653660
return -1;
654661
}
655662

0 commit comments

Comments
 (0)