Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Jun 2019 18:33:06 +0000
From:      bugzilla-noreply@freebsd.org
To:        bugs@FreeBSD.org
Subject:   [Bug 238725] Severe NFS exports(5) -maproot regression for :group definition
Message-ID:  <bug-238725-227-HjNeoeGsYZ@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-238725-227@https.bugs.freebsd.org/bugzilla/>
References:  <bug-238725-227@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238725

--- Comment #2 from Harald Schmalzbauer <bugzilla.freebsd@omnilan.de> ---
Sorry for the bad reproduction info.  The choosen values are nonsense and
inconsistent in the example...
But I guess that doesn't matter much.

Here's the breaking part:

Index: usr.sbin/mountd/mountd.c
===================================================================
--- usr.sbin/mountd/mountd.c    (Revision 349275)
+++ usr.sbin/mountd/mountd.c    (Arbeitskopie)
@@ -2958,8 +2958,8 @@
        /*
         * Get the user's password table entry.
         */
-       names = namelist;
-       name = strsep_quote(&names, ":");
+       names = strsep_quote(&namelist, " \t\n");
+       name = strsep(&names, ":");
        /* Bug?  name could be NULL here */
        if (isdigit(*name) || *name == '-')
                pw = getpwuid(atoi(name));

This reverse-diff restores working state.
I have no idea what the intention was.
At least dlim seems to be unintentionally changed?

Thanks,

-harry

P.S.: Like mentioned, I consider this as severe surprise for useres updating to
11.3 and wondering why NFS doesn't work as before...

-- 
You are receiving this mail because:
You are the assignee for the bug.


Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-238725-227-HjNeoeGsYZ>