aboutsummaryrefslogtreecommitdiffstats
path: root/extension
diff options
context:
space:
mode:
Diffstat (limited to 'extension')
-rw-r--r--extension/ChangeLog9
-rw-r--r--extension/readfile.c4
2 files changed, 11 insertions, 2 deletions
diff --git a/extension/ChangeLog b/extension/ChangeLog
index ca92b2a6..bbdf27a0 100644
--- a/extension/ChangeLog
+++ b/extension/ChangeLog
@@ -1,6 +1,13 @@
+2022-02-22 Arnold D. Robbins <arnold@skeeve.com>
+
+ Fix resource links found by Coverity. Thanks to
+ Jakub Martisko <jamartis@redhat.com> for the report.
+
+ * readfile.c (do_readfile): Close fd if text == NULL.
+
2021-12-10 Andrew J. Schorr <aschorr@telemetry-investments.com>
- *rwarray.c (write_number): Update comment to reflect that we are
+ * rwarray.c (write_number): Update comment to reflect that we are
now using mpfr_get_default_rounding_mode() instead of MPFR_RNDN.
2021-12-10 Arnold D. Robbins <arnold@skeeve.com>
diff --git a/extension/readfile.c b/extension/readfile.c
index 6c3307cd..def414ee 100644
--- a/extension/readfile.c
+++ b/extension/readfile.c
@@ -121,8 +121,10 @@ do_readfile(int nargs, awk_value_t *result, struct awk_ext_func *unused)
}
text = read_file_to_buffer(fd, & sbuf);
- if (text == NULL)
+ if (text == NULL) {
+ close(fd);
goto done; /* ERRNO already updated */
+ }
close(fd);
make_malloced_string(text, sbuf.st_size, result);