linux-kernel - Re: [34-longterm 006/247] USB: serial: handle Data Carrier Detect changes

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]

Message-ID: <4E0493E0.9080701@windriver.com>
Date:	Fri, 24 Jun 2011 09:40:48 -0400
From:	Paul Gortmaker <paul.gortmaker@...driver.com>
To:	Libor Pechacek <lpechacek@...e.cz>
CC:	<stable@...nel.org>, <linux-kernel@...r.kernel.org>,
	<stable-review@...nel.org>, Greg Kroah-Hartman <gregkh@...e.de>
Subject: Re: [34-longterm 006/247] USB: serial: handle Data Carrier Detect
 changes

On 11-06-24 08:13 AM, Libor Pechacek wrote:
> Hi Paul,
> 
>> @@ -726,6 +726,10 @@ static void spcp8x5_read_bulk_callback(struct urb *urb)
>>  		/* overrun is special, not associated with a char */
>>  		if (status & UART_OVERRUN_ERROR)
>>  			tty_insert_flip_char(tty, 0, TTY_OVERRUN);
>> +		if (status & UART_DCD)
>> +			usb_serial_handle_dcd_change(port, tty,
>> +				   priv->line_status & MSR_STATUS_LINE_DCD);
>> +
>>  		tty_insert_flip_string_fixed_flag(tty, data, tty_flag,
>>  							urb->actual_length);
>>  		tty_flip_buffer_push(tty);
> 
> The above code is mistakenly placed inside the "if (tty && urb->actual_length)"
> block.  Should be right after it.

Thanks -- that is one of those cases where what seems like the
"obvious choice" just so happens to also be the wrong one.  :(

In any case, I appreciate the review, it is now fixed.

Paul.

> 
> For kernels v2.6.31 - v2.6.34 use the following patch
> http://git.kernel.org/?p=linux/kernel/git/longterm/linux-2.6.32.y.git;a=commitdiff;h=6fe7015007299e14f5ea212a503bcba5958c7086
> 
> For kernels v2.6.35 - v2.6.37 use
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.37.y.git;a=commitdiff;h=2631217445b84e5d31341eaaf6e7951bf8907f52
> 
> Libor
--
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/