diff options
| author | Karel Zak | 2017-09-15 14:30:06 +0200 |
|---|---|---|
| committer | Karel Zak | 2017-09-15 14:30:06 +0200 |
| commit | d369dc42cb4974d6a928ae28ed32fa11b528e5fb (patch) | |
| tree | ffa3e5f491285c84cc5cad85d73b929917fabc30 /libmount/src/context_mount.c | |
| parent | libsmartcols: fix heap-buffer-overflow when move columns (diff) | |
| download | kernel-qcow2-util-linux-d369dc42cb4974d6a928ae28ed32fa11b528e5fb.tar.gz kernel-qcow2-util-linux-d369dc42cb4974d6a928ae28ed32fa11b528e5fb.tar.xz kernel-qcow2-util-linux-d369dc42cb4974d6a928ae28ed32fa11b528e5fb.zip | |
libmount: export locking errors as MNT_EX_FILEIO
The mount man page assumes locking errors mapped to MNT_EX_FILEIO (16)
return code. Unfortunately, this is internally not exported as a
special error code, so it's returned as a generic (errno based)
stuff. This patch fixes this issue.
Note that we still use locking for example for utab or when enabled
/etc/mtab (disabled by default).
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'libmount/src/context_mount.c')
| -rw-r--r-- | libmount/src/context_mount.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/libmount/src/context_mount.c b/libmount/src/context_mount.c index 65f7dbfd0..e84947f00 100644 --- a/libmount/src/context_mount.c +++ b/libmount/src/context_mount.c @@ -1379,6 +1379,10 @@ int mnt_context_get_mount_excode( if (buf) snprintf(buf, bufsz, _("overlapping loop device exists for %s"), src); return MNT_EX_FAIL; + case -MNT_ERR_LOCK: + if (buf) + snprintf(buf, bufsz, _("locking failed")); + return MNT_EX_FILEIO; default: return mnt_context_get_generic_excode(rc, buf, bufsz, _("mount failed: %m")); } @@ -1388,7 +1392,12 @@ int mnt_context_get_mount_excode( * mount(2) syscall success, but something else failed * (probably error in mtab processing). */ - if (rc < 0) + if (rc == -MNT_ERR_LOCK) { + if (buf) + snprintf(buf, bufsz, _("filesystem was mounted, but failed to update userspace mount table")); + return MNT_EX_FILEIO; + + } else if (rc < 0) return mnt_context_get_generic_excode(rc, buf, bufsz, _("filesystem was mounted, but any subsequent operation failed: %m")); |