linux-kernel - Re: The robot is broken (was: Re: [PATCH 1/3] USB: serial: return errors from break handling)

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: <ZH1lAtO0nG801FrZ@rli9-mobl>
Date:   Mon, 5 Jun 2023 12:30:58 +0800
From:   Philip Li <philip.li@...el.com>
To:     Johan Hovold <johan@...nel.org>
CC:     kernel test robot <lkp@...el.com>, <oe-kbuild@...ts.linux.dev>,
        "Dan Carpenter" <error27@...il.com>, <linux-kernel@...r.kernel.org>
Subject: Re: The robot is broken (was: Re: [PATCH 1/3] USB: serial: return
 errors from break handling)

On Sun, Jun 04, 2023 at 02:41:27PM +0200, Johan Hovold wrote:
> On Sat, Jun 03, 2023 at 10:40:42AM +0800, kernel test robot wrote:
> > BCC: lkp@...el.com
> > CC: oe-kbuild-all@...ts.linux.dev
> > In-Reply-To: <20230602124642.19076-2-johan@...nel.org>
> > References: <20230602124642.19076-2-johan@...nel.org>
> > TO: Johan Hovold <johan@...nel.org>
> > TO: Johan Hovold <johan@...nel.org>
> > CC: "Greg Kroah-Hartman" <gregkh@...uxfoundation.org>
> > CC: linux-usb@...r.kernel.org
> > CC: linux-kernel@...r.kernel.org
> > CC: Corey Minyard <minyard@....org>
> > 
> > Hi Johan,
> > 
> > kernel test robot noticed the following build warnings:
> 
> > New smatch warnings:
> > drivers/usb/serial/io_edgeport.c:1601 edge_break() error: uninitialized symbol 'status'.
> 
> Thanks for the report. Now fixed up in v2:
> 
> 	https://lore.kernel.org/lkml/20230604123505.4661-1-johan@kernel.org
> 
> The robot currently appears to be broken though as this report was never
> sent to me or the relevant mailing lists (and I just happened to see it
> by chance).

Thanks a lot Johan for checking this. For now, the bot has 2 similar mailing lists,
one is oe-kbuild-all@...ts.linux.dev, which contains the high confidence report that
is sent directly to author and related CC. Another one is this oe-kbuild@...ts.linux.dev,
that we will send low confidence issues here and they will only be really reported
out after manual check. So we can reduce false positives.

But sometimes, we may miss some reports of real issues, sorry about this.

> 
> Looks like there's a stray newline in the output of your scripts which
> causes the mail headers above to be included in the body instead.

Thanks, yes, this is by design, when the report is confirmed, it helps to send
out the report to actual users by removing this extra line.

> 
> Johan
>