From 0ac746db72a0879bbd44325fe15b3ae33c63ecef Mon Sep 17 00:00:00 2001 From: "Andrew J. Schorr" Date: Thu, 22 Jun 2017 11:02:08 -0400 Subject: In readfile extension, no need to zero out the buffer before overwriting it. --- extension/ChangeLog | 6 ++++++ extension/readfile.c | 14 ++++++-------- 2 files changed, 12 insertions(+), 8 deletions(-) (limited to 'extension') diff --git a/extension/ChangeLog b/extension/ChangeLog index 378de373..9574865c 100644 --- a/extension/ChangeLog +++ b/extension/ChangeLog @@ -1,3 +1,9 @@ +2017-06-22 Andrew J. Schorr + + * readfile.c (read_file_to_buffer): Use emalloc instead of ezalloc, + since there's no need to initialize the memory to zero before + overwriting it with the file's contents. + 2017-06-21 Andrew J. Schorr * filefuncs.c (do_fts): Replace emalloc+memset with ezalloc. diff --git a/extension/readfile.c b/extension/readfile.c index f470237b..b600f27a 100644 --- a/extension/readfile.c +++ b/extension/readfile.c @@ -73,24 +73,22 @@ int plugin_is_GPL_compatible; static char * read_file_to_buffer(int fd, const struct stat *sbuf) { - char *text = NULL; - int ret; + char *text; if ((sbuf->st_mode & S_IFMT) != S_IFREG) { errno = EINVAL; update_ERRNO_int(errno); - goto done; + return NULL; } - ezalloc(text, char *, sbuf->st_size + 1, "do_readfile"); + emalloc(text, char *, sbuf->st_size + 1, "do_readfile"); - if ((ret = read(fd, text, sbuf->st_size)) != sbuf->st_size) { + if (read(fd, text, sbuf->st_size) != sbuf->st_size) { update_ERRNO_int(errno); gawk_free(text); - text = NULL; - /* fall through to return */ + return NULL; } -done: + text[sbuf->st_size] = '\0'; return text; } -- cgit v1.2.3