c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2025-10-26 18:05:04 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix null dereference in spa_vdev_remove_cancel_sync()

Browse Source 
We don't really need to access space map to know where the metaslab
ends, while msp->ms_sm might be NULL.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Rob Norris <robn@despairlabs.com>
Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com>
Signed-off-by:	Alexander Motin <mav@FreeBSD.org>
Sponsored by:	iXsystems, Inc.
Fixes #17164
Fixes #17359
Closes #17361
This commit is contained in:
Alexander Motin 2025-05-22 10:47:43 -04:00 committed by GitHub
parent a6f20250de
commit 5c30b24381
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
module/zfs/vdev_removal.c
View File

@ -1931,10 +1931,9 @@ spa_vdev_remove_cancel_sync(void *arg, dmu_tx_t *tx)
* because we have not allocated mappings for it yet. * because we have not allocated mappings for it yet.
*/ */
uint64_t syncd = vdev_indirect_mapping_max_offset(vim); uint64_t syncd = vdev_indirect_mapping_max_offset(vim);
uint64_t sm_end = msp->ms_sm->sm_start + uint64_t ms_end = msp->ms_start + msp->ms_size;
msp->ms_sm->sm_size; if (ms_end > syncd)
if (sm_end > syncd) zfs_range_tree_clear(segs, syncd, ms_end - syncd);
zfs_range_tree_clear(segs, syncd, sm_end - syncd);
zfs_range_tree_vacate(segs, free_mapped_segment_cb, vd); zfs_range_tree_vacate(segs, free_mapped_segment_cb, vd);
} }