Re: [PATCH] crypto: zip - Memory corruption in zip_clear_stats()

2017-03-24 Thread Herbert Xu
On Fri, Mar 17, 2017 at 11:46:21PM +0300, Dan Carpenter wrote: > There is a typo here. It should be "stats" instead of "state". The > impact is that we clear 224 bytes instead of 80 and we zero out memory > that we shouldn't. > > Fixes: 09ae5d37e093 ("crypto: zip - Add Compression/Decompression

Re: [PATCH] crypto: zip - Memory corruption in zip_clear_stats()

2017-03-20 Thread Mahipal Reddy
On Sat, Mar 18, 2017 at 4:29 PM, Dan Carpenter wrote: > On Sat, Mar 18, 2017 at 11:24:34AM +0100, walter harms wrote: >> >> >> Am 17.03.2017 21:46, schrieb Dan Carpenter: >> > There is a typo here. It should be "stats" instead of "state". The >> > impact is that we

Re: [PATCH] crypto: zip - Memory corruption in zip_clear_stats()

2017-03-18 Thread Dan Carpenter
On Sat, Mar 18, 2017 at 11:24:34AM +0100, walter harms wrote: > > > Am 17.03.2017 21:46, schrieb Dan Carpenter: > > There is a typo here. It should be "stats" instead of "state". The > > impact is that we clear 224 bytes instead of 80 and we zero out memory > > that we shouldn't. > > > >

Re: [PATCH] crypto: zip - Memory corruption in zip_clear_stats()

2017-03-18 Thread walter harms
Am 17.03.2017 21:46, schrieb Dan Carpenter: > There is a typo here. It should be "stats" instead of "state". The > impact is that we clear 224 bytes instead of 80 and we zero out memory > that we shouldn't. > > Fixes: 09ae5d37e093 ("crypto: zip - Add Compression/Decompression statistics") >

[PATCH] crypto: zip - Memory corruption in zip_clear_stats()

2017-03-17 Thread Dan Carpenter
There is a typo here. It should be "stats" instead of "state". The impact is that we clear 224 bytes instead of 80 and we zero out memory that we shouldn't. Fixes: 09ae5d37e093 ("crypto: zip - Add Compression/Decompression statistics") Signed-off-by: Dan Carpenter