fix get_binary

docs/lfinput.md

@@ -6,14 +6,15 @@ I/O module to assist with consuming data from files
 
 ### get_binary
 
-Reads a file at `path` and returns the contents as a char* buffer. The buffer is allocated inside the function and
+Reads a file at `path` and returns the contents as a char* buffer. The `fsz` parameter stores the length of the
-the user is responsible for freeing it when finished.
+returned array. The buffer is allocated inside the function, so the user is responsible for freeing it when finished.
 
 ```c
-char *get_binary(const char *path);
+char *get_binary(const char *path, size_t *fsz);
 
 /* Usage */
-char *buf = get_binary("/home/evan/binfile");
+size_t fsz = 0;
+char *buf = get_binary("/home/evan/binfile", &fsz);
 free(buf);
 ```
 

include/lfinput.h

@@ -3,7 +3,7 @@
 
 #include <stdlib.h>
 
-char *get_binary(const char *);
+char *get_binary(const char *, size_t *fsz);
 
 char *get_input(const char *);
 

src/lfinput.c

@@ -26,22 +26,21 @@ static FILE* open_file(const char *path, size_t *fsz) {
     return fp;
 }
 
-char *get_binary(const char *path) {
+char *get_binary(const char *path, size_t *fsz) {
-    size_t fsz = 0;
+    FILE *fp = open_file(path, fsz);
-    FILE *fp = open_file(path, &fsz);
     if (fp == NULL) {
         return NULL;
     }
 
     char *buf = NULL;
-    buf = malloc(fsz);
+    buf = malloc(*fsz);
     if (buf == NULL) {
         fprintf(stderr, "Failed to malloc buf. Returning NULL\n");
         fclose(fp);
         return NULL;
     }
 
-    fread(buf, 1, fsz, fp);
+    fread(buf, 1, *fsz, fp);
     fclose(fp);
 
     return buf;