netdev - Re: [PATCH net,stable] cdc_ncm: fix divide-by-zero caused by invalid wMaxPacketSize

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20190921123026.7f26e537@cakuba.netronome.com>
Date:   Sat, 21 Sep 2019 12:30:26 -0700
From:   Jakub Kicinski <jakub.kicinski@...ronome.com>
To:     Bjørn Mork <bjorn@...k.no>
Cc:     netdev@...r.kernel.org, linux-usb@...r.kernel.org,
        Oliver Neukum <oliver@...kum.org>,
        syzbot+ce366e2b8296e25d84f5@...kaller.appspotmail.com
Subject: Re: [PATCH net,stable] cdc_ncm: fix divide-by-zero caused by
 invalid wMaxPacketSize

On Wed, 18 Sep 2019 14:01:46 +0200, Bjørn Mork wrote:
> Endpoints with zero wMaxPacketSize are not usable for transferring
> data. Ignore such endpoints when looking for valid in, out and
> status pipes, to make the driver more robust against invalid and
> meaningless descriptors.
> 
> The wMaxPacketSize of the out pipe is used as divisor. So this change
> fixes a divide-by-zero bug.
> 
> Reported-by: syzbot+ce366e2b8296e25d84f5@...kaller.appspotmail.com
> Signed-off-by: Bjørn Mork <bjorn@...k.no>

Applied, queued, thank you!