Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 25 Apr 2023 07:15:39 +0200
From:   Greg KH <gregkh@linuxfoundation.org>
To:     ktestrobot@126.com, U201911841@hust.edu.cn
Cc:     lidaxian@hust.edu.cn, hust-os-kernel-patches@googlegroups.com,
        dzm91@hust.edu.cn, linux-usb@vger.kernel.org,
        linux-kernel@vger.kernel.org, balbi@kernel.org, s.shtylyov@omp.ru
Subject: Re: [PATCH v2] usb: phy: phy-tahvo: fix memory leak in
 tahvo_usb_probe()
Message-ID: <ZEdh-_Jv02qb6K4n@kroah.com>
References: <644741EB.013E97.00008@m126.mail.126.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <644741EB.013E97.00008@m126.mail.126.com>
Precedence: bulk

On Tue, Apr 25, 2023 at 10:58:51AM +0800, ktestrobot@126.com wrote:
> Hi, Li Yang
> This email is automatically replied by KTestRobot(Beta). Please do not reply to this email.

But I will!

> If you have any questions or suggestions about KTestRobot, please contact ZhongYong <U201911841@hust.edu.cn>

First question, why are you responding from an email that is not allowed
to be responded to and forced to have us manually add an address?
That's not very nice or helpful for us who have to see these messages,
please fix that.

> --- Changed Paths ---
> drivers/usb/phy/phy-tahvo.c
> --- Log Message ---
> Smatch reports:
> drivers/usb/phy/phy-tahvo.c: tahvo_usb_probe()
> warn: missing unwind goto?
> 
> After geting irq, if ret < 0, it will return without error handling to
> free memory.
> Just add error handling to fix this problem.
> 
> --- Test Result ---
> *** CheckPatch	PASS ***
> *** CheckSmatch	PASS ***
> *** ApplyToLinuxNext	PASS ***

Why is only "does the patch apply" matter for this bot?  What is it
supposed to be doing here?  Is it going to run on every single patch to
this mailing list, and if so, how can we turn it off as "it applied!" is
a very very low bar to be testing something for, and we already have a
bot that does this.

thanks,

greg k-h