Messages in this thread | | | Subject | Re: [PATCH] decompress_bunzip2: fix sizeof type in start_bunzip | From | Joe Perches <> | Date | Sun, 12 Jul 2020 14:49:44 -0700 |
| |
On Sun, 2020-07-12 at 08:12 -0700, Tom Rix wrote: > On 7/12/20 6:09 AM, H. Peter Anvin wrote: > > On 2020-07-12 05:59, trix@redhat.com wrote: > > > From: Tom Rix <trix@redhat.com> [] > > > So change the type in sizeof to 'unsigned int' > > You must be kidding. > > > > If you want to change it, change it to sizeof(bd->dbuf) instead, but this flag > > is at least in my opinion a total joke. For sizeof(int) != sizeof(unsigned > > int) is beyond bizarre, no matter how stupid the platform. > > Using the actual type is more correct that using a type of the same size.
Sure.
But this hardly matters as this same type conversion from signed to unsigned or the other way round is _everywhere_ in the kernel.
And especially the cc of stable is unnecessary.
| |