diff options
author | Miao Xie <miaox@cn.fujitsu.com> | 2013-07-03 21:25:10 +0800 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2013-07-03 14:06:54 -0400 |
commit | 0daa219ee1119e30059077da5be03c4506963fe2 (patch) | |
tree | 7fc42861f945e853321147bf7a4a0f0e556ef270 /btrfs-find-root.c | |
parent | 32a8c1dd905714e91b7d590aa7d833c475e737d4 (diff) |
Btrfs-progs: don't close the file descriptor 0 when closing a device
As we know, the file descriptor 0 is a special number, so we shouldn't
use it to initialize the file descriptor of the devices, or we might
close this special file descriptor by mistake when we close the devices.
"-1" is a better choice.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'btrfs-find-root.c')
-rw-r--r-- | btrfs-find-root.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/btrfs-find-root.c b/btrfs-find-root.c index 810d8358..3e1396d3 100644 --- a/btrfs-find-root.c +++ b/btrfs-find-root.c @@ -76,7 +76,10 @@ static int close_all_devices(struct btrfs_fs_info *fs_info) list = &fs_info->fs_devices->devices; list_for_each(next, list) { device = list_entry(next, struct btrfs_device, dev_list); - close(device->fd); + if (device->fd != -1) { + close(device->fd); + device->fd = -1; + } } return 0; } |