|
 |
361e071 |
From 7b6dfd6390476ec7b811c76e4e2653db2994cad4 Mon Sep 17 00:00:00 2001
|
|
|
361e071 |
From: Jan Kara <jack@suse.cz>
|
|
|
361e071 |
Date: Mon, 5 Feb 2018 15:54:54 +0100
|
|
|
361e071 |
Subject: [PATCH] quotacheck: Report error when caching of quota file fails
|
|
|
361e071 |
MIME-Version: 1.0
|
|
|
361e071 |
Content-Type: text/plain; charset=UTF-8
|
|
|
361e071 |
Content-Transfer-Encoding: 8bit
|
|
|
361e071 |
|
|
|
361e071 |
Currently quotacheck returns with zero exit code even though caching of
|
|
|
361e071 |
quota file fails. Fix it to return with non-zero exit code in that case
|
|
|
361e071 |
as expected.
|
|
|
361e071 |
|
|
|
361e071 |
Reported-by: Christoph Biedl
|
|
|
361e071 |
Signed-off-by: Jan Kara <jack@suse.cz>
|
|
|
361e071 |
Signed-off-by: Petr Písař <ppisar@redhat.com>
|
|
|
361e071 |
---
|
|
|
361e071 |
quotacheck.c | 29 ++++++++++++++++++++++-------
|
|
|
361e071 |
1 file changed, 22 insertions(+), 7 deletions(-)
|
|
|
361e071 |
|
|
|
361e071 |
diff --git a/quotacheck.c b/quotacheck.c
|
|
|
361e071 |
index 9d7940b..b1302b6 100644
|
|
|
361e071 |
--- a/quotacheck.c
|
|
|
361e071 |
+++ b/quotacheck.c
|
|
|
361e071 |
@@ -924,6 +924,7 @@ static int check_dir(struct mount_entry *mnt)
|
|
|
361e071 |
struct stat st;
|
|
|
361e071 |
int remounted = 0;
|
|
|
361e071 |
int failed = 0;
|
|
|
361e071 |
+ int ret;
|
|
|
361e071 |
|
|
|
361e071 |
if (lstat(mnt->me_dir, &st) < 0)
|
|
|
361e071 |
die(2, _("Cannot stat mountpoint %s: %s\n"), mnt->me_dir, strerror(errno));
|
|
|
361e071 |
@@ -939,14 +940,22 @@ static int check_dir(struct mount_entry *mnt)
|
|
|
361e071 |
*/
|
|
|
361e071 |
if (cfmt == QF_XFS)
|
|
|
361e071 |
goto start_scan;
|
|
|
361e071 |
- if (ucheck)
|
|
|
361e071 |
- if (process_file(mnt, USRQUOTA) < 0)
|
|
|
361e071 |
+ if (ucheck) {
|
|
|
361e071 |
+ ret = process_file(mnt, USRQUOTA);
|
|
|
361e071 |
+ if (ret < 0) {
|
|
|
361e071 |
+ failed |= ret;
|
|
|
361e071 |
ucheck = 0;
|
|
|
361e071 |
- if (gcheck)
|
|
|
361e071 |
- if (process_file(mnt, GRPQUOTA) < 0)
|
|
|
361e071 |
+ }
|
|
|
361e071 |
+ }
|
|
|
361e071 |
+ if (gcheck) {
|
|
|
361e071 |
+ ret = process_file(mnt, GRPQUOTA);
|
|
|
361e071 |
+ if (ret < 0) {
|
|
|
361e071 |
+ failed |= ret;
|
|
|
361e071 |
gcheck = 0;
|
|
|
361e071 |
+ }
|
|
|
361e071 |
+ }
|
|
|
361e071 |
if (!ucheck && !gcheck) /* Nothing to check? */
|
|
|
361e071 |
- return 0;
|
|
|
361e071 |
+ return failed;
|
|
|
361e071 |
if (!(flags & FL_NOREMOUNT)) {
|
|
|
361e071 |
/* Now we try to remount fs read-only to prevent races when scanning filesystem */
|
|
|
361e071 |
if (mount
|
|
|
361e071 |
@@ -978,8 +987,11 @@ start_scan:
|
|
|
361e071 |
!strcmp(mnt->me_type, MNTTYPE_EXT3) ||
|
|
|
361e071 |
!strcmp(mnt->me_type, MNTTYPE_NEXT3) ||
|
|
|
361e071 |
!strcmp(mnt->me_type, MNTTYPE_EXT4)) {
|
|
|
361e071 |
- if ((failed = ext2_direct_scan(mnt->me_devname)) < 0)
|
|
|
361e071 |
+ ret = ext2_direct_scan(mnt->me_devname);
|
|
|
361e071 |
+ if (ret < 0) {
|
|
|
361e071 |
+ failed |= ret;
|
|
|
361e071 |
goto out;
|
|
|
361e071 |
+ }
|
|
|
361e071 |
}
|
|
|
361e071 |
else {
|
|
|
361e071 |
#else
|
|
|
361e071 |
@@ -987,8 +999,11 @@ start_scan:
|
|
|
361e071 |
#endif
|
|
|
361e071 |
if (flags & FL_VERYVERBOSE)
|
|
|
361e071 |
putchar('\n');
|
|
|
361e071 |
- if ((failed = scan_dir(mnt->me_dir)) < 0)
|
|
|
361e071 |
+ ret = scan_dir(mnt->me_dir);
|
|
|
361e071 |
+ if (ret < 0) {
|
|
|
361e071 |
+ failed |= ret;
|
|
|
361e071 |
goto out;
|
|
|
361e071 |
+ }
|
|
|
361e071 |
}
|
|
|
361e071 |
dirs_done++;
|
|
|
361e071 |
if (flags & FL_VERBOSE || flags & FL_DEBUG)
|
|
|
361e071 |
--
|
|
|
361e071 |
2.13.6
|
|
|
361e071 |
|