Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <385f9b15d528a678333b4d008a8dbacc513f2408.camel@gmail.com>
Subject: Re: [PATCH] tty: fix memory leak in vc_deallocate
From:   Pavel Skripkin <paskripkin@gmail.com>
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     jirislaby@kernel.org, linux-kernel@vger.kernel.org,
        syzbot+bcc922b19ccc64240b42@syzkaller.appspotmail.com
Date:   Sun, 28 Mar 2021 11:49:40 +0300
In-Reply-To: <YGBCKBtrirh5GmVP@kroah.com>
References: <20210327214443.21548-1-paskripkin@gmail.com>
         <YGBCKBtrirh5GmVP@kroah.com>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.38.4 
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hi!
On Sun, 2021-03-28 at 10:45 +0200, Greg KH wrote:
> On Sun, Mar 28, 2021 at 12:44:43AM +0300, Pavel Skripkin wrote:
> > syzbot reported memory leak in tty/vt.
> > The problem was in VT_DISALLOCATE ioctl cmd.
> > After allocating unimap with PIO_UNIMAP it wasn't
> > freed via VT_DISALLOCATE, but vc_cons[currcons].d was
> > zeroed.
> > 
> > Signed-off-by: Pavel Skripkin <paskripkin@gmail.com>
> > Reported-by: syzbot+bcc922b19ccc64240b42@syzkaller.appspotmail.com
> > ---
> >  drivers/tty/vt/vt.c | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Is this patch tested by syzbot to fix the problem?
> 
Yeah, it's tested.
https://syzkaller.appspot.com/bug?id=083cb8bd8468537151a57339ae72d505bb5bded0
> thanks,
> 
> greg k-h

-- 
With regards,
Pavel Skripkin