In D32993#746038, @rmacklem wrote:

It now looks good to me.

Do you have a src commit bit?

If not, I can commit it with "Submitted by: yocalebo_gmail.com".

Thanks for the patch, rick

In D32993#745850, @rmacklem wrote:

With the one additional change I just commented inline,
I think that will do it.
(Isn't it fun to work with decades old code and still maintain
all the semantics it carries around. Personally, I live in fear
of mountd bugs because the code can be such a beast to
work with.;-)

Okay, I've tried to fix the style to be consistent with what freeBSD expects. I've removed the call to cr->cr_uid = atoi(name) because it's a no-op and could never be reached with or without my logic changes. Lastly, I've ensured that if strsep_quote returns NULL that we account for that so we don't crash by passing a NULL value to the various getpw* or getgr calls.

Uploading new diff addressing Rick Macklem's review.

In D18958#658726, @freqlabs wrote:

This is still needed but the code has changed so the patch must be adapted.

I don't believe it will need a man page change because the dump_lockstate function already prints a blank for this column when a call back address isn't found.