From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net] vhost: validate log when IOTLB is enabled Date: Thu, 29 Mar 2018 16:23:12 -0400 (EDT) Message-ID: <20180329.162312.1243047104828828329.davem@davemloft.net> References: <1522310404-8486-1-git-send-email-jasowang@redhat.com> <20180329154803-mutt-send-email-mst@kernel.org> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: jasowang@redhat.com, kvm@vger.kernel.org, virtualization@lists.linux-foundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: mst@redhat.com Return-path: In-Reply-To: <20180329154803-mutt-send-email-mst@kernel.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: "Michael S. Tsirkin" Date: Thu, 29 Mar 2018 15:48:22 +0300 > On Thu, Mar 29, 2018 at 04:00:04PM +0800, Jason Wang wrote: >> Vq log_base is the userspace address of bitmap which has nothing to do >> with IOTLB. So it needs to be validated unconditionally otherwise we >> may try use 0 as log_base which may lead to pin pages that will lead >> unexpected result (e.g trigger BUG_ON() in set_bit_to_user()). >> >> Fixes: 6b1e6cc7855b0 ("vhost: new device IOTLB API") >> Reported-by: syzbot+6304bf97ef436580fede@syzkaller.appspotmail.com >> Signed-off-by: Jason Wang > > Acked-by: Michael S. Tsirkin > > stable material I guess. Applied and queued up for -stable.