Re: divide error in alauda_transport

From: Alan Stern
Date: Wed May 22 2024 - 09:50:06 EST


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@xxxxxxxxxxxxxxxxxxx> 于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
> >