On 10/13/23 04:48, Mariusz Tkaczyk wrote: > On Wed, 11 Oct 2023 21:25:37 +0800 > Xiao Ni <xni@xxxxxxxxxx> wrote: > >> Sorry, the title should be [PATCH 1/1] mdadm/ddf: Abort when raid disk >> is smaller than 0 in getinfo_super_ddf >> >> If you want me to send v2, I can do it. >> >> Best Regards >> Xiao >> >> >> On Wed, Oct 11, 2023 at 9:06 PM Xiao Ni <xni@xxxxxxxxxx> wrote: >>> >>> The metadata is corrupted when the raid_disk<0. So abort directly. >>> This also can avoid a building error: >>> super-ddf.c:1988:58: error: array subscript -1 is below array bounds of >>> ‘struct phys_disk_entry[0]’ >>> >>> Suggested-by: Mariusz Tkaczyk <mariusz.tkaczyk@xxxxxxxxxxxxxxx> >>> Ackedy-by: Xiao Ni <xni@xxxxxxxxxx> >>> --- >>> super-ddf.c | 6 ++++-- >>> 1 file changed, 4 insertions(+), 2 deletions(-) >>> >>> diff --git a/super-ddf.c b/super-ddf.c >>> index 7213284e0a59..7b98333ecd51 100644 >>> --- a/super-ddf.c >>> +++ b/super-ddf.c >>> @@ -1984,12 +1984,14 @@ static void getinfo_super_ddf(struct supertype *st, >>> struct mdinfo *info, char *m info->disk.number = >>> be32_to_cpu(ddf->dlist->disk.refnum); info->disk.raid_disk = find_phys(ddf, >>> ddf->dlist->disk.refnum); >>> >>> + if (info->disk.raid_disk < 0) >>> + return; >>> + >>> info->data_offset = be64_to_cpu(ddf->phys-> >>> entries[info->disk.raid_disk]. >>> config_size); >>> info->component_size = ddf->dlist->size - info->data_offset; >>> - if (info->disk.raid_disk >= 0) >>> - pde = ddf->phys->entries + info->disk.raid_disk; >>> + pde = ddf->phys->entries + info->disk.raid_disk; >>> if (pde && >>> !(be16_to_cpu(pde->state) & DDF_Failed) && >>> !(be16_to_cpu(pde->state) & DDF_Missing)) >>> -- >>> 2.32.0 (Apple Git-132) >>> >> > > Hi Xiao, > I don't see a need to send v2. Moving to awaiting upstream: > > Acked-by: Mariusz Tkaczyk <mariusz.tkaczyk@xxxxxxxxxxxxxxx> Applied! Thanks, Jes