userns: bump idmap limits, fixes & tweaks
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 patches 23 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | userns: Don't special case a count of 0kernel/ | +3 | 0 replies | |
| 2/5 | userns: Simplify the user and group mapping functionskernel/ | +58 | 0 replies | |
| 3/5 | userns: Don't read extents twice in m_startkernel/ | +4 | 18 replies | |
| 4/5 | userns: Make map_id_down a wrapper for map_id_range_downkernel/ | +1 | 0 replies | |
| 5/5 | userns: Simplify insert_extentkernel/ | +8 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/5] userns: bump idmap limits, fixes & tweaks
Christian I have looked through your code and I have found one real issue and of things I want to twak
The real issue is reading nr_extents multiple times when reading a map. That can introduce races that will allow walking past the end of the array, if the first read is 0 but the second read is > 5.
I have also found a couple of tweaks that look like they are worth implementing.
As all of these are very small and very straight forward I have tested these and applied them all to my for-next branch
Eric W. Biederman (5):
userns: Don't special case a count of 0
userns: Simplify the user and group mapping functions
userns: Don't read extents twice in m_start
userns: Make map_id_down a wrapper for map_id_range_down
userns: Simplify insert_extent
kernel/user_namespace.c | 159 ++++++++++++++++-------------------------------- 1 file changed, 51 insertions(+), 108 deletions(-)