Received-SPF: pass (google.com: domain of linux-kernel+bounces-186301-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 22 May 2024 09:49:55 -0400
From: Alan Stern <stern@rowland.harvard.edu>
To: Shichao Lai <phyhac@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
  linux-usb@vger.kernel.org, usb-storage@lists.one-eyed-alien.net,
  linux-kernel@vger.kernel.org, syzkaller@googlegroups.com
Subject: Re: divide error in alauda_transport
Message-ID: <b82e133a-62a3-4f23-9803-1af79b06b8f4@rowland.harvard.edu>
References: <CAEk6kZszgHVYN8r-AsPw9YtGGf459j0uDMYLdOoTixhE3YJCKw@mail.gmail.com>
 <2024052224-motion-perish-82cd@gregkh>
 <CAEk6kZt4qcV0xUxHFRp3DTdZ0f_3fLE0q4e_AWHGyjba=19g9g@mail.gmail.com>
 <CAEk6kZsgEJNSmFFe-tdA+RhM=+Pjm=JvHiNJ4QjZONPXHt-gjw@mail.gmail.com>
 <2024052242-hunter-exception-930c@gregkh>
 <CAEk6kZsLqJ5RbWYMrcbB+LkRq8jzRgTm3F8kq8FKB14y8ST2GQ@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CAEk6kZsLqJ5RbWYMrcbB+LkRq8jzRgTm3F8kq8FKB14y8ST2GQ@mail.gmail.com>

On Wed, May 22, 2024 at 06:01:57PM +0800, Shichao Lai wrote:
> Thanks for your patience. I am a beginner and initially attempted to report
> errors to the community.
> However, I have just discovered that the issue mentioned above has already
> been proposed in the previous three months of the article, and there is an
> ongoing discussion.

To call the discussion "ongoing" is optimistic.  There hasn't been any 
reply to my last post, which was made on March 17, more than two months 
ago.

Maybe you can carry out the tests I described in that post.  That 
would be a big help.

Alan Stern

> I think this post can be closed later.
> 
> https://groups.google.com/g/syzkaller-bugs/c/m8CuxSpLKoQ/m/JfUy1xGnAwAJ
> 
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> 于2024年5月22日周三 16:59写道：
> 
> > On Wed, May 22, 2024 at 04:22:18PM +0800, Shichao Lai wrote:
> > > patch like this:
> > > ```c
> > > while (sectors > 0) {
> > >     if (!uzonesize) {
> > >         return USB_STOR_TRANSPORT_ERROR;
> > >     }
> > >     unsigned int zone = lba / uzonesize; /* integer division */
> > >     unsigned int lba_offset = lba - (zone * uzonesize);
> > >     unsigned int pages;
> > > ```
> >
> > Please submit this properly and we will be glad to review it.
> >
> > thanks,
> >
> > greg k-h
> >