]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit - include/linux/ptr_ring.h
ptr_ring: wrap back ->producer in __ptr_ring_swap_queue()
authorCong Wang <xiyou.wangcong@gmail.com>
Sun, 30 Dec 2018 20:43:42 +0000 (12:43 -0800)
committerKleber Sacilotto de Souza <kleber.souza@canonical.com>
Wed, 14 Aug 2019 09:18:49 +0000 (11:18 +0200)
commit32872d0e1d2a0e27afaba0b75c2ccb240bb93292
tree338bd70643ff27aacad77ee194ef0bd1f3642d39
parent00e2445622ab2597eb7c4b391a90a1694b0311e0
ptr_ring: wrap back ->producer in __ptr_ring_swap_queue()

BugLink: https://bugs.launchpad.net/bugs/1837257
[ Upstream commit aff6db454599d62191aabc208930e891748e4322 ]

__ptr_ring_swap_queue() tries to move pointers from the old
ring to the new one, but it forgets to check if ->producer
is beyond the new size at the end of the operation. This leads
to an out-of-bound access in __ptr_ring_produce() as reported
by syzbot.

Reported-by: syzbot+8993c0fa96d57c399735@syzkaller.appspotmail.com
Fixes: 5d49de532002 ("ptr_ring: resize support")
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: John Fastabend <john.fastabend@gmail.com>
Cc: Jason Wang <jasowang@redhat.com>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
Signed-off-by: Khalid Elmously <khalid.elmously@canonical.com>
include/linux/ptr_ring.h