>>> On 01.12.17 at 22:38, <sstabell...@kernel.org> wrote: > On Thu, 30 Nov 2017, Jan Beulich wrote: >> Jann validly points out that with a caller bogusly requesting a zero- >> element batch with non-zero high command bits (the ones used for >> continuation encoding), the assertion right before the call to >> hypercall_create_continuation() would trigger. A similar situation would >> arise afaict for non-empty batches with op and/or length zero in every >> element. >> >> While we want the former to succeed (as we do elsewhere for similar >> no-op requests), the latter can clearly be converted to an error, as >> this is a state that can't be the result of a prior operation. >> >> Take the opportunity and also correct the order of argument checks: >> We shouldn't accept zero-length elements with unknown bits set in "op". >> Also constify cache_flush()'s first parameter. >> >> Reported-by: Jann Horn <ja...@google.com> >> Signed-off-by: Jan Beulich <jbeul...@suse.com> > > Acked-by: Stefano Stabellini <sstabell...@kernel.org>
Thanks. Since this and the other patch mainly affect ARM, I'd like to have your opinion please regarding their backporting. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
