Merge pull request #224 from jow-/lib-fs-readline-leak

fs: fix potential memory leak on i/o errors in .read()
jow- · Sep 23, 2024 · 6e88c62 · 6e88c62
2 parents b610860 + 67cd123
commit 6e88c62
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/fs.c b/lib/fs.c
@@ -130,8 +130,10 @@ uc_fs_read_common(uc_vm_t *vm, size_t nargs, const char *type)
 		if (llen == 4 && !strcmp(lstr, "line")) {
 			llen = getline(&p, &rlen, *fp);
 
-			if (llen == -1)
+			if (llen == -1) {
+				free(p);
 				err_return(errno);
+			}
 
 			len = (size_t)llen;
 		}
@@ -158,8 +160,10 @@ uc_fs_read_common(uc_vm_t *vm, size_t nargs, const char *type)
 		else if (llen == 1) {
 			llen = getdelim(&p, &rlen, *lstr, *fp);
 
-			if (llen == -1)
+			if (llen == -1) {
+				free(p);
 				err_return(errno);
+			}
 
 			len = (size_t)llen;
 		}