mirror/bbfdm
1
0
Fork 0
mirror of https://dev.iopsys.eu/bbf/bbfdm.git synced 2025-12-10 07:44:39 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

security: fix memory and file descriptor leak on DM_STOP

Browse source
This commit is contained in:
Erik Karlsson 2022-04-01 16:43:15 +02:00
parent 36c88d403f
commit e8047b0cde
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
dmtree/tr181/security.c
View file

@ -141,8 +141,8 @@ static int browseSecurityCertificateInst(struct dmctx *dmctx, DMNODE *parent_nod
struct certificate_profile certificateprofile = {};
get_certificate_paths();
int i;
for (i = 0; i < MAX_CERT; i++) {
int i, status;
for (i = 0, status = DM_OK; i < MAX_CERT && status != DM_STOP; i++) {
if(!DM_STRLEN(certifcates_paths[i]))
break;
FILE *fp = fopen(certifcates_paths[i], "r");
@ -162,8 +162,7 @@ static int browseSecurityCertificateInst(struct dmctx *dmctx, DMNODE *parent_nod
inst = handle_instance(dmctx, parent_node, dmmap_sect, "security_certificate_instance", "security_certificate_alias");
if (DM_LINK_INST_OBJ(dmctx, parent_node, (void *)&certificateprofile, inst) == DM_STOP)
break;
status = DM_LINK_INST_OBJ(dmctx, parent_node, (void *)&certificateprofile, inst);
X509_free(cert);
cert = NULL;