hw/nvme/ctrl: Buffer types cleanups

[PATCH-for-7.0 0/2] hw/nvme/ctrl: Buffer types cleanups

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

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211111154552.2263410-1-philmd@redhat.com

hw/nvme/nvme.h |  4 ++--
hw/nvme/ctrl.c | 12 ++++++------
2 files changed, 8 insertions(+), 8 deletions(-)

Expand all Fold all

[PATCH-for-7.0 0/2] hw/nvme/ctrl: Buffer types cleanups

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

Some trivial notes I took while reviewing CVE-2021-3947:
https://lore.kernel.org/qemu-devel/20211111153125.2258176-1-philmd@redhat.com/

Based-on: <20211111153125.2258176-1-philmd@redhat.com>

*** BLURB HERE ***

Philippe Mathieu-Daudé (2):
  hw/nvme/ctrl: Have nvme_addr_write() take const buffer
  hw/nvme/ctrl: Pass buffers as 'void *' types

 hw/nvme/nvme.h |  4 ++--
 hw/nvme/ctrl.c | 12 ++++++------
 2 files changed, 8 insertions(+), 8 deletions(-)

-- 
2.31.1

Re: [PATCH-for-7.0 0/2] hw/nvme/ctrl: Buffer types cleanups

Posted by Klaus Jensen 4 years, 2 months ago

On Nov 11 16:45, Philippe Mathieu-Daudé wrote:
> Some trivial notes I took while reviewing CVE-2021-3947:
> https://lore.kernel.org/qemu-devel/20211111153125.2258176-1-philmd@redhat.com/
> 
> Based-on: <20211111153125.2258176-1-philmd@redhat.com>
> 
> *** BLURB HERE ***
> 
> Philippe Mathieu-Daudé (2):
>   hw/nvme/ctrl: Have nvme_addr_write() take const buffer
>   hw/nvme/ctrl: Pass buffers as 'void *' types
> 
>  hw/nvme/nvme.h |  4 ++--
>  hw/nvme/ctrl.c | 12 ++++++------
>  2 files changed, 8 insertions(+), 8 deletions(-)
> 

Thanks Philippe, LGTM.

Reviewed-by: Klaus Jensen <k.jensen@samsung.com>

Re: [PATCH-for-7.0 0/2] hw/nvme/ctrl: Buffer types cleanups

Posted by Klaus Jensen 4 years ago

On Nov 11 19:09, Klaus Jensen wrote:
> On Nov 11 16:45, Philippe Mathieu-Daudé wrote:
> > Some trivial notes I took while reviewing CVE-2021-3947:
> > https://lore.kernel.org/qemu-devel/20211111153125.2258176-1-philmd@redhat.com/
> > 
> > Based-on: <20211111153125.2258176-1-philmd@redhat.com>
> > 
> > *** BLURB HERE ***
> > 
> > Philippe Mathieu-Daudé (2):
> >   hw/nvme/ctrl: Have nvme_addr_write() take const buffer
> >   hw/nvme/ctrl: Pass buffers as 'void *' types
> > 
> >  hw/nvme/nvme.h |  4 ++--
> >  hw/nvme/ctrl.c | 12 ++++++------
> >  2 files changed, 8 insertions(+), 8 deletions(-)
> > 
> 
> Thanks Philippe, LGTM.
> 
> Reviewed-by: Klaus Jensen <k.jensen@samsung.com>

Applied to nvme-next. Thanks!