linux-kernel - Re: [PATCH v2 2/4] cdc-acm: fix possible invalid access when processing notification

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: <dee5264a-f4ed-7d9f-abd9-787e23022d31@cogentembedded.com>
Date:   Sun, 19 Mar 2017 12:50:24 +0300
From:   Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:     Tobias Herzog <t-herzog@....de>, oneukum@...e.com
Cc:     gregkh@...uxfoundation.org, linux-usb@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/4] cdc-acm: fix possible invalid access when
 processing notification

On 3/18/2017 9:52 PM, Tobias Herzog wrote:

> Notifications may only be 8 bytes so long. Accessing the 9th and

    s/so//.
    "So long and thanks for all the fish!" :-)

> 10th byte of unimplemented/unknown notifications may be insecure.
> Also check the length of known notifications before accessing anything
> behind the 8th byte.
>
> Signed-off-by: Tobias Herzog <t-herzog@....de>
[...]

MBR, Sergei