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
8d19f9538f
commit
e320afbfb1
1 changed files with 9 additions and 0 deletions
|
@ -2312,6 +2312,10 @@ sub process {
|
|||
"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
|
||||
my $sig_nospace = $line;
|
||||
|
@ -2442,6 +2446,11 @@ sub process {
|
|||
"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
|
||||
if ($realcnt != 0 && $line !~ m{^(?:\+|-| |\\ No newline|$)}) {
|
||||
ERROR("CORRUPTED_PATCH",
|
||||
|
|
Loading…
Add table
Reference in a new issue