Re: [PATCH 0/4] block/vvfat: Fix leaks and out of bounds writes

[Johannes Schindelin]

Hi Philippe,

On Fri, 30 Apr 2021, Philippe Mathieu-Daudé wrote:

> The first 3 patches are trivial leak fixes, the last
> one is a RFC since I have no clue about this code.
>
> Johannes, you wrote this 18 years ago, do you still
> remember? =)

I do remember writing them, but I remember almost nothing of the details.
The patch series looks good to me (but keep in mind that I am not active
in the QEMU project, and that I never used that driver except for a little
playing around).

Ciao,
Johannes

>
> Philippe Mathieu-Daudé (4):
>   block/vvfat: Fix leak of BDRVVVFATState::qcow_filename
>   block/vvfat: Fix leak of BDRVVVFATState::used_clusters
>   block/vvfat: Fix leak of mapping_t::path
>   block/vvfat: Avoid out of bounds write in create_long_filename()
>
>  block/vvfat.c | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
>
> --
> 2.26.3
>
>
>