]> git.ipfire.org Git - thirdparty/ldns.git/commitdiff
16-unit-tests-edns: Fix leak & warnings 191/head
authorFelipe Gasper <felipe@felipegasper.com>
Tue, 18 Oct 2022 02:15:29 +0000 (22:15 -0400)
committerFelipe Gasper <felipe@felipegasper.com>
Tue, 18 Oct 2022 02:15:29 +0000 (22:15 -0400)
This addresses a couple issues found via
https://github.com/NLnetLabs/ldns/pull/186.

test/16-unit-tests-edns.tpkg/16-unit-tests-edns.c

index 1607df0b05626b41d9dcacd9fadf42929ab33b2b..aaa90317679ad43604c4ad9d570632bf7268c653 100644 (file)
@@ -34,10 +34,12 @@ check_option_entries(ldns_edns_option *edns, ldns_edns_option_code code,
 
        buf = ldns_edns_get_wireformat_buffer(edns);
        if (ldns_buffer_read_u16(buf) != code) {
+               ldns_buffer_free(buf);
                printf("Error: EDNS type is incorrect\n");
                return 0;
        }
        if (ldns_buffer_read_u16(buf) != size) {
+               ldns_buffer_free(buf);
                printf("Error: EDNS length is incorrect\n");
                return 0;       
        }
@@ -46,15 +48,17 @@ check_option_entries(ldns_edns_option *edns, ldns_edns_option_code code,
                if (ldns_buffer_read_u8_at(buf, i+4) != hex_data[i]) {
                        printf("Error: EDNS data is incorrect: %d, %d\n",
                                ldns_buffer_read_u8_at(buf, i+4), hex_data[i]);
+                       ldns_buffer_free(buf);
                        return 0;
                }
        }
 
+       ldns_buffer_free(buf);
        return 1;
 }
 
 static int
-check_option()
+check_option(void)
 {
        ldns_edns_option *edns;
        ldns_edns_option *clone;
@@ -119,7 +123,7 @@ static int check_option_list_entries(ldns_edns_option_list *list,
 }
 
 static int
-check_option_list()
+check_option_list(void)
 {
        size_t size, i;
        ldns_edns_option_list* list;