Re: [PATCH] staging: dgnc: Fix a NULL pointer dereference
On Tue, May 03, 2016 at 10:56:09PM -0700, Greg KH wrote: > On Wed, May 04, 2016 at 02:41:22PM +0900, Daeseok Youn wrote: > > The error handling for print_drive after calling tty_alloc_driver() > > was needed. But there was error handling but too late for this. > > Error handling code moved after tty_alloc_driver() call. > > > > Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of > > kcalloc") > > Reported-by: Dan Carpenter> > Signed-off-by: Daeseok Youn > > --- > > drivers/staging/dgnc/dgnc_tty.c | 10 +- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > Patch doesn't apply at all, does it need something else to be merged > first? Oh nevermind, I tried applying it to the wrong git tree, my fault... I need some sleep... greg k-h
Re: [PATCH] staging: dgnc: Fix a NULL pointer dereference
On Tue, May 03, 2016 at 10:56:09PM -0700, Greg KH wrote: > On Wed, May 04, 2016 at 02:41:22PM +0900, Daeseok Youn wrote: > > The error handling for print_drive after calling tty_alloc_driver() > > was needed. But there was error handling but too late for this. > > Error handling code moved after tty_alloc_driver() call. > > > > Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of > > kcalloc") > > Reported-by: Dan Carpenter > > Signed-off-by: Daeseok Youn > > --- > > drivers/staging/dgnc/dgnc_tty.c | 10 +- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > Patch doesn't apply at all, does it need something else to be merged > first? Oh nevermind, I tried applying it to the wrong git tree, my fault... I need some sleep... greg k-h
Re: [PATCH] staging: dgnc: Fix a NULL pointer dereference
On Wed, May 04, 2016 at 02:41:22PM +0900, Daeseok Youn wrote: > The error handling for print_drive after calling tty_alloc_driver() > was needed. But there was error handling but too late for this. > Error handling code moved after tty_alloc_driver() call. > > Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of kcalloc") > Reported-by: Dan Carpenter> Signed-off-by: Daeseok Youn > --- > drivers/staging/dgnc/dgnc_tty.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) Patch doesn't apply at all, does it need something else to be merged first? confused, greg k-h
Re: [PATCH] staging: dgnc: Fix a NULL pointer dereference
On Wed, May 04, 2016 at 02:41:22PM +0900, Daeseok Youn wrote: > The error handling for print_drive after calling tty_alloc_driver() > was needed. But there was error handling but too late for this. > Error handling code moved after tty_alloc_driver() call. > > Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of kcalloc") > Reported-by: Dan Carpenter > Signed-off-by: Daeseok Youn > --- > drivers/staging/dgnc/dgnc_tty.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) Patch doesn't apply at all, does it need something else to be merged first? confused, greg k-h
[PATCH] staging: dgnc: Fix a NULL pointer dereference
The error handling for print_drive after calling tty_alloc_driver() was needed. But there was error handling but too late for this. Error handling code moved after tty_alloc_driver() call. Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of kcalloc") Reported-by: Dan CarpenterSigned-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_tty.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 2cecdb0..392d532 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -224,6 +224,11 @@ int dgnc_tty_register(struct dgnc_board *brd) TTY_DRIVER_DYNAMIC_DEV | TTY_DRIVER_HARDWARE_BREAK); + if (IS_ERR(brd->print_driver)) { + rc = PTR_ERR(brd->print_driver); + goto unregister_serial_driver; + } + snprintf(brd->print_name, MAXTTYNAMELEN, "pr_dgnc_%d_", brd->boardnum); brd->print_driver->name = brd->print_name; @@ -235,11 +240,6 @@ int dgnc_tty_register(struct dgnc_board *brd) brd->print_driver->init_termios = DgncDefaultTermios; brd->print_driver->driver_name = DRVSTR; - if (IS_ERR(brd->print_driver)) { - rc = PTR_ERR(brd->print_driver); - goto unregister_serial_driver; - } - /* * Entry points for driver. Called by the kernel from * tty_io.c and n_tty.c. -- 2.8.2
[PATCH] staging: dgnc: Fix a NULL pointer dereference
The error handling for print_drive after calling tty_alloc_driver() was needed. But there was error handling but too late for this. Error handling code moved after tty_alloc_driver() call. Fixes: 60b3109e5e2d ("staging: dgnc: use tty_alloc_driver instead of kcalloc") Reported-by: Dan Carpenter Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_tty.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 2cecdb0..392d532 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -224,6 +224,11 @@ int dgnc_tty_register(struct dgnc_board *brd) TTY_DRIVER_DYNAMIC_DEV | TTY_DRIVER_HARDWARE_BREAK); + if (IS_ERR(brd->print_driver)) { + rc = PTR_ERR(brd->print_driver); + goto unregister_serial_driver; + } + snprintf(brd->print_name, MAXTTYNAMELEN, "pr_dgnc_%d_", brd->boardnum); brd->print_driver->name = brd->print_name; @@ -235,11 +240,6 @@ int dgnc_tty_register(struct dgnc_board *brd) brd->print_driver->init_termios = DgncDefaultTermios; brd->print_driver->driver_name = DRVSTR; - if (IS_ERR(brd->print_driver)) { - rc = PTR_ERR(brd->print_driver); - goto unregister_serial_driver; - } - /* * Entry points for driver. Called by the kernel from * tty_io.c and n_tty.c. -- 2.8.2