i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

kernel/vm: Set CLONEABLE_AREA before cloning areas for transfer.

Browse Source
This commit is contained in:
Augustin Cavalier
2019-08-10 19:45:57 -04:00
parent 40b0fbbb5d
commit 329866d1d0
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/system/kernel/vm/vm.cpp
+8
View File
@@ -5992,6 +5992,11 @@ transfer_area(area_id id, void** _address, uint32 addressSpec, team_id target,
if (info.team != thread_get_current_thread()->team->id) if (info.team != thread_get_current_thread()->team->id)
return B_PERMISSION_DENIED; return B_PERMISSION_DENIED;
// We need to mark the area cloneable so the following operations work.
status = set_area_protection(id, info.protection | B_CLONEABLE_AREA);
if (status != B_OK)
return status;
area_id clonedArea = vm_clone_area(target, info.name, _address, area_id clonedArea = vm_clone_area(target, info.name, _address,
addressSpec, info.protection, REGION_NO_PRIVATE_MAP, id, kernel); addressSpec, info.protection, REGION_NO_PRIVATE_MAP, id, kernel);
if (clonedArea < 0) if (clonedArea < 0)
@@ -6003,6 +6008,9 @@ transfer_area(area_id id, void** _address, uint32 addressSpec, team_id target,
return status; return status;
} }
// Now we can reset the protection to whatever it was before.
set_area_protection(clonedArea, info.protection);
// TODO: The clonedArea is B_SHARED_AREA, which is not really desired. // TODO: The clonedArea is B_SHARED_AREA, which is not really desired.
return clonedArea; return clonedArea;