>>> On 02.05.17 at 16:13, <rcojoc...@bitdefender.com> wrote:
> So you would prefer something like this?
Not exactly:
> --- a/xen/common/hvm/save.c
> +++ b/xen/common/hvm/save.c
> @@ -113,6 +113,10 @@ int hvm_save_one(struct domain *d, uint16_t
> typecode, uint16_t instance,
> const struct hvm_save_descriptor *desc;
>
> rv = -ENOENT;
> +
> + if ( !ctxt.cur )
if ( ctxt.cur < sizeof(*desc) )
> + goto out;
> +
> for ( off = 0; off < (ctxt.cur - sizeof(*desc)); off +=
> desc->length )
> {
> desc = (void *)(ctxt.data + off);
> @@ -132,6 +136,7 @@ int hvm_save_one(struct domain *d, uint16_t
> typecode, uint16_t instance,
> }
> }
>
> +out:
And this properly indented.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
