resolved: use _cleanup_ in one more place
No functional change.
This commit is contained in:
parent
8530efc1c3
commit
15c533103a
|
@ -1167,7 +1167,7 @@ static int digest_to_gcrypt_md(uint8_t algorithm) {
|
||||||
|
|
||||||
int dnssec_verify_dnskey_by_ds(DnsResourceRecord *dnskey, DnsResourceRecord *ds, bool mask_revoke) {
|
int dnssec_verify_dnskey_by_ds(DnsResourceRecord *dnskey, DnsResourceRecord *ds, bool mask_revoke) {
|
||||||
char owner_name[DNSSEC_CANONICAL_HOSTNAME_MAX];
|
char owner_name[DNSSEC_CANONICAL_HOSTNAME_MAX];
|
||||||
gcry_md_hd_t md = NULL;
|
_cleanup_(gcry_md_closep) gcry_md_hd_t md = NULL;
|
||||||
size_t hash_size;
|
size_t hash_size;
|
||||||
int md_algorithm, r;
|
int md_algorithm, r;
|
||||||
void *result;
|
void *result;
|
||||||
|
@ -1223,16 +1223,10 @@ int dnssec_verify_dnskey_by_ds(DnsResourceRecord *dnskey, DnsResourceRecord *ds,
|
||||||
gcry_md_write(md, dnskey->dnskey.key, dnskey->dnskey.key_size);
|
gcry_md_write(md, dnskey->dnskey.key, dnskey->dnskey.key_size);
|
||||||
|
|
||||||
result = gcry_md_read(md, 0);
|
result = gcry_md_read(md, 0);
|
||||||
if (!result) {
|
if (!result)
|
||||||
r = -EIO;
|
return -EIO;
|
||||||
goto finish;
|
|
||||||
}
|
|
||||||
|
|
||||||
r = memcmp(result, ds->ds.digest, ds->ds.digest_size) != 0;
|
return memcmp(result, ds->ds.digest, ds->ds.digest_size) != 0;
|
||||||
|
|
||||||
finish:
|
|
||||||
gcry_md_close(md);
|
|
||||||
return r;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
int dnssec_verify_dnskey_by_ds_search(DnsResourceRecord *dnskey, DnsAnswer *validated_ds) {
|
int dnssec_verify_dnskey_by_ds_search(DnsResourceRecord *dnskey, DnsAnswer *validated_ds) {
|
||||||
|
|
Loading…
Reference in New Issue