Re: [PATCH] scripts/spdxcheck: fix file descriptor leak in read_spdxdata()

[Jonathan Corbet <corbet@lwn.net>]

3237174131@qq.com writes:

> Hi all,
>
> This is my first kernel patch. Please let me know if anything is wrong.

Sending the patch as an attachment and the lack of a useful changelog
are both problems; please see
Documentation/process/submitting-patches.rst for information on how to
do this properly.

But, beyond that:

> Signed-off-by: Xia Zhoxin <3237174131@qq.com>
> From: Xia Zhoxin <3237174131@qq.com>
> Subject: [PATCH] scripts/spdxcheck: fix file descriptor leak in read_spdxdata()
> To: linux-kernel@vger.kernel.org
> Cc: tglx@kernel.org, gregkh@linuxfoundation.org
>
> Hi all,
>
> This is my first kernel patch. Please let me know if anything is wrong.
>
> Signed-off-by: Xia Zhoxin <3237174131@qq.com>
> ---
>  scripts/spdxcheck.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/spdxcheck.py b/scripts/spdxcheck.py
> --- a/scripts/spdxcheck.py
> +++ b/scripts/spdxcheck.py
> @@ -72,7 +72,8 @@ def read_spdxdata(repo):
>              exception = None
> -            for l in open(el.path, encoding="utf-8").readlines():
> +            with open(el.path, encoding="utf-8") as f:
> +                for l in f:
>                      if l.startswith('Valid-License-Identifier:'):

open() returns a file object (*not* a file descriptor) that will
immediately go to a reference count of zero once .readlines() completes.
Garbage collection will then clean it up.  So there is no actual leak
here.

Thanks,

jon