Change in libosmocore[master]: osmo-config-merge: fclose() what we fopen()

[Harald Welte]

Harald Welte has uploaded this change for review. ( https://gerrit.osmocom.org/11396


Change subject: osmo-config-merge: fclose() what we fopen()
......................................................................

osmo-config-merge: fclose() what we fopen()

Change-Id: Ib44f902e5bca2277f1b340d34fabbd2a298d683e
Fixes: Coverity CID#188851
---
M utils/osmo-config-merge.c
1 file changed, 2 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/96/11396/1

diff --git a/utils/osmo-config-merge.c b/utils/osmo-config-merge.c
index a872d64..afaf86b 100644
--- a/utils/osmo-config-merge.c
+++ b/utils/osmo-config-merge.c
@@ -144,6 +144,7 @@
 			if (indent > cur_indent+1) {
 				fprintf(stderr, "File '%s' isn't well-formed in line %u, aborting!\n",
 					fname, line_num);
+				fclose(infile);
 				return NULL;
 			}
 			/* new child to last node */
@@ -166,6 +167,7 @@
 		cur_indent = indent;
 	}
 
+	fclose(infile);
 	return root;
 }
 

-- 
To view, visit https://gerrit.osmocom.org/11396
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib44f902e5bca2277f1b340d34fabbd2a298d683e
Gerrit-Change-Number: 11396
Gerrit-PatchSet: 1
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181021/49756523/attachment.htm>