Allow quoted user/group names in exports file

[kithrup_mac.com]

Summary

We've run into cases where (usually due to Windows) group names have spaces in them. mountd does not currently allow quoting or escaping of strings; this change adds support (using a strsep-like function) for quoting them. (e.g., -maproot=nobody:"System Daemon")

Test Plan

First with an existing /etc/exports file.
Second with quoting a user and/or group name in the existing /etc/exports file.
Third using a group name with a space (quoting it as necessary). (Requires having such a group, of course. Directory services comes into play here.)

[delphij]

usr.sbin/mountd/mountd.c

293

Perhaps use a more meaningful name for 'retval' (maybe 'token' or something like that)?

325

We don't issue an error here, but the code could behave mysteriously when a \ is at the end of string and suggests that we may want to tell the user something, like "\ is not balanced at string end". The current code structure doesn't seem to provide a good way of implementing it and we may end up with a refactor eventually.

344

My OCD: Personally I would use '\0' instead of 0 here, they have the same effect but '\0' is a char.

2904

I think we should log the event and return?

3220

I think this is still needed, no? (Otherwise a SIGTERM won't terminate mountd, code around "Expand svc_run() here so that we can call get_exportlist()").

[kithrup_mac.com]

Incorporate some of Xin's feedback.

[alfred]

sys/vm/vm_page.c

2632 ↗

(On Diff #10405)

Why are there changes to vm system for this patch? Can this please be removed?