Re: [Xen-devel] [PATCH V3 6/6]: PVH:privcmd changes.

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

> @@ -439,6 +490,19 @@ static long privcmd_ioctl(struct file *file,
>       return ret;
>  }
>  
> +static void privcmd_close(struct vm_area_struct *vma)
> +{
> +     struct page **pages = vma ? vma->vm_private_data : NULL;

Can VMA really be NULL?...

> +     int numpgs = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;

...I assume not since you unconditionally dereference it here.

> +     if (!pages || !numpgs || !xen_feature(XENFEAT_auto_translated_physmap))

In the non-xlat case pages will (or should!) be 1 here which will pass
the first clause of the test.

Although the later clauses will catch this I think it would be worth
ordering the checks such that they are each valid, perhaps by pulling
the feature check to the front or by separating the !xlat case from the
other two which are valid iff xlat == True.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel