From 6a353c81587e3050b7e48fe522b517de9b29e2f3 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 19 Nov 2006 17:34:57 +0000 Subject: [PATCH] mount: if FILE* is NULL, it's not wise to use it. --- util-linux/mount.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/util-linux/mount.c b/util-linux/mount.c index 4069416d9..027adc568 100644 --- a/util-linux/mount.c +++ b/util-linux/mount.c @@ -261,14 +261,16 @@ static int mount_it_now(struct mntent *mp, int vfsflags, char *filteropts) /* If the mount was successful, and we're maintaining an old-style * mtab file by hand, add the new entry to it now. */ -mtab: + mtab: if (ENABLE_FEATURE_MTAB_SUPPORT && useMtab && !rc && !(vfsflags & MS_REMOUNT)) { char *fsname; FILE *mountTable = setmntent(bb_path_mtab_file, "a+"); int i; - if (!mountTable) + if (!mountTable) { bb_error_msg("no %s",bb_path_mtab_file); + goto ret; + } // Add vfs string flags @@ -300,7 +302,7 @@ mtab: free(fsname); } } - + ret: return rc; } -- 2.25.1