linux-kernel - Re: [PATCH 4/5] USB : serial : invoke dcd

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 9 Sep 2013 09:36:48 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Paul Chavent <Paul.Chavent@...ra.fr>
Cc:	linux-usb@...r.kernel.org, jhovold@...il.com,
	fschaefer.oss@...glemail.com, jslaby@...e.cz, max@...e.de,
	giometti@...eenne.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/5] USB : serial : invoke dcd_change ldisc's handler.

On Mon, Sep 09, 2013 at 06:01:19PM +0200, Paul Chavent wrote:
> In order to have the PPS line discipline to work with usb devices.
> 
> Signed-off-by: Paul Chavent <paul.chavent@...ra.fr>
> ---
>  drivers/usb/serial/generic.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
> index 91f0592..a18a086 100644
> --- a/drivers/usb/serial/generic.c
> +++ b/drivers/usb/serial/generic.c
> @@ -567,6 +567,13 @@ void usb_serial_handle_dcd_change(struct usb_serial_port *usb_port,
>  {
>  	struct tty_port *port = &usb_port->port;
>  	struct tty_struct *tty = tty_port_tty_get(port);
> +	struct tty_ldisc *ld = tty ? tty_ldisc_ref(tty) : NULL;
> +
> +	if (ld) {

Minor nit, I hate the ? : mode of C, and as you are doing another if
statement right after it, it's pretty redundant here.  So can you just
merge them together into one if chain?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/