diff options
| author | Bryan Huntsman <bryanh@codeaurora.org> | 2010-05-04 17:31:32 -0700 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:51:50 -0700 |
| commit | e320afbfb14a96c441d3798113559439a87cc0ba (patch) | |
| tree | f2dcd0bd21c2655e1f597d7db828ce7211897c0f /scripts/checkpatch.pl | |
| parent | 8d19f9538f6f732375344c55039e2d8195429d4a (diff) | |
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>
Diffstat (limited to 'scripts/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 3eda411c042a..c45af9f9da2e 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -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", |