libmount: improve EPERM interpretation

In some cases mount(2)/open_tree(2) returns EPERM for root user. In
this case libmount reports it as "mount point is not a directory".

It does not makes sense for bind mount where target could be a
regular file.

This patch is not ideal, the error handler is generic, but semantic
for new mount API and mount(2) is different. For example now it checks
for regular file, but the new API supports bind over symlinks, so
proper fix will require lstat() and S_ISLNK(), etc. We need to move
error messages to hook_mount.c and mount_mount_legacy.c to make it
more specific.

Fixes: https://github.com/util-linux/util-linux/issues/2413
Signed-off-by: Karel Zak <kzak@redhat.com>

diff --git a/libmount/src/context_mount.c b/libmount/src/context_mount.c
index f7e280954582ab84351371774588fe7121d8e82b..41986e74b1603a0720fc7bb414b8e8844292a85c 100644 (file)
--- a/libmount/src/context_mount.c
+++ b/libmount/src/context_mount.c
@@ -1577,10 +1577,13 @@ int mnt_context_get_mount_excode(
                if (!buf)
                        break;
                if (geteuid() == 0) {
-                       if (mnt_safe_stat(tgt, &st) || !S_ISDIR(st.st_mode))
-                               snprintf(buf, bufsz, _("mount point is not a directory"));
-                       else
+
+                       if (mnt_safe_stat(tgt, &st) == 0
+                           && ((mflags & MS_BIND && S_ISREG(st.st_mode))
+                               || S_ISDIR(st.st_mode)))
                                snprintf(buf, bufsz, _("permission denied"));
+                       else
+                               snprintf(buf, bufsz, _("mount point is not a directory"));
                } else
                        snprintf(buf, bufsz, _("must be superuser to use mount"));
                break;