Hello Alistair, +-- On Tue, 7 Feb 2017, Alistair Francis wrote --+ | As it doesn't look like we have ever supported an infinite transfer | and we don't want infinite loops occurring I think we should add an | unsupported print statement here saying that we don't support infinite | transfers and then I'm ok with this patch.
Okay, will do. | > - MASKED_WRITE(s->trnmod, mask, value); | > + MASKED_WRITE(s->trnmod, mask, value & 0x0037); | | What is this change doing? It is possible to set 's->trnmod' to an invalid value(> 6 bits). Above mask(0x0037) would limit 'value' to the 6 bits mentioned in the spec. Thank you. -- Prasad J Pandit / Red Hat Product Security Team 47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F