oysteikt/heimdal
0
0
Fork 0
Code Issues 5 Pull Requests Releases Activity

httpkadmind: Fix FD leak

Browse Source 
Using `mkstemp()` is nice, but if you don't need the FD, then don't
forget to close it!
This commit is contained in:
Nicolas Williams
2020-09-15 15:24:29 -05:00
parent 4a4ceaf287
commit c12ca129b6
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
kdc/httpkadmind.c
View File

@@ -990,6 +990,8 @@ make_keytab(kadmin_request_desc r)
ret = errno; ret = errno;
if (ret == 0) if (ret == 0)
ret = krb5_kt_resolve(r->context, r->keytab_name, &r->keytab); ret = krb5_kt_resolve(r->context, r->keytab_name, &r->keytab);
if (fd != -1)
(void) close(fd);
return ret; return ret;
} }