tree: Use SAFE_FOPEN()

Signed-off-by: Petr Vorel <[email protected]>

Author: pevik

testcases/kernel/mem/hugetlb/hugemmap/hugemmap34.c

@@ -21,6 +21,7 @@
 #define _GNU_SOURCE
 #include "lapi/mmap.h"
 #include "hugetlb.h"
+#include "tst_safe_stdio.h"
 #include <errno.h>
 #include <inttypes.h>
 #include <sched.h>
@@ -80,12 +81,9 @@ static void run_test(void)
 }
 
 /* Return start address of next mapping or 0 */
-static uintptr_t  get_next_mapping_start(uintptr_t addr)
+static uintptr_t get_next_mapping_start(uintptr_t addr)
 {
-	FILE *fp = fopen("/proc/self/maps", "r");
-
-	if (fp == NULL)
-		tst_brk(TBROK | TERRNO, "Failed to open /proc/self/maps.");
+	FILE *fp = SAFE_FOPEN("/proc/self/maps", "r");
 
 	while (!feof(fp)) {
 		uintptr_t start, end;
@@ -94,7 +92,7 @@ static uintptr_t  get_next_mapping_start(uintptr_t addr)
 		ret = fscanf(fp, "%"PRIxPTR"-%"PRIxPTR" %*[^\n]\n", &start, &end);
 		if (ret != 2) {
 			fclose(fp);
-			tst_brk(TBROK | TERRNO, "Couldn't parse /proc/self/maps line.");
+			tst_brk(TBROK | TERRNO, "Couldn't parse /proc/self/maps line");
 		}
 
 		if (start > addr) {

testcases/kernel/mem/vma/vma02.c

@@ -112,9 +112,7 @@ int main(int argc, char **argv)
 
 		/* /proc/self/maps in the form of
 		   "00400000-00406000 r-xp 00000000". */
-		fp = fopen("/proc/self/maps", "r");
-		if (fp == NULL)
-			tst_brkm(TBROK | TERRNO, NULL, "fopen");
+		fp = SAFE_FOPEN(NULL, "/proc/self/maps", "r");
 
 		while (fgets(buf, BUFSIZ, fp) != NULL) {
 			if (sscanf(buf, "%p-%p ", &start, &end) != 2)

testcases/kernel/mem/vma/vma04.c

@@ -202,9 +202,8 @@ static void get_vmas(char *retbuf, void *addr_s, void *addr_e)
 
 	retbuf[0] = '\0';
 	flag = 0;
-	fp = fopen("/proc/self/maps", "r");
-	if (fp == NULL)
-		tst_brkm(TBROK | TERRNO, cleanup, "fopen");
+	fp = SAFE_FOPEN(NULL, "/proc/self/maps", "r");
+
 	while (fgets(buf, BUFSIZ, fp) != NULL) {
 		if (sscanf(buf, "%p-%p ", &s, &t) != 2)
 			continue;

testcases/kernel/security/kallsyms/kallsyms.c

@@ -99,9 +99,7 @@ static void read_proc_self_maps(void)
 	FILE *fp;
 
 	ranges_len = 0;
-	fp = fopen("/proc/self/maps", "r");
-	if (fp == NULL)
-		tst_brk(TBROK | TERRNO, "Failed to open /proc/self/maps.");
+	fp = SAFE_FOPEN("/proc/self/maps", "r");
 
 	while (!feof(fp)) {
 		unsigned long start, end;