From 4595ada2f69b42670c85a63c7d2344af63f2afe7 Mon Sep 17 00:00:00 2001
From: Joseph Hunkeler <jhunkeler@gmail.com>
Date: Sun, 29 Dec 2019 01:35:09 -0500
Subject: Minor fixes:

* size_t in place of int
* Moved some variables closer to their execution scope
* Add some error checks
---
 src/find.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'src/find.c')

diff --git a/src/find.c b/src/find.c
index d2fc5a2..6db46e7 100644
--- a/src/find.c
+++ b/src/find.c
@@ -39,6 +39,7 @@ char *find_file(const char *root, const char *filename) {
     }
 
     if (!(rootpath = realpath(root, NULL))) {
+        free(path);
         return NULL;
     }
 
@@ -103,11 +104,13 @@ int find_in_file(const char *filename, const char *pattern) {
     }
 
     long int file_len = get_file_size(filename);
-    if (file_len < 0) {
+    if (file_len == -1) {
+        fclose(fp);
         return -1;
     }
     char *buffer = (char *)calloc((size_t) file_len, sizeof(char));
     if (!buffer) {
+        fclose(fp);
         return -1;
     }
     size_t pattern_len = strlen(pattern);
-- 
cgit