Re: [PATCH v12 2/3] block: change annotation of rdb_CylBlocks in affs_hardblocks.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael,

On Sun, Jun 18, 2023 at 5:10 AM Michael Schmitz <schmitzmic@xxxxxxxxx> wrote:
> Am 17.06.2023 um 23:08 schrieb Geert Uytterhoeven:
> > On Sat, Jun 17, 2023 at 12:36 AM Michael Schmitz <schmitzmic@xxxxxxxxx> wrote:
> >> The Amiga partition parser module uses signed int for partition sector
> >> address and count, which will overflow for disks larger than 1 TB.
> >>
> >> Use u64 as type for sector address and size to allow using disks up to
> >> 2 TB without LBD support, and disks larger than 2 TB with LBD. The RBD
> >> format allows to specify disk sizes up to 2^128 bytes (though native
> >> OS limitations reduce this somewhat, to max 2^68 bytes), so check for
> >> u64 overflow carefully to protect against overflowing sector_t.
> >>
> >> This bug was reported originally in 2012, and the fix was created by
> >> the RDB author, Joanne Dow <jdow@xxxxxxxxxxxxx>. A patch had been
> >> discussed and reviewed on linux-m68k at that time but never officially
> >> submitted (now resubmitted as patch 1 of this series).
> >>
> >> Patch 3 (this series) adds additional error checking and warning
> >> messages. One of the error checks now makes use of the previously
> >> unused rdb_CylBlocks field, which causes a 'sparse' warning
> >> (cast to restricted __be32).
> >>
> >> Annotate all 32 bit fields in affs_hardblocks.h as __be32, as the
> >> on-disk format of RDB and partition blocks is always big endian.
> >>
> >> Reported-by: Martin Steigerwald <Martin@xxxxxxxxxxxx>
> >> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=43511
> >> Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
> >> Message-ID: <201206192146.09327.Martin@xxxxxxxxxxxx>
> >> Cc: <stable@xxxxxxxxxxxxxxx> # 5.2
> >> Signed-off-by: Michael Schmitz <schmitzmic@xxxxxxxxx>
> >> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> >
> > Reviewed-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
>
> Thanks - now I notice the patch title for this one doesn't fit too well
> anymore.
>
> Would a change of title mess up the common patch tracking tools?

You mean changing one patch subject in v13?
Nah, happens all the time, so the tooling should handle that fine.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux