fix segfault and naming of runtime opts

[Qemu-devel] [PATCH V3 0/2] fix segfault and naming of runtime opts

Peter Lieven posted 2 patches 7 years, 2 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1485942829-10756-1-git-send-email-pl@kamp.de

Test checkpatch passed

Test docker passed

Test s390x passed

block/nfs.c | 49 +++++++++++++++++++++++++------------------------
1 file changed, 25 insertions(+), 24 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH V3 0/2] fix segfault and naming of runtime opts

Posted by Peter Lieven 7 years, 2 months ago

commit 94d6a7a accidently left the naming of runtime opts and QAPI
scheme inconsistent. Furthermore a NULL pointer dereference resulted
in a segfault when parsing URI parameters.

v1->v2: Patch 1: fixed type in commit msg [Eric]
        Patch 2: noted backwards incompatiblity with 2.8 [Eric]
v2->v3: Patch 2: also change names in nfs_refresh_filename [Max]
                 fix typo in commit msg [Eric]

Peter Lieven (2):
  block/nfs: fix NULL pointer dereference in URI parsing
  block/nfs: fix naming of runtime opts

 block/nfs.c | 49 +++++++++++++++++++++++++------------------------
 1 file changed, 25 insertions(+), 24 deletions(-)

-- 
1.9.1

Re: [Qemu-devel] [PATCH V3 0/2] fix segfault and naming of runtime opts

Posted by Max Reitz 7 years, 2 months ago

On 01.02.2017 10:53, Peter Lieven wrote:
> commit 94d6a7a accidently left the naming of runtime opts and QAPI
> scheme inconsistent. Furthermore a NULL pointer dereference resulted
> in a segfault when parsing URI parameters.
> 
> v1->v2: Patch 1: fixed type in commit msg [Eric]
>         Patch 2: noted backwards incompatiblity with 2.8 [Eric]
> v2->v3: Patch 2: also change names in nfs_refresh_filename [Max]
>                  fix typo in commit msg [Eric]
> 
> Peter Lieven (2):
>   block/nfs: fix NULL pointer dereference in URI parsing
>   block/nfs: fix naming of runtime opts
> 
>  block/nfs.c | 49 +++++++++++++++++++++++++------------------------
>  1 file changed, 25 insertions(+), 24 deletions(-)

Thanks, applied to my block tree (with the spelling fix proposed by Eric):

https://github.com/XanClic/qemu/commits/block

Max