d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

xen/gndev: use UOFF_TO_IDX instead of OFF_TO_IDX

Browse Source 
The Xen grant table device treats the mmap offset parameter as an unsigned
type, and as so it must use the newly introduced UOFF_TO_IDX.

Sponsored by:   Citrix Systems R&D
MFC after:      2 weeks
X-MFC-with:     r313690
This commit is contained in:
royger 2017-02-23 13:14:28 +00:00
parent deaac43580
commit 3945d6ddf9
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sys/dev/xen/gntdev/gntdev.c
View File

@ -796,8 +796,8 @@ gntdev_gmap_pg_fault(vm_object_t object, vm_ooffset_t offset, int prot,
relative_offset = offset - gmap->file_index;
pidx = OFF_TO_IDX(offset);
ridx = OFF_TO_IDX(relative_offset);
pidx = UOFF_TO_IDX(offset);
ridx = UOFF_TO_IDX(relative_offset);
if (ridx >= gmap->count ||
gmap->grant_map_ops[ridx].status != GNTST_okay)
return (VM_PAGER_FAIL);
@ -1067,7 +1067,7 @@ mmap_gref(struct per_user_data *priv_user, struct gntdev_gref *gref_start,
break;
vm_page_insert(gref->page, mem_obj,
OFF_TO_IDX(gref->file_index));
UOFF_TO_IDX(gref->file_index));
count--;
}
@ -1207,7 +1207,7 @@ gntdev_mmap_single(struct cdev *cdev, vm_ooffset_t *offset, vm_size_t size,
if (error != 0)
return (EINVAL);
count = OFF_TO_IDX(size);
count = UOFF_TO_IDX(size);
gref_start = gntdev_find_grefs(priv_user, *offset, count);
if (gref_start) {