getpwnam resolves local system accounts to AD accounts. This is normally how the lookup is done: getent passwd ADDOMAIN/alice ADDOMAIN/alice:*:100000:100006::/home/ADDOMAIN/Domain Users/alice:/bin/false This should not resolve to an AD user, unless 'winbind use default domain = yes' is set: getent passwd alice ADDOMAIN/alice:*:100000:100006::/home/ADDOMAIN/Domain Users/alice:/bin/false Patch will follow.
Created attachment 14280 [details] Patch for 4.8 cherry-picked from master
Karolin, please add the patch to v4.8. Thanks
(In reply to Andreas Schneider from comment #2) Pushed to autobuild-v4-8-test.
autobuild-v4-8-test fails with: [344(1874)/2266 at 22m9s] samba3.wbinfo_user_info(ad_member:local) UNEXPECTED(failure): samba3.wbinfo_user_info.getpwnam.local.alice(ad_member:local) REASON: Exception: Exception: getent passwd alice return code: 2, expected return code is: 0
Removed from v4-8-test to get the other changes through autobuild.
Created attachment 14405 [details] patch for 4.8
Re-assigning to Karolin for inclusion in 4.8.next.
Pushed to autobuild-v4-8-test.
(In reply to Karolin Seeger from comment #8) Autobuild is failing with the same error again. Re-trying without this patchset.
Created attachment 14444 [details] patch without test for v4.8 This means that $ENVNAME is not set on autobuild and it selects the wrong return code checks. This works totally fine if I run it locally in v4-8-test: [1(0)/1 at 0s] samba3.wbinfo_user_info(ad_member:local) ALL OK (13 tests in 1 testsuites) If the test doesn't work on 4.8 (it works on 4.9 and master), can you please just add the fix without the test? I've attached the fix without the tests.
(In reply to Andreas Schneider from comment #10) Pushed the patch without test. Thanks!
Pushed to v4-8-test. Closing out bug report. Thanks!