xen/blkback: rework connect_ring() to avoid inconsistent xenstore 'ring-page-order...
authorDongli Zhang <dongli.zhang@oracle.com>
Sun, 24 Feb 2019 15:17:27 +0000 (10:17 -0500)
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Sun, 24 Feb 2019 15:17:56 +0000 (10:17 -0500)
commit4a8c31a1c6f526ec96a35e613f2a71e26ffbd7dd
tree78b6973f8ef763634ab1a235adacd896f43e6ba0
parent5b8e432dbb0e20bd8e99033f4a0bfa0d38c0e08e
xen/blkback: rework connect_ring() to avoid inconsistent xenstore 'ring-page-order' set by malicious blkfront

The xenstore 'ring-page-order' is used globally for each blkback queue and
therefore should be read from xenstore only once. However, it is obtained
in read_per_ring_refs() which might be called multiple times during the
initialization of each blkback queue.

If the blkfront is malicious and the 'ring-page-order' is set in different
value by blkfront every time before blkback reads it, this may end up at
the "WARN_ON(i != (XEN_BLKIF_REQS_PER_PAGE * blkif->nr_ring_pages));" in
xen_blkif_disconnect() when frontend is destroyed.

This patch reworks connect_ring() to read xenstore 'ring-page-order' only
once.

Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
drivers/block/xen-blkback/xenbus.c