Re: Missing closing files in linux/scripts/kconfig/confdata.c

From: Masahiro Yamada
Date: Fri Jan 14 2022 - 02:43:15 EST


On Thu, Dec 30, 2021 at 12:11 PM Ryan Cai <ycaibb@xxxxxxxxx> wrote:
>
> Dear Kernel maintainers,
>
> 1. In linux/scripts/kconfig/confdata.c (conf_write_autoconf_cmd), the file opened at Line 946 may not closed when going to Line 981.
> Location: https://github.com/torvalds/linux/blob/e851dfae4371d3c751f1e18e8eb5eba993de1467/scripts/kconfig/confdata.c#L964-L981
>
> 2. In linux/scripts/kconfig/confdata.c (__conf_write_autoconf), the file opened at Line1081 may not closed when going to Line 1095,
> Location: https://github.com/torvalds/linux/blob/e851dfae4371d3c751f1e18e8eb5eba993de1467/scripts/kconfig/confdata.c#L1081-L1095
>
> I think, the fix is inserting fclose before the returning. Should it be a bug? I can send a patch for these.
>
>
> Best,
> Ryan
>
>

Can you send a patch,
or do you want me to do it?


--
Best Regards
Masahiro Yamada