diff options
author | Justin Hibbits <jhibbits@FreeBSD.org> | 2018-02-13 17:40:09 +0000 |
---|---|---|
committer | Justin Hibbits <jhibbits@FreeBSD.org> | 2018-02-13 17:40:09 +0000 |
commit | 08a3b42fdb4d439298a3bf24cf56003cb349b703 (patch) | |
tree | af9d186891268ed13f90bafbeaaac9bca889499b /sys/geom/part/g_part_apm.c | |
parent | 157f3d764973a807959284721312f5dda2149f8e (diff) | |
download | src-08a3b42fdb4d439298a3bf24cf56003cb349b703.tar.gz src-08a3b42fdb4d439298a3bf24cf56003cb349b703.zip |
Narrow a race, and fix a leak, in g_part_wither
A race in g_part_wither() can lead to I/O being performed with a freed GEOM
when the device disappears. Close the race as best as we can for now,
following the code patterns from g_part_ctl_destroy() and g_part_ctl_undo().
This also fixes a leak, as g_wither_geom() does not wither providers, it
only orphans them, so the partition entries would never get destroyed in
g_wither_washer().
Note, this is not a complete fix, it can still race with g_part_start(), the
race has merely been narrowed.
Reviewed by: markj
Sponsored by: Dell EMC Isilon
Notes
Notes:
svn path=/head/; revision=329225
Diffstat (limited to 'sys/geom/part/g_part_apm.c')
0 files changed, 0 insertions, 0 deletions