Re: [Qemu-devel] [PATCH 03/10] linux-user: init_guest_space: Clean up if we can't initialize the commpage
Le 28/12/2017 à 19:08, Luke Shumaker a écrit : > From: Luke Shumaker> > We'll just exit with an error anyway, so it doesn't really matter, but it > is cleaned up in all of the other places were we error out. > > Signed-off-by: Luke Shumaker > --- > linux-user/elfload.c | 1 + > 1 file changed, 1 insertion(+) Applied to my 'linux-user-for-2.12' branch. Thanks, Laurent
Re: [Qemu-devel] [PATCH 03/10] linux-user: init_guest_space: Clean up if we can't initialize the commpage
On 28 December 2017 at 18:08, Luke Shumakerwrote: > From: Luke Shumaker > > We'll just exit with an error anyway, so it doesn't really matter, but it > is cleaned up in all of the other places were we error out. > > Signed-off-by: Luke Shumaker > --- > linux-user/elfload.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/linux-user/elfload.c b/linux-user/elfload.c > index 453394239c..1a6b660b25 100644 > --- a/linux-user/elfload.c > +++ b/linux-user/elfload.c > @@ -1886,6 +1886,7 @@ unsigned long init_guest_space(unsigned long host_start, > if (valid == 1) { > break; > } else if (valid == -1) { > +munmap((void *)real_start, host_size); > return (unsigned long)-1; > } > /* valid == 0, so try again. */ > -- > 2.15.1 Reviewed-by: Peter Maydell thanks -- PMM
[Qemu-devel] [PATCH 03/10] linux-user: init_guest_space: Clean up if we can't initialize the commpage
From: Luke ShumakerWe'll just exit with an error anyway, so it doesn't really matter, but it is cleaned up in all of the other places were we error out. Signed-off-by: Luke Shumaker --- linux-user/elfload.c | 1 + 1 file changed, 1 insertion(+) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index 453394239c..1a6b660b25 100644 --- a/linux-user/elfload.c +++ b/linux-user/elfload.c @@ -1886,6 +1886,7 @@ unsigned long init_guest_space(unsigned long host_start, if (valid == 1) { break; } else if (valid == -1) { +munmap((void *)real_start, host_size); return (unsigned long)-1; } /* valid == 0, so try again. */ -- 2.15.1