linux-kernel - Re: [PATCH v2] kbuild: uapi: Strip comments before size type check

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <mvm347wjj90.fsf@linux-m68k.org>
Date: Mon, 06 Oct 2025 14:40:43 +0200
From: Andreas Schwab <schwab@...ux-m68k.org>
To: Geert Uytterhoeven <geert@...ux-m68k.org>
Cc: Nathan Chancellor <nathan@...nel.org>,  Nicolas Schier
 <nicolas.schier@...ux.dev>,  Thomas Weißschuh
 <thomas.weissschuh@...utronix.de>,  linux-kbuild@...r.kernel.org,
  linux-m68k@...r.kernel.org,  linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] kbuild: uapi: Strip comments before size type check

On Okt 06 2025, Geert Uytterhoeven wrote:

> diff --git a/usr/include/headers_check.pl b/usr/include/headers_check.pl
> index 21c2fb9520e6af2d..16c238aadfebb061 100755
> --- a/usr/include/headers_check.pl
> +++ b/usr/include/headers_check.pl
> @@ -155,6 +155,8 @@ sub check_sizetypes
>  	if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
>  		check_include_typesh($included);
>  	}
> +	# strip comments (single-line only)
> +	$line =~ s@\/\*.*?\*\/@@;

I don't think you need to quote the forward slashes in the regexp.

-- 
Andreas Schwab, schwab@...ux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."