Re: [PATCH v3 00/11] hw/sd/sdcard: Fix CVE-2020-13253 & cleanups
On 7/6/20 6:31 PM, Alistair Francis wrote: > On Mon, Jun 8, 2020 at 10:48 AM Philippe Mathieu-Daudé > wrote: >> >> Hi Alistair, >> >> On 6/5/20 12:22 PM, Philippe Mathieu-Daudé wrote: >>> Patches 2 & 3 fix CVE-2020-13253. >>> The rest are (accumulated) cleanups. >>> >>> Supersedes: <20200604182502.24228-1-f4...@amsat.org> >>> >>> Philippe Mathieu-Daudé (11): >>> MAINTAINERS: Cc qemu-block mailing list >>> hw/sd/sdcard: Update coding style to make checkpatch.pl happy >>> hw/sd/sdcard: Do not switch to ReceivingData if address is invalid >>> hw/sd/sdcard: Restrict Class 6 commands to SCSD cards >>> hw/sd/sdcard: Update the SDState documentation >>> hw/sd/sdcard: Simplify cmd_valid_while_locked() >>> hw/sd/sdcard: Constify sd_crc*()'s message argument >>> hw/sd/sdcard: Make iolen unsigned >>> hw/sd/sdcard: Correctly display the command name in trace events >>> hw/sd/sdcard: Display offset in read/write_data() trace events >>> hw/sd/sdcard: Simplify realize() a bit >> >> I forgot to Cc you. >> >> Since you already reviewed a bunch of SD patches in the >> past, do you mind having a look a this series? It should >> be quite trivial. > > Hey, > > Sorry I took so long but I have reviewed a few. Let me know if there > are anymore you want reviewed. No problem, just in time :) Thank you a lot for all your reviews! Phil. > > Alistair
Re: [PATCH v3 00/11] hw/sd/sdcard: Fix CVE-2020-13253 & cleanups
On Mon, Jun 8, 2020 at 10:48 AM Philippe Mathieu-Daudé wrote: > > Hi Alistair, > > On 6/5/20 12:22 PM, Philippe Mathieu-Daudé wrote: > > Patches 2 & 3 fix CVE-2020-13253. > > The rest are (accumulated) cleanups. > > > > Supersedes: <20200604182502.24228-1-f4...@amsat.org> > > > > Philippe Mathieu-Daudé (11): > > MAINTAINERS: Cc qemu-block mailing list > > hw/sd/sdcard: Update coding style to make checkpatch.pl happy > > hw/sd/sdcard: Do not switch to ReceivingData if address is invalid > > hw/sd/sdcard: Restrict Class 6 commands to SCSD cards > > hw/sd/sdcard: Update the SDState documentation > > hw/sd/sdcard: Simplify cmd_valid_while_locked() > > hw/sd/sdcard: Constify sd_crc*()'s message argument > > hw/sd/sdcard: Make iolen unsigned > > hw/sd/sdcard: Correctly display the command name in trace events > > hw/sd/sdcard: Display offset in read/write_data() trace events > > hw/sd/sdcard: Simplify realize() a bit > > I forgot to Cc you. > > Since you already reviewed a bunch of SD patches in the > past, do you mind having a look a this series? It should > be quite trivial. Hey, Sorry I took so long but I have reviewed a few. Let me know if there are anymore you want reviewed. Alistair > > Thanks! > > Phil. > >
Re: [PATCH v3 00/11] hw/sd/sdcard: Fix CVE-2020-13253 & cleanups
On 6/5/20 12:22 PM, Philippe Mathieu-Daudé wrote: > Patches 2 & 3 fix CVE-2020-13253. Ping for the CVE fix?... > The rest are (accumulated) cleanups. > > Supersedes: <20200604182502.24228-1-f4...@amsat.org> > > Philippe Mathieu-Daudé (11): > MAINTAINERS: Cc qemu-block mailing list > hw/sd/sdcard: Update coding style to make checkpatch.pl happy > hw/sd/sdcard: Do not switch to ReceivingData if address is invalid > hw/sd/sdcard: Restrict Class 6 commands to SCSD cards > hw/sd/sdcard: Update the SDState documentation > hw/sd/sdcard: Simplify cmd_valid_while_locked() > hw/sd/sdcard: Constify sd_crc*()'s message argument > hw/sd/sdcard: Make iolen unsigned > hw/sd/sdcard: Correctly display the command name in trace events > hw/sd/sdcard: Display offset in read/write_data() trace events > hw/sd/sdcard: Simplify realize() a bit > > hw/sd/sd.c | 122 + > MAINTAINERS| 1 + > hw/sd/trace-events | 4 +- > 3 files changed, 83 insertions(+), 44 deletions(-) >
Re: [PATCH v3 00/11] hw/sd/sdcard: Fix CVE-2020-13253 & cleanups
Hi Alistair, On 6/5/20 12:22 PM, Philippe Mathieu-Daudé wrote: > Patches 2 & 3 fix CVE-2020-13253. > The rest are (accumulated) cleanups. > > Supersedes: <20200604182502.24228-1-f4...@amsat.org> > > Philippe Mathieu-Daudé (11): > MAINTAINERS: Cc qemu-block mailing list > hw/sd/sdcard: Update coding style to make checkpatch.pl happy > hw/sd/sdcard: Do not switch to ReceivingData if address is invalid > hw/sd/sdcard: Restrict Class 6 commands to SCSD cards > hw/sd/sdcard: Update the SDState documentation > hw/sd/sdcard: Simplify cmd_valid_while_locked() > hw/sd/sdcard: Constify sd_crc*()'s message argument > hw/sd/sdcard: Make iolen unsigned > hw/sd/sdcard: Correctly display the command name in trace events > hw/sd/sdcard: Display offset in read/write_data() trace events > hw/sd/sdcard: Simplify realize() a bit I forgot to Cc you. Since you already reviewed a bunch of SD patches in the past, do you mind having a look a this series? It should be quite trivial. Thanks! Phil.
[PATCH v3 00/11] hw/sd/sdcard: Fix CVE-2020-13253 & cleanups
Patches 2 & 3 fix CVE-2020-13253. The rest are (accumulated) cleanups. Supersedes: <20200604182502.24228-1-f4...@amsat.org> Philippe Mathieu-Daudé (11): MAINTAINERS: Cc qemu-block mailing list hw/sd/sdcard: Update coding style to make checkpatch.pl happy hw/sd/sdcard: Do not switch to ReceivingData if address is invalid hw/sd/sdcard: Restrict Class 6 commands to SCSD cards hw/sd/sdcard: Update the SDState documentation hw/sd/sdcard: Simplify cmd_valid_while_locked() hw/sd/sdcard: Constify sd_crc*()'s message argument hw/sd/sdcard: Make iolen unsigned hw/sd/sdcard: Correctly display the command name in trace events hw/sd/sdcard: Display offset in read/write_data() trace events hw/sd/sdcard: Simplify realize() a bit hw/sd/sd.c | 122 + MAINTAINERS| 1 + hw/sd/trace-events | 4 +- 3 files changed, 83 insertions(+), 44 deletions(-) -- 2.21.3