[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 2/2] s390x: Fix SCLP return code when buffer too small
From: |
Thomas Huth |
Subject: |
Re: [PATCH v1 2/2] s390x: Fix SCLP return code when buffer too small |
Date: |
Thu, 26 Sep 2019 14:26:42 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 26/09/2019 14.20, Thomas Huth wrote:
> On 26/09/2019 13.33, Claudio Imbrenda wrote:
>> Return the correct error code when the SCCB buffer is too small to
>> contain all of the output, for the Read SCP Information and
>> Read CPU Information commands.
>>
>> Signed-off-by: Claudio Imbrenda <address@hidden>
>> Reviewed-by: Jason J. Herne <address@hidden>
>> ---
>> hw/s390x/sclp.c | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
>> index 76feac8..8f7fe1c 100644
>> --- a/hw/s390x/sclp.c
>> +++ b/hw/s390x/sclp.c
>> @@ -68,6 +68,11 @@ static void read_SCP_info(SCLPDevice *sclp, SCCB *sccb)
>>
>> read_info->ibc_val = cpu_to_be32(s390_get_ibc_val());
>>
>> + if (sccb->h.length < (sizeof(ReadInfo) + cpu_count * sizeof(CPUEntry)))
>> {
>
> Don't you need a cpu16_to_cpu() around sccb->h.length?
I meant be16_to_cpu(), obviously.
Thomas
[PATCH v1 2/2] s390x: Fix SCLP return code when buffer too small, Claudio Imbrenda, 2019/09/26
Re: [PATCH v1 0/2] s390x: SCLP error cleanup, David Hildenbrand, 2019/09/27