Am 28.02.2012 10:33, schrieb Stefan Hajnoczi:
> On Mon, Feb 27, 2012 at 4:27 PM, Kevin Wolf <kw...@redhat.com> wrote:
>> +        if (ext.len > 65536) {
>> +            error_report("Header extension larger than 64k - this looks 
>> wrong");
>> +            return -ENOTSUP;
>> +        }
> 
> This is an implementation limit and not in the spec, but I think it's
> reasonable.
> 
> Reviewed-by: Stefan Hajnoczi <stefa...@linux.vnet.ibm.com>

Hm, actually, now that I look at this patch again, I think there's a
much better error condition that even matches the spec:

    if (offset + ext.len > end_offset)

I'll send a changed version of the patch.

Kevin

Reply via email to