diff --git a/0028-mdadm-fix-coredump-of-mdadm-monitor-r.patch b/0028-mdadm-fix-coredump-of-mdadm-monitor-r.patch new file mode 100644 index 0000000000000000000000000000000000000000..cb86a718a363cee96e22074435b0faeace0e1cb4 --- /dev/null +++ b/0028-mdadm-fix-coredump-of-mdadm-monitor-r.patch @@ -0,0 +1,39 @@ +From 546047688e1c64638f462147c755b58119cabdc8 Mon Sep 17 00:00:00 2001 +From: Wu Guanghao +Date: Mon, 16 Aug 2021 15:24:51 +0800 +Subject: [PATCH] mdadm: fix coredump of mdadm --monitor -r + +The --monitor -r option requires a parameter, otherwise a null pointer will be manipulated +when converting to integer data, and a coredump will appear. + +Segmentation fault (core dumped) + +Signed-off-by: Wu Guanghao +Acked-by: Mariusz Tkaczyk +Signed-off-by: Jes Sorensen +--- + ReadMe.c | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +diff --git a/ReadMe.c b/ReadMe.c +index 978796ac..37d84625 100644 +--- a/ReadMe.c ++++ b/ReadMe.c +@@ -81,11 +81,11 @@ char Version[] = "mdadm - v" VERSION " - " VERS_DATE EXTRAVERSION "\n"; + * found, it is started. + */ + +-char short_options[]="-ABCDEFGIQhVXYWZ:vqbc:i:l:p:m:n:x:u:c:d:z:U:N:sarfRSow1tye:k:"; ++char short_options[]="-ABCDEFGIQhVXYWZ:vqbc:i:l:p:m:r:n:x:u:c:d:z:U:N:safRSow1tye:k"; + char short_bitmap_options[]= +- "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:n:x:u:c:d:z:U:N:sarfRSow1tye:k:"; ++ "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:r:n:x:u:c:d:z:U:N:sarfRSow1tye:k:"; + char short_bitmap_auto_options[]= +- "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:n:x:u:c:d:z:U:N:sa:rfRSow1tye:k:"; ++ "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:r:n:x:u:c:d:z:U:N:sa:rfRSow1tye:k:"; + + struct option long_options[] = { + {"manage", 0, 0, ManageOpt}, +-- +2.27.0 + diff --git a/0029-mdadm-Fix-mdadm-r-remove-option-regresision.patch b/0029-mdadm-Fix-mdadm-r-remove-option-regresision.patch new file mode 100644 index 0000000000000000000000000000000000000000..4ae6c086c483f2f36528a3f6262f8553c1d98e13 --- /dev/null +++ b/0029-mdadm-Fix-mdadm-r-remove-option-regresision.patch @@ -0,0 +1,82 @@ +From 7b94dd6398bc08ec58088c3ca93a8bcd6e4c2c04 Mon Sep 17 00:00:00 2001 +From: Logan Gunthorpe +Date: Thu, 12 May 2022 10:30:55 -0600 +Subject: [PATCH] mdadm: Fix mdadm -r remove option regresision + +The commit noted below globally adds a parameter to the -r option but missed +the fact that -r is used for another purpose: --remove. + +After that commit, a command such as: + + mdadm /dev/md0 -r /dev/loop0 + +will do nothing seeing the device parameter will be consumed as a +argument to the -r option; thus, there will only be one device +seen one the command line, devs_found will only be 1 and nothing will +happen. + +This caused the 01r5integ and 01raid6integ tests to hang indefinitely +as mdadm did not remove the failed device. With the device not removed, +it would not be readded. Then the loop waiting for the array status to +change would loop forever. + +To fix this, revert the changes in the noted patch and create a new subopt +type for the monitor mode with parameters required for -r. + +Fixes: 546047688e1c ("mdadm: fix coredump of mdadm --monitor -r") +Cc: Wu Guanghao +Cc: Mariusz Tkaczyk +Signed-off-by: Logan Gunthorpe +--- + ReadMe.c | 7 ++++--- + mdadm.c | 1 + + mdadm.h | 1 + + 3 files changed, 6 insertions(+), 3 deletions(-) + +diff --git a/ReadMe.c b/ReadMe.c +index 8f873c48..556104f7 100644 +--- a/ReadMe.c ++++ b/ReadMe.c +@@ -81,11 +81,12 @@ char Version[] = "mdadm - v" VERSION " - " VERS_DATE EXTRAVERSION "\n"; + * found, it is started. + */ + +-char short_options[]="-ABCDEFGIQhVXYWZ:vqbc:i:l:p:m:r:n:x:u:c:d:z:U:N:safRSow1tye:k"; ++char short_options[]="-ABCDEFGIQhVXYWZ:vqbc:i:l:p:m:n:x:u:c:d:z:U:N:sarfRSow1tye:k"; ++char short_monitor_options[]="-ABCDEFGIQhVXYWZ:vqbc:i:l:p:m:r:n:x:u:c:d:z:U:N:safRSow1tye:k"; + char short_bitmap_options[]= +- "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:r:n:x:u:c:d:z:U:N:sarfRSow1tye:k:"; ++ "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:n:x:u:c:d:z:U:N:sarfRSow1tye:k:"; + char short_bitmap_auto_options[]= +- "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:r:n:x:u:c:d:z:U:N:sa:rfRSow1tye:k:"; ++ "-ABCDEFGIQhVXYWZ:vqb:c:i:l:p:m:n:x:u:c:d:z:U:N:sa:rfRSow1tye:k:"; + + struct option long_options[] = { + {"manage", 0, 0, ManageOpt}, +diff --git a/mdadm.c b/mdadm.c +index be40686c..d0c5e6de 100644 +--- a/mdadm.c ++++ b/mdadm.c +@@ -227,6 +227,7 @@ int main(int argc, char *argv[]) + shortopt = short_bitmap_auto_options; + break; + case 'F': newmode = MONITOR; ++ shortopt = short_monitor_options; + break; + case 'G': newmode = GROW; + shortopt = short_bitmap_options; +diff --git a/mdadm.h b/mdadm.h +index 09915a00..559da3f6 100644 +--- a/mdadm.h ++++ b/mdadm.h +@@ -419,6 +419,7 @@ enum mode { + }; + + extern char short_options[]; ++extern char short_monitor_options[]; + extern char short_bitmap_options[]; + extern char short_bitmap_auto_options[]; + extern struct option long_options[]; +-- +2.27.0 + diff --git a/mdadm.spec b/mdadm.spec index 3ec31a5d2fea5794f26a53d429c7c9d67ce69df6..69176cee2d3c8486de69537cdfd2cda504ba68c6 100644 --- a/mdadm.spec +++ b/mdadm.spec @@ -1,6 +1,6 @@ Name: mdadm Version: 4.1 -Release: 5 +Release: 6 Summary: The software RAID arrays user manage tools License: GPLv2+ URL: http://www.kernel.org/pub/linux/utils/raid/mdadm/ @@ -37,6 +37,8 @@ Patch24: 0024-Block-overwriting-existing-links-while-manual-assemb.patch Patch25: 0025-Detect-too-small-device-error-rather-than-underflow-.patch Patch26: 0026-mdadm-Grow-prevent-md-s-fd-from-being-occupied-durin.patch Patch27: 0027-mdadm-remove-Werror-to-fix-Werror-address-of-packed-.patch +Patch28: 0028-mdadm-fix-coredump-of-mdadm-monitor-r.patch +Patch29: 0029-mdadm-Fix-mdadm-r-remove-option-regresision.patch BuildRequires: systemd gcc binutils Requires(post): systemd coreutils @@ -102,6 +104,9 @@ install -d -m 710 %{buildroot}/var/run/mdadm/ %{_mandir}/man*/* %changelog +* Sat Jun 25 2022 wuguanghao - 4.1-6 +- fix segfault of --monitor -r + * Wed Nov 4 2020 lixiaokeng - 4.1-5 - add make test