Wang Shilong
2013-Dec-11 11:22 UTC
[PATCH] Btrfs-progs: make sure we are not opening a block device in open_file_or_dir()
Now, open_file_or_dir() will return successfully if we pass a block device
for it, this is wrong, we should check if it is a block device before trying
to open it.
Before this patch:
# btrfs sub list /dev/sda8
It will output the following message:
ERROR: can''t perform the search -Inappropriate ioctl for device
ERROR: can''t get rootid for ''/dev/sda8
After applying the patch, it will output:
ERROR: can''t access ''/dev/sda8''
Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com>
---
utils.c | 50 ++++++++++++++++++++++++++++----------------------
1 file changed, 28 insertions(+), 22 deletions(-)
diff --git a/utils.c b/utils.c
index f499023..1c168e2 100644
--- a/utils.c
+++ b/utils.c
@@ -755,6 +755,31 @@ out:
return ret;
}
+static int __open_file_or_dir(const char *fname, DIR **dirstream)
+{
+ int ret;
+ struct stat st;
+ int fd;
+
+ ret = stat(fname, &st);
+ if (ret < 0)
+ return -1;
+ if (S_ISDIR(st.st_mode)) {
+ *dirstream = opendir(fname);
+ if (!*dirstream)
+ return -2;
+ fd = dirfd(*dirstream);
+ } else {
+ fd = open(fname, O_RDWR);
+ }
+ if (fd < 0) {
+ fd = -3;
+ if (*dirstream)
+ closedir(*dirstream);
+ }
+ return fd;
+}
+
/*
* Given a pathname, return a filehandle to:
* the original pathname or,
@@ -776,7 +801,7 @@ int open_path_or_dev_mnt(const char *path, DIR **dirstream)
errno = EINVAL;
return -1;
}
- fdmnt = open_file_or_dir(mp, dirstream);
+ fdmnt = __open_file_or_dir(mp, dirstream);
} else {
fdmnt = open_file_or_dir(path, dirstream);
}
@@ -1557,28 +1582,9 @@ u64 parse_size(char *s)
int open_file_or_dir(const char *fname, DIR **dirstream)
{
- int ret;
- struct stat st;
- int fd;
-
- ret = stat(fname, &st);
- if (ret < 0) {
+ if (is_block_device(fname))
return -1;
- }
- if (S_ISDIR(st.st_mode)) {
- *dirstream = opendir(fname);
- if (!*dirstream)
- return -2;
- fd = dirfd(*dirstream);
- } else {
- fd = open(fname, O_RDWR);
- }
- if (fd < 0) {
- fd = -3;
- if (*dirstream)
- closedir(*dirstream);
- }
- return fd;
+ return __open_file_or_dir(fname, dirstream);
}
void close_file_or_dir(int fd, DIR *dirstream)
--
1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html