On Fri, Apr 21, 2023 at 02:20:19PM +0100, Jonathan Cameron via wrote: > From: Hao Zeng <zeng...@kylinos.cn> > > Open file descriptor not closed in error paths. Fix by replace > open coded handling of read of whole file into a buffer with > g_file_get_contents() > > Fixes: aba578bdac ("hw/cxl: CDAT Data Object Exchange implementation") > Signed-off-by: Zeng Hao <zeng...@kylinos.cn> > Suggested-by: Philippe Mathieu-Daudé <phi...@linaro.org> > Suggested-by: Peter Maydell <peter.mayd...@linaro.org> > Suggested-by: Jonathan Cameron via <qemu-devel@nongnu.org> > Signed-off-by: Jonathan Cameron <jonathan.came...@huawei.com> >
Reviewed-by: Fan Ni <fan...@samsung.com> > -- > Changes since v5: > - Drop if guard on g_free() as per checkpatch warning. > --- > hw/cxl/cxl-cdat.c | 29 ++++++++--------------------- > 1 file changed, 8 insertions(+), 21 deletions(-) > > diff --git a/hw/cxl/cxl-cdat.c b/hw/cxl/cxl-cdat.c > index 137abd0992..056711d63d 100644 > --- a/hw/cxl/cxl-cdat.c > +++ b/hw/cxl/cxl-cdat.c > @@ -110,29 +110,18 @@ static void ct3_load_cdat(CDATObject *cdat, Error > **errp) > g_autofree CDATEntry *cdat_st = NULL; > uint8_t sum = 0; > int num_ent; > - int i = 0, ent = 1, file_size = 0; > + int i = 0, ent = 1; > + gsize file_size = 0; > CDATSubHeader *hdr; > - FILE *fp = NULL; > + GError *error = NULL; > > /* Read CDAT file and create its cache */ > - fp = fopen(cdat->filename, "r"); > - if (!fp) { > - error_setg(errp, "CDAT: Unable to open file"); > + if (!g_file_get_contents(cdat->filename, (gchar **)&cdat->buf, > + &file_size, &error)) { > + error_setg(errp, "CDAT: File read failed: %s", error->message); > + g_error_free(error); > return; > } > - > - fseek(fp, 0, SEEK_END); > - file_size = ftell(fp); > - fseek(fp, 0, SEEK_SET); > - cdat->buf = g_malloc0(file_size); > - > - if (fread(cdat->buf, file_size, 1, fp) == 0) { > - error_setg(errp, "CDAT: File read failed"); > - return; > - } > - > - fclose(fp); > - > if (file_size < sizeof(CDATTableHeader)) { > error_setg(errp, "CDAT: File too short"); > return; > @@ -218,7 +207,5 @@ void cxl_doe_cdat_release(CXLComponentState *cxl_cstate) > cdat->free_cdat_table(cdat->built_buf, cdat->built_buf_len, > cdat->private); > } > - if (cdat->buf) { > - free(cdat->buf); > - } > + g_free(cdat->buf); > } > -- > 2.37.2 > > >