linux-kernel - Re: [PATCH] checkpatch.pl: Don't complain about nominal authors if there isn't one

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]

Message-ID: <Pine.LNX.4.44L0.1909131014410.1466-200000@iolanthe.rowland.org>
Date:   Fri, 13 Sep 2019 10:22:05 -0400 (EDT)
From:   Alan Stern <stern@...land.harvard.edu>
To:     Joe Perches <joe@...ches.com>
cc:     Andy Whitcroft <apw@...onical.com>,
        Kernel development list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] checkpatch.pl: Don't complain about nominal authors if
 there isn't one

On Thu, 12 Sep 2019, Joe Perches wrote:

> On Thu, 2019-09-12 at 16:55 -0400, Alan Stern wrote:
> > checkpatch.pl shouldn't warn about a "Missing Signed-off-by: line by
> > nominal patch author" if there is no nominal patch author.  Without
> > this change, checkpatch always gives me the following warning:
> > 
> >         WARNING: Missing Signed-off-by: line by nominal patch author ''
> 
> When/how does this occur?  Example please.

The patch itself is a good example.  Attached to this email is the
patch file in the form I keep it (from quilt, not git; note that quilt
doesn't do a good job of handling the "---" line so I leave it out and
insert it when submitting the patch).  Try saving the attachment and
running it through checkpatch.pl.  Here's what I get:


$ scripts/checkpatch.pl /tmp/checkpatch-author-fix.patch 
WARNING: Missing Signed-off-by: line by nominal patch author ''

total: 0 errors, 1 warnings, 8 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/tmp/checkpatch-author-fix.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.


Would you like me to resubmit the patch with this example added to the
patch description?

Alan Stern

View attachment "checkpatch-author-fix.patch" of type "TEXT/PLAIN" (944 bytes)