Re: [patch v3] checkpatch: Signature format verification

[Joe Perches]

On Mon, 2011-05-23 at 23:07 +0530, anish singh wrote:
> On Mon, May 23, 2011 at 10:48 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Mon, 2011-05-23 at 22:36 +0530, anish singh wrote:
> > >  if ($1 !~ /[\sa-zA-Z\"]*\s<.*>/i) {
> > >  WARN("Space required b/w Full Name & Mail-id:\n" .
> > >       $herecurr);
> > > Quote has been taken care too now.
> > If you're going to try to validate email addresses,
> > please do it reasonably correctly or not at all.
> > At a minimum you need to add digits, single quote,
> > period, and comma.
> I think there is a confusion.Let me state it once again.
> This patch is used to check space between "signs" & name/ name &
> mail-id.

I'm not confused.

You are only checking names with this form (without quotes)
"First Last <fl@xxxxxxxxxx>", now quoted as well.

Names can have many forms.

8 bit chars, commas, quotes, apostrophes, all sorts of things.

for instance:

$ git log v2.6.36.. | grep -P "by:.*" | \
	grep -vP "by:[\sa-zA-Z\"]*\s<.*>" | \
	sort | uniq -c | sort -rn | head -20
   3421     Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
   3137     Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx>
   2232     Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
    386     Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@xxxxxxxxx>
    292     Signed-off-by: Gustavo F. Padovan <padovan@xxxxxxxxxxxxxx>
    257     Signed-off-by: Uwe Kleine-KÃnig <u.kleine-koenig@xxxxxxxxxxxxxx>
    223     Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx>
    193     Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxx>
    167     Signed-off-by: "Theodore Ts'o" <tytso@xxxxxxx>
    166     Signed-off-by: H. Peter Anvin <hpa@xxxxxxxxxxxxxxx>
    159     Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>
    126     Signed-off-by: Jean-FranÃois Moine <moinejf@xxxxxxx>
    121     Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
    107     Signed-off-by: MichaÅ MirosÅaw <mirq-linux@xxxxxxxxxxxx>
    106     Acked-by: David S. Miller <davem@xxxxxxxxxxxxx>
    103     Signed-off-by: RafaÅ MiÅecki <zajec5@xxxxxxxxx>
     76     Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx>
     69     Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
     66     Signed-off-by: Stephen M. Cameron <scameron@xxxxxxxxxxxxxxxxxx>
     63     Signed-off-by: Justin P. Mattock <justinmattock@xxxxxxxxx>

I still think what I suggested after your first patch
attempt is simple and appropriate.

our $Valid_Signatures "(?:Signed-off-by:|Reviewed-by:|Acked-by:)"
...
        if ($line =~ /^(\s*)($ValidSignatures)(\s*)(.*)$/i) {
                my $space_before = $1;
                my $sign_off = $2;
                my $space_after = $3;
                my $email = $4;
                if (defined $space_before && $space_before ne "") {
                        warning (no space before...)
                }
                if ($sign_off !~ /$Valid_Signature/) {
                        warning (signature case...)
                }
                if (!defined $space_after || $space_after ne " ") {
                        warning (need only one space after colon...)
                }
                if (!validate_email($4)) {
                        warning (bad email...)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/