On Wed, 20 May 2026 11:01:26 +0800 lihongtao <[email protected]> wrote:
> From: Hongtao Lee <[email protected]> > > If data calloc failed, free the buf before return. > Thanks! Let me pick it. > Fixes: 950313ebf79c ("tools: bootconfig: Add bootconfig command") > Signed-off-by: Hongtao Lee <[email protected]> > --- > V1 -> V2: Change Email Signed name > tools/bootconfig/main.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/tools/bootconfig/main.c b/tools/bootconfig/main.c > index 643f707b8f1d..ddabde20585f 100644 > --- a/tools/bootconfig/main.c > +++ b/tools/bootconfig/main.c > @@ -390,8 +390,10 @@ static int apply_xbc(const char *path, const char > *xbc_path) > > /* Backup the bootconfig data */ > data = calloc(size + BOOTCONFIG_ALIGN + BOOTCONFIG_FOOTER_SIZE, 1); > - if (!data) > + if (!data) { > + free(buf); > return -ENOMEM; > + } > memcpy(data, buf, size); > > /* Check the data format */ > -- > 2.25.1 > -- Masami Hiramatsu (Google) <[email protected]>
