hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

[PATCH 0/3] hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

Philippe Mathieu-Daudé posted 3 patches 4 years, 7 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210702155900.148665-1-f4bug@amsat.org

hw/sd/sd.c                     | 40 ++++++++++++++-------
tests/qtest/fuzz-sdcard-test.c | 66 ++++++++++++++++++++++++++++++++++
MAINTAINERS                    |  3 +-
tests/qtest/meson.build        |  1 +
4 files changed, 96 insertions(+), 14 deletions(-)
create mode 100644 tests/qtest/fuzz-sdcard-test.c

Expand all Fold all

[PATCH 0/3] hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

Posted by Philippe Mathieu-Daudé 4 years, 7 months ago

Trivial fix for https://gitlab.com/qemu-project/qemu/-/issues/450

Missing review: patch #3

Philippe Mathieu-Daudé (3):
  hw/sd: When card is in wrong state, log which state it is
  hw/sd: Extract address_in_range() helper, log invalid accesses
  hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

 hw/sd/sd.c                     | 40 ++++++++++++++-------
 tests/qtest/fuzz-sdcard-test.c | 66 ++++++++++++++++++++++++++++++++++
 MAINTAINERS                    |  3 +-
 tests/qtest/meson.build        |  1 +
 4 files changed, 96 insertions(+), 14 deletions(-)
 create mode 100644 tests/qtest/fuzz-sdcard-test.c

-- 
2.31.1

Re: [PATCH 0/3] hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

Posted by Philippe Mathieu-Daudé 4 years, 7 months ago

On 7/2/21 5:58 PM, Philippe Mathieu-Daudé wrote:
> Trivial fix for https://gitlab.com/qemu-project/qemu/-/issues/450
> 
> Missing review: patch #3
> 
> Philippe Mathieu-Daudé (3):
>   hw/sd: When card is in wrong state, log which state it is
>   hw/sd: Extract address_in_range() helper, log invalid accesses
>   hw/sd: Check for valid address range in SEND_WRITE_PROT (CMD30)

Thanks, series applied to sdmmc-next tree.