linux-kernel - Re: [PATCH] checkpatch: Add check for common mailing list helper checks

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]

Date:   Fri, 02 Jul 2021 20:27:23 -0700
From:   Joe Perches <joe@...ches.com>
To:     "John 'Warthog9' Hawley (VMware)" <warthog9@...lescrag.net>,
        linux-kernel@...r.kernel.org, Andy Whitcroft <apw@...onical.com>,
        Dwaipayan Ray <dwaipayanray1@...il.com>,
        Lukas Bulwahn <lukas.bulwahn@...il.com>
Subject: Re: [PATCH] checkpatch: Add check for common mailing list helper
 checks

On Fri, 2021-07-02 at 15:37 -0700, John 'Warthog9' Hawley (VMware)
wrote:
> From: John 'Warthog9' Hawley <warthog9@...lescrag.net>
> 
> Mailing lists in an attempt to try and avoid sending certain
> administrative e-mails to the list, will check the first few lines
> (usually ~10) looking for keywords.  If those key words are found it
> shunts the e-mail to the list admin contact instead of potentially
> passing it through to the list.
> 
> This adds a known list of the potential things that are checked for, and
> while it may search deeper into the message (it goes till it believes
> it's into the patch section) than the mailing list software this should
> help give some warning if the wrong word is potentially added in the
> wrong place in the beginning of a patch message.
[]
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
[]
> @@ -2852,6 +2852,40 @@ sub process {
[]
> +# check if this may trip up common mailing list helpers to redirect email to the admin contact
> +		if ($in_commit_log &&
> +		    ($line =~ /\bcancel\b/i ||
> +		    $line =~ /\badd me\b/i ||
> +		    $line =~ /\bdelete me\b/i ||
> +		    $line =~ /\bremove\s+me\b/i ||
> +		    $line =~ /\bchange\b.*\baddress\b/ ||
> +		    $line =~ /\bsubscribe\b/i ||
> +		    $line =~ /^sub\b/i ||
> +		    $line =~ /\bunsubscribe\b/i ||
> +		    $line =~ /^unsub\b/i ||
> +		    $line =~ /^\s*help\s*$/i ||
> +		    $line =~ /^\s*info\s*$/i ||
> +		    $line =~ /^\s*info\s+\S+\s*$/i ||
> +		    $line =~ /^\s*lists\s*$/i ||
> +		    $line =~ /^\s*which\s*$/i ||
> +		    $line =~ /^\s*which\s+\S+\s*$/i ||
> +		    $line =~ /^\s*index\s*$/i ||
> +		    $line =~ /^\s*index\s+\S+\s*$/i ||
> +		    $line =~ /^\s*who\s*$/i ||
> +		    $line =~ /^\s*who\s+\S+\s*$/i ||
> +		    $line =~ /^\s*get\s+\S+\s*$/i ||
> +		    $line =~ /^\s*get\s+\S+\s+\S+\s*$/i ||
> +		    $line =~ /^\s*approve\b/i ||
> +		    $line =~ /^\s*passwd\b/i ||
> +		    $line =~ /^\s*newinfo\b/i ||
> +		    $line =~ /^\s*config\b/i ||
> +		    $line =~ /^\s*newconfig\b/i ||
> +		    $line =~ /^\s*writeconfig\b/i ||
> +		    $line =~ /^\s*mkdigest\b/i)){

Have you checked this list against actual commits?

> +			WARN("MAILING LIST HELPER",

This must use underscores for the spaces

	MAILING_LIST_HELPER