scripts: checkpatch.pl: warn on invalid credentials
@quicinc.com identities are not allowed. Check the "From:" field in the patch, equivalent to the author in the git commit, and the Signed-off-by field. Signed-off-by: Bryan Huntsman <bryanh@codeaurora.org> Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org> [imaund@codeaurora.org: Resolved context conflicts] Signed-off-by: Ian Maund <imaund@codeaurora.org>
This commit is contained in:
parent
eefa722455
commit
1cbfbe862f
|
@ -2278,6 +2278,10 @@ sub process {
|
||||||
"email address '$email' might be better as '$suggested_email$comment'\n" . $herecurr);
|
"email address '$email' might be better as '$suggested_email$comment'\n" . $herecurr);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
if ($line =~ /^\s*signed-off-by:.*(quicinc|qualcomm)\.com/i) {
|
||||||
|
WARN("BAD_SIGN_OFF",
|
||||||
|
"invalid Signed-off-by identity\n" . $line );
|
||||||
|
}
|
||||||
|
|
||||||
# Check for duplicate signatures
|
# Check for duplicate signatures
|
||||||
my $sig_nospace = $line;
|
my $sig_nospace = $line;
|
||||||
|
@ -2331,6 +2335,11 @@ sub process {
|
||||||
"added, moved or deleted file(s), does MAINTAINERS need updating?\n" . $herecurr);
|
"added, moved or deleted file(s), does MAINTAINERS need updating?\n" . $herecurr);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#check the patch for invalid author credentials
|
||||||
|
if ($line =~ /^From:.*(quicinc|qualcomm)\.com/) {
|
||||||
|
WARN("BAD_AUTHOR", "invalid author identity\n" . $line );
|
||||||
|
}
|
||||||
|
|
||||||
# Check for wrappage within a valid hunk of the file
|
# Check for wrappage within a valid hunk of the file
|
||||||
if ($realcnt != 0 && $line !~ m{^(?:\+|-| |\\ No newline|$)}) {
|
if ($realcnt != 0 && $line !~ m{^(?:\+|-| |\\ No newline|$)}) {
|
||||||
ERROR("CORRUPTED_PATCH",
|
ERROR("CORRUPTED_PATCH",
|
||||||
|
|
Loading…
Reference in New Issue