28.05.2015 14:13, Shannon Zhao wrote: > From: Shannon Zhao <shannon.z...@linaro.org> > > Signed-off-by: Shannon Zhao <zhaoshengl...@huawei.com> > Signed-off-by: Shannon Zhao <shannon.z...@linaro.org> > --- > hw/display/cg3.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/display/cg3.c b/hw/display/cg3.c > index 1e6ff2b..b86e5c0 100644 > --- a/hw/display/cg3.c > +++ b/hw/display/cg3.c > @@ -302,6 +302,7 @@ static void cg3_realizefn(DeviceState *dev, Error **errp) > if (fcode_filename) { > ret = load_image_targphys(fcode_filename, s->prom_addr, > FCODE_MAX_ROM_SIZE); > + g_free(fcode_filename); > if (ret < 0 || ret > FCODE_MAX_ROM_SIZE) { > error_report("cg3: could not load prom '%s'", CG3_ROM_FILE); > }
BTW, here, maybe it is better to rework error message too, to include the actual filename it tried to load, not the base name of it. Is the error fatal? Thanks, /mjt