Martchus
/
mdadm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Correct checking if file descriptors are valid 

In some cases file descriptors equal to 0 are treated as invalid.
Fix it.

Signed-off-by: Mateusz Grzonka <mateusz.grzonka@intel.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
This commit is contained in:
Mateusz Grzonka 2021-11-24 11:48:33 +01:00 committed by Jes Sorensen
parent b2e4f08414
commit b71de056ce
5 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Assemble.c
View File

@ -569,8 +569,7 @@ static int select_devices(struct mddev_dev *devlist,
if (dfd < 0 ||
st->ss->load_super(st, dfd, NULL))
tmpdev->used = 2;
if (dfd > 0)
close(dfd);
close_fd(&dfd);
}
}

5
Grow.c
View File

@ -2334,10 +2334,7 @@ size_change_error:
* number of devices (On-Line Capacity Expansion) must be
* performed at the level of the container
*/
if (fd > 0) {
close(fd);
fd = -1;
}
close_fd(&fd);
rv = reshape_container(container, devname, -1, st, &info,
c->force, c->backup_file, c->verbose,
0, 0, 0);

4
Incremental.c
View File

@ -1702,8 +1702,8 @@ int IncrementalRemove(char *devname, char *id_path, int verbose)
return 1;
}
mdfd = open_dev_excl(ent->devnm);
if (mdfd > 0) {
close(mdfd);
if (is_fd_valid(mdfd)) {
close_fd(&mdfd);
if (sysfs_get_str(&mdi, NULL, "array_state",
buf, sizeof(buf)) > 0) {
if (strncmp(buf, "active", 6) == 0 ||

3
mdadm.c
View File

@ -1770,8 +1770,7 @@ int main(int argc, char *argv[])
}
if (locked)
cluster_release_dlmlock();
if (mdfd > 0)
close(mdfd);
close_fd(&mdfd);
exit(rv);
}

2
super-ddf.c
View File

@ -4914,7 +4914,7 @@ static int raid10_degraded(struct mdinfo *info)
pr_err("BUG: invalid raid disk\n");
goto out;
}
if (d->state_fd > 0)
if (is_fd_valid(d->state_fd))
found[i]++;
}
ret = 2;