Re: general protection fault in af_alg_free_areq_sgls

2017-12-01 Thread Eric Biggers
On Fri, Dec 01, 2017 at 10:01:01AM -0800, syzbot wrote: > kasan: CONFIG_KASAN_INLINE enabled > kasan: GPF could be caused by NULL-ptr deref or user memory access > general protection fault: [#1] SMP KASAN > Dumping ftrace buffer: >(ftrace buffer empty) > Modules linked in: > CPU: 1 PID: 30

Re: general protection fault in af_alg_free_areq_sgls

2017-11-29 Thread Herbert Xu
On Wed, Nov 29, 2017 at 11:51:09AM -0800, Eric Biggers wrote: > > Herbert, if it's not too late can you fix the subject? It got split into two > lines: Sorry, it's already pushed out with other patches sitting on top of it. Cheers, -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~he

Re: general protection fault in af_alg_free_areq_sgls

2017-11-29 Thread Eric Biggers
On Wed, Nov 29, 2017 at 04:23:53PM +1100, Herbert Xu wrote: > On Tue, Nov 28, 2017 at 01:02:52AM -0800, Eric Biggers wrote: > > > > >From 1a7a7f86f09c50652f1fff75b8d3a32712826b32 Mon Sep 17 00:00:00 2001 > > From: Eric Biggers > > Date: Tue, 28 Nov 2017 00:46:24 -0800 > > Subject: [PATCH] crypto:

Re: general protection fault in af_alg_free_areq_sgls

2017-11-28 Thread Herbert Xu
On Tue, Nov 28, 2017 at 01:02:52AM -0800, Eric Biggers wrote: > > >From 1a7a7f86f09c50652f1fff75b8d3a32712826b32 Mon Sep 17 00:00:00 2001 > From: Eric Biggers > Date: Tue, 28 Nov 2017 00:46:24 -0800 > Subject: [PATCH] crypto: af_alg - fix NULL pointer dereference in > af_alg_free_areq_sgls() >

Re: general protection fault in af_alg_free_areq_sgls

2017-11-28 Thread Eric Biggers
On Tue, Nov 28, 2017 at 10:10:55AM +0100, Stephan Mueller wrote: > > diff --git a/crypto/af_alg.c b/crypto/af_alg.c > > index 358749c38894..415a54ced4d6 100644 > > --- a/crypto/af_alg.c > > +++ b/crypto/af_alg.c > > @@ -672,14 +672,15 @@ void af_alg_free_areq_sgls(struct af_alg_async_req > > *areq)

Re: general protection fault in af_alg_free_areq_sgls

2017-11-28 Thread Stephan Mueller
Am Dienstag, 28. November 2017, 10:02:52 CET schrieb Eric Biggers: Hi Eric, > --- > crypto/af_alg.c | 13 +++-- > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/crypto/af_alg.c b/crypto/af_alg.c > index 358749c38894..415a54ced4d6 100644 > --- a/crypto/af_alg.c > +++ b/

Re: general protection fault in af_alg_free_areq_sgls

2017-11-28 Thread Eric Biggers
On Mon, Nov 27, 2017 at 10:56:47AM -0800, syzbot wrote: > Hello, > > syzkaller hit the following crash on > b0a84f19a5161418d4360cd57603e94ed489915e > git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/master > compiler: gcc (GCC) 7.1.1 20170620 > .config is attached > Raw console o