Re: [PATCH-for-6.2] hw/nvme/ctrl: Fix buffer overrun (CVE-2021-3947)

2021-11-11 Thread Klaus Jensen
On Nov 11 19:46, Philippe Mathieu-Daudé wrote: > On 11/11/21 19:08, Klaus Jensen wrote: > > On Nov 11 16:31, Philippe Mathieu-Daudé wrote: > >> Both 'buf_len' and 'off' arguments are under guest control. > >> Since nvme_c2h() doesn't check out of boundary access, the > >> caller must check for even

Re: [PATCH-for-6.2] hw/nvme/ctrl: Fix buffer overrun (CVE-2021-3947)

2021-11-11 Thread Philippe Mathieu-Daudé
On 11/11/21 19:08, Klaus Jensen wrote: > On Nov 11 16:31, Philippe Mathieu-Daudé wrote: >> Both 'buf_len' and 'off' arguments are under guest control. >> Since nvme_c2h() doesn't check out of boundary access, the >> caller must check for eventual buffer overrun on 'trans_len'. >> >> Cc: qemu-sta...

Re: [PATCH-for-6.2] hw/nvme/ctrl: Fix buffer overrun (CVE-2021-3947)

2021-11-11 Thread Klaus Jensen
On Nov 11 16:31, Philippe Mathieu-Daudé wrote: > Both 'buf_len' and 'off' arguments are under guest control. > Since nvme_c2h() doesn't check out of boundary access, the > caller must check for eventual buffer overrun on 'trans_len'. > > Cc: qemu-sta...@nongnu.org > Reported-by: Qiuhao Li > Fixes

[PATCH-for-6.2] hw/nvme/ctrl: Fix buffer overrun (CVE-2021-3947)

2021-11-11 Thread Philippe Mathieu-Daudé
Both 'buf_len' and 'off' arguments are under guest control. Since nvme_c2h() doesn't check out of boundary access, the caller must check for eventual buffer overrun on 'trans_len'. Cc: qemu-sta...@nongnu.org Reported-by: Qiuhao Li Fixes: f432fdfa121 ("support changed namespace asynchronous event"