Re: [Qemu-devel] [PATCH] curses ui: add missing iconv_close
On Thu, 14 Mar 2019 at 17:27, Samuel Thibault wrote: > > Peter Maydell, le jeu. 14 mars 2019 17:21:54 +, a ecrit: > > On Thu, 14 Mar 2019 at 17:20, Samuel Thibault > > wrote: > > > > > > Signed-off-by: Samuel Thibault > > > --- > > > ui/curses.c | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/ui/curses.c b/ui/curses.c > > > index 3a7e8649f3..1f83a15a1c 100644 > > > --- a/ui/curses.c > > > +++ b/ui/curses.c > > > @@ -646,6 +646,7 @@ static void font_setup(void) > > > } > > > } > > > } > > > +iconv_close(ucs_to_wchar_conv); > > > } > > > > Don't you need to iconv_close() font_conv and > > wchar_to_ucs_conv as well ? > > Right, sorry, since I didn't have coverity access I didn't realize there > were three leaks. I did mention all 3 variables in my email about the issue :-) thanks -- PMM
Re: [Qemu-devel] [PATCH] curses ui: add missing iconv_close
Peter Maydell, le jeu. 14 mars 2019 17:21:54 +, a ecrit: > On Thu, 14 Mar 2019 at 17:20, Samuel Thibault > wrote: > > > > Signed-off-by: Samuel Thibault > > --- > > ui/curses.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/ui/curses.c b/ui/curses.c > > index 3a7e8649f3..1f83a15a1c 100644 > > --- a/ui/curses.c > > +++ b/ui/curses.c > > @@ -646,6 +646,7 @@ static void font_setup(void) > > } > > } > > } > > +iconv_close(ucs_to_wchar_conv); > > } > > Don't you need to iconv_close() font_conv and > wchar_to_ucs_conv as well ? Right, sorry, since I didn't have coverity access I didn't realize there were three leaks. Samuel
Re: [Qemu-devel] [PATCH] curses ui: add missing iconv_close
On Thu, 14 Mar 2019 at 17:20, Samuel Thibault wrote: > > The iconv_t is opened but never closed. > > Spotted by Coverity: CID 1399708 > Spotted by Coverity: CID 1399709 > Spotted by Coverity: CID 1399713 > > Signed-off-by: Samuel Thibault > --- > ui/curses.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/ui/curses.c b/ui/curses.c > index 3a7e8649f3..1f83a15a1c 100644 > --- a/ui/curses.c > +++ b/ui/curses.c > @@ -646,6 +646,7 @@ static void font_setup(void) > } > } > } > +iconv_close(ucs_to_wchar_conv); > } Don't you need to iconv_close() font_conv and wchar_to_ucs_conv as well ? thanks -- PMM
[Qemu-devel] [PATCH] curses ui: add missing iconv_close
The iconv_t is opened but never closed. Spotted by Coverity: CID 1399708 Spotted by Coverity: CID 1399709 Spotted by Coverity: CID 1399713 Signed-off-by: Samuel Thibault --- ui/curses.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ui/curses.c b/ui/curses.c index 3a7e8649f3..1f83a15a1c 100644 --- a/ui/curses.c +++ b/ui/curses.c @@ -646,6 +646,7 @@ static void font_setup(void) } } } +iconv_close(ucs_to_wchar_conv); } static void curses_setup(void) -- 2.20.1