2022-12-09 12:40:09

by Sven Schnelle

Subject: [PATCH v2 0/2] fix out-of-bounds access when specifying invalid console

Hi,

these two patches fix a crash in the tty driver when a user specifies an
invalid console like 'console=tty3000'. The first patch adds a check to
tty_driver_lookup_tty(), the second one prevents that such a console gets
registered in the vt driver.

Changes in v2:
- trim commit message in first patch
- add second patch as suggested by Jiri Slaby

Sven Schnelle (2):
tty: fix out-of-bounds access in tty_driver_lookup_tty()
tty/vt: prevent registration of console with invalid number

drivers/tty/tty_io.c | 8 +++++---
drivers/tty/vt/vt.c | 6 ++++++
2 files changed, 11 insertions(+), 3 deletions(-)

--
2.34.1

2023-01-04 07:57:56

by Sven Schnelle

Subject: Re: [PATCH v2 0/2] fix out-of-bounds access when specifying invalid console

Sven Schnelle <[email protected]> writes:

> Hi,
>
> these two patches fix a crash in the tty driver when a user specifies an
> invalid console like 'console=tty3000'. The first patch adds a check to
> tty_driver_lookup_tty(), the second one prevents that such a console gets
> registered in the vt driver.
>
> Changes in v2:
> - trim commit message in first patch
> - add second patch as suggested by Jiri Slaby
>
> Sven Schnelle (2):
> tty: fix out-of-bounds access in tty_driver_lookup_tty()
> tty/vt: prevent registration of console with invalid number
>
> drivers/tty/tty_io.c | 8 +++++---
> drivers/tty/vt/vt.c | 6 ++++++
> 2 files changed, 11 insertions(+), 3 deletions(-)

Gentle ping... I couldn't find that this was applied anywhere?

Thanks
Sven

2023-01-04 08:30:58

by Greg Kroah-Hartman

Subject: Re: [PATCH v2 0/2] fix out-of-bounds access when specifying invalid console

On Wed, Jan 04, 2023 at 08:20:49AM +0100, Sven Schnelle wrote:
> Sven Schnelle <[email protected]> writes:
>
> > Hi,
> >
> > these two patches fix a crash in the tty driver when a user specifies an
> > invalid console like 'console=tty3000'. The first patch adds a check to
> > tty_driver_lookup_tty(), the second one prevents that such a console gets
> > registered in the vt driver.
> >
> > Changes in v2:
> > - trim commit message in first patch
> > - add second patch as suggested by Jiri Slaby
> >
> > Sven Schnelle (2):
> > tty: fix out-of-bounds access in tty_driver_lookup_tty()
> > tty/vt: prevent registration of console with invalid number
> >
> > drivers/tty/tty_io.c | 8 +++++---
> > drivers/tty/vt/vt.c | 6 ++++++
> > 2 files changed, 11 insertions(+), 3 deletions(-)
>
> Gentle ping... I couldn't find that this was applied anywhere?

It's in my to-review queue, which is about 2000+ patches right now, give
me some time to dig it out. In the meantime, please feel free to review
other pending patches on the list to help out with the workload.

thanks,

greg k-h

2023-01-19 14:30:16

by Greg Kroah-Hartman

Subject: Re: [PATCH v2 0/2] fix out-of-bounds access when specifying invalid console

On Wed, Jan 04, 2023 at 08:20:49AM +0100, Sven Schnelle wrote:
> Sven Schnelle <[email protected]> writes:
>
> > Hi,
> >
> > these two patches fix a crash in the tty driver when a user specifies an
> > invalid console like 'console=tty3000'. The first patch adds a check to
> > tty_driver_lookup_tty(), the second one prevents that such a console gets
> > registered in the vt driver.
> >
> > Changes in v2:
> > - trim commit message in first patch
> > - add second patch as suggested by Jiri Slaby
> >
> > Sven Schnelle (2):
> > tty: fix out-of-bounds access in tty_driver_lookup_tty()
> > tty/vt: prevent registration of console with invalid number
> >
> > drivers/tty/tty_io.c | 8 +++++---
> > drivers/tty/vt/vt.c | 6 ++++++
> > 2 files changed, 11 insertions(+), 3 deletions(-)
>
> Gentle ping... I couldn't find that this was applied anywhere?

Sorry, still digging out of a lot of patches...