[v3,0/4] mirror: Do not dereference invalid pointers

Message

Max Reitz Oct. 14, 2019, 3:39 p.m. UTC

Hi,

v2’s cover letter should explain everything:

https://lists.nongnu.org/archive/html/qemu-block/2019-09/msg01079.html


v3:
- Patch 2: Use input visitor as proposed by Vladimir

git-backport-diff against v2:

Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/4:[----] [--] 'mirror: Do not dereference invalid pointers'
002/4:[0041] [FC] 'blkdebug: Allow taking/unsharing permissions'
003/4:[----] [--] 'iotests: Add @error to wait_until_completed'
004/4:[----] [--] 'iotests: Add test for failing mirror complete'


Max Reitz (4):
  mirror: Do not dereference invalid pointers
  blkdebug: Allow taking/unsharing permissions
  iotests: Add @error to wait_until_completed
  iotests: Add test for failing mirror complete

 qapi/block-core.json          | 14 +++++-
 block/blkdebug.c              | 91 ++++++++++++++++++++++++++++++++++-
 block/mirror.c                | 13 +++--
 tests/qemu-iotests/041        | 44 +++++++++++++++++
 tests/qemu-iotests/041.out    |  4 +-
 tests/qemu-iotests/iotests.py | 18 ++++---
 6 files changed, 170 insertions(+), 14 deletions(-)

Comments

Max Reitz Oct. 28, 2019, 10:50 a.m. UTC | #1

On 14.10.19 17:39, Max Reitz wrote:
> Hi,
> 
> v2’s cover letter should explain everything:
> 
> https://lists.nongnu.org/archive/html/qemu-block/2019-09/msg01079.html
> 
> 
> v3:
> - Patch 2: Use input visitor as proposed by Vladimir
> 
> git-backport-diff against v2:
> 
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively
> 
> 001/4:[----] [--] 'mirror: Do not dereference invalid pointers'
> 002/4:[0041] [FC] 'blkdebug: Allow taking/unsharing permissions'
> 003/4:[----] [--] 'iotests: Add @error to wait_until_completed'
> 004/4:[----] [--] 'iotests: Add test for failing mirror complete'
> 
> 
> Max Reitz (4):
>   mirror: Do not dereference invalid pointers
>   blkdebug: Allow taking/unsharing permissions
>   iotests: Add @error to wait_until_completed
>   iotests: Add test for failing mirror complete

I’ve applied patch 1 to my block branch, and I’ll send a for-next series
(for-5.0, I suppose?) for the rest.

Max