This adds winbind-generated groups showing up in 'getent group'. It is not
very efficient though, it only does one group at a time. Needs improving, but
the structures are not particularly easy to set up, so check in the basically
working part for others to review.

I'm close to saying that I would like to remove aliases from general group
mapping. These can not be reflected correctly in /etc/group, winbind could do
a better job here.

And having aliases only on machines with nss_winbind at least for me is not a
too severe limitation.

Comments?

Volker

allow the 'printing' parameter to be set on a per share basis.
The problem was that the current_printif struct was set during
print_backend_init() based on the 'printcap name'.  So you could
not use cups and then override the setting for a specific printer
by setting 'printing = bsd' (a common setup for pdf generation
print services.

There is a subtle change in behavior in that the print
interface functions are selecting on the basis of lp_printing()
and not lp_printcap_name(), but the new behavior seems more
intuitive IMHO.

Idra, your privileges patch allowed login only with tdbsam. The problem is
that pdb_default_get_privilege_set returns NOT_IMPLEMENTED and not a privilege
set that does not grant anything. I don't really understand privileges yet, so
work around that by not failing if pdb_get_privilege_set fails.

Volker

Sorry, forgot to commit an essential piece.

Expand aliases for winbind-generated groups.

Do:

wbinfo -C alias
net groupmap set alias alias -L
net rpc group addmem alias DOMAIN\\group -S localhost -Uroot%secret
getent group alias

And hopefully the members of domain\\group show up :-)

Still have to get them to show up in 'getent group'.

Volker

merging from 3.0

BUG 1147; bad pointer case in get_stored_queue_info() causing seg fault

Ok here it is my latest work on privileges

This patch add privilege support for samba
Currently it is implemented only for tdbsam backend but estending it to
other sam backends is straightforward.

I must make a big thank to JFM for his teachings on the matter and the
functions at the base of this work.

At thye moment only samr_create_user honours SeAddUsersPrivilege and
SeMachineAccountPrivilege to permit any user to add machines and/or users to
the server.

The command "net priv" has been provided to manipulate the privileges
database.

There are still many things to do (like support in "net rpc vampire") but
the working core is here.

Feel free to comment/extend on this work.

Of course I will deny that any bug may affect this code :-)
Simo.

This patch adds also my patch about add share command enhancements.

Fix Tim's fix for #1144. Tim, could you look at that again?

With only your fix in for example smb.conf was expected in /smb.conf...

Thanks,

Volker

When asked to delete an alias member, don't add it ...

Volker

Add aliases to winbindd_getgroups().

su - WINDOWS\\vl

now includes the locally defined aliases I'm member of.

Next will be getent group.

Volker

Fix bugzilla #1144 where running configure --with-fhs=anything would
turn on fhs compliant paths.  Spotted by Timur Bakeyev.

Apply my experimental aliases support to HEAD. This will be a bit difficult to
merge to 3_0, as the pdb interfaces has changed a bit between the two.

This has not been tested too severly (which means it's completely broken ;-),
but I want it in for review. Feel free to revert it :-)

TODO:

make 'net groupmap' a bit more friendly for alias members.

Put that stuff into pdb_ldap.

Getting the information over to winbind. One plan without linking pdb into
winbind would be to fill group_mapping.tdb with the membership information and
have that as a cache (or use gencache.tdb?). smbd on a PDC or stand-alone
could trigger that itself, the problem is a BDC using LDAP. This needs to do
it on a regular basis. The BDC smbd needs to be informed about SAM changes
somehow...

Volker

net_rpc.c: Don't complain if [add|del]mem was successful.

srv_samr_nt.c: Correctly report that a user is not member of an alias.

Volker

Trivial commit to make 'smbdiff 3_0/source head/source' two files smaller.

Volker

Print an informative error message if trying to add/remove members from
something not a group.

Volker

Add 'net rpc group [add|del]mem' for domain groups and aliases.

Volker

Fix my fix to net rpc group list. We can certainly have more than a single
set of groups.

Volker

Can't set allocation size on directories, return correct error code on
fail if file exists and target is a directory. gentest.
Jeremy.

More gentest fixes. Fix up regression in IS_NAME_VALID and renames.
Jeremy.

Added copyrights I forgot about looong ago...

rafal

Adding ability to operate on trust passwords to pdbedit. This enables
new functionality and testing interface for new pdb functions.

Also, quite a bit of objects is being added to pdbedit in order to make
it able to find domain sid automatically (if not given explicitly).
If such amount of "wisdom" is not required to be in pdbedit, I'll move
it to 'net' which will have this new functionality, anyway.

rafal

Disabling GUMS (gums_tdbsam2) temporarily, to fix the build.

rafal

Fix for writable printerdata problem - bugzilla #1112.

That const was one too many

Implement 'net groupmap set' and 'net groupmap cleanup'.

I was rather annoyed by the net groupmap syntax, I could never get it
right.

net groupmap set "domain admins" domadm

creates a mapping,

net groupmap set "domain admins" -C "Comment" -N "newntname"

should also do what you expect.

net groupmap cleanup

solves a problem I've had two times now: Our SID changed, and a user's primary
group was mapped to a SID that is not ours. net groupmap cleanup removes all
mappings that are not from our domain sid.

Volker

And another little const

Apply some const

Volker

Interesting fact found by IFSTEST /t LockOverlappedTest...
Even if it's our own lock context, we need to wait here as
there may be an unlock on the way.
So I removed a "&& !my_lock_ctx" from the following
if statement.

if ((lock_timeout != 0) && lp_blocking_locks(SNUM(conn)) && ERROR_WAS_LOCK_DENIED(status)) {

Jeremy.

(merge from 3.0)

Fix bug in previous global_sam_sid() commit.  I broke the 'read from
MACHINE.SID' file functionality.

Also, before we print out the results of 'net getlocalsid' and 'net
getdomainsid', ensure we have tried to read that file, or have
generated one.

Andrew Bartlett

Adds some more tests of renaming an open file that has been opened with
different share modes. It also has a commented out test to see if we can
actually open a renamed file, and it turns out that we can, it seems. I am
not sure at this stage how long that is the case for, though.

(merge from 3.0)

I *hate* global variables...

OK, what was happening here was that we would invalidate global_sam_sid
when we set the sid into secrets.tdb, to force a re-read.

The problem was, we would do *two* writes into the TDB, and the second one
(in the PDC/BDC case) would be of a NULL pointer.  This caused smbd startups
to fail, on a blank TDB.

By using a local variable in the pdb_generate_sam_sid() code, we avoid this
particular trap.

I've also added better debugging for the case where this all matters, which
is particularly for LDAP, where it finds out a domain SID from the sambaDomain
object.

Andrew Bartlett

Fixup strange rename error case (gentest).
Jeremy.

Do the query part of the previous fix...reset time and duration are set in minutes, not seconds.  Works from usrmgr.

reset time and duration are set in minutes, not seconds.  Works from usrmgr.

Figured out a new flags bit with gentest and ethereal....
Jeremy.

More gentest fun :-). NB. I'm not fixing OpenX breakage 'cos if you look
at what W2K3 accepts here it's COMPLETELY BROKEN ! :-).
Jeremy.

More gentest fixes.
Jeremy.

Add 'net rpc group add'. For this parse_samr.c had to be changed: The
group_info4 in set_dom_group_info also has the level in the record
itself. This seems not to be an align. Tested with NT4 usrmgr.exe. It can
still create a domain group on a samba machine.

Volker

Remove unused variable.

Volker

'net idmap restore' is too useful to be left broken :-)

Set the HWM values correctly after having manipulated the tdb.

Volker

Ensure '.' and '..' don't match in delete requests.
Jeremy.

More gentest error fixups.
Jeremy.

Fixup bad-path error found by gentest.
Jeremy.

Added NTrename SMB (0xA5) - how did we miss this.... ?
Jeremy.

Fixup correct timeout values for blocking lock timeouts (tested at connectathon
by Herb).
Jeremy.

(merge from 3.0)

Found by Fabien Chevalier <fabien.chevalier@supelec.fr> and
JustFillBug <mozbugbox@yahoo.com.au> on the Samba lists - a 'max
password age' of zero should be considered as 'never expire'.

For the timebeing we just set it like -1, but we might revisit this
for closer-to-ms behaviour.

Andrew Bartlett

Fix "unable to initialize" bug when smbd hasn't been run with
new system and a user is being added via pdbedit/smbpasswd.
Found at Connectathon setup.
Jeremy.

Add bad password count/time attributes

Add bad password count/time attributes

Add calls to password lockout functions.  Should now work against tdbsam only.

printf -> d_printf

Make SMB_FILE_ACCESS_INFORMATION call work correctly.
Jeremy.

Make us bug-for-bug compatible with W2K3 - to get delete on close semantics
on an initial open the desired_access field *must* contain DELETE_ACCESS,
simply having it map from a GENERIC_ALL won't do. Fixes delete on close test.
Jeremy.

AIX has already has a function called msleep().  Bugzilla #1098.

handle both 0 and -1 as disabling
reset count and lockout duration

Make default lockout duration and
reset count time 30 minutes, to match windows, and also be valid (0 was invalid)

<attr/xattr.h> & <sys/xattr.h> are mutually exclusive it seems; fix build on SuSE 8.2

Fixup returns from lookupsids in the same way as lookupnames. Inspired by
Volkers patch.
Jeremy.

check for <sys/xattr.h>; needed by Debian testing

remove some unused local variables

Enable checking/resetting of account lockout and bad password based on policy

Clean up bad pw count and autolock flag update fn()s

comment out server portion of DsEnumerateDomainTrusts() since it causes 2k/xp clients to fail to join the domain (see comments in srv_netlog_nt.c for details)

Add bad pw count and autolock flag update fn()s

fixing error in logic when checking flags changed field on user primary group sid

merging smbmnt fix from 3.0

Add bad password reset and display of bad password count/time

Add functions to get/set bad password time

Add bad password time fields

reformatting

Fix the "too many fcntl locks" scalability problem raised by tridge.
I've now tested this in daemon mode and also on xinetd and I'm pretty
sure it's working.
Jeremy.

Fix based on Volker's code to correctly return code from lsa_lookup_sids
on unmapped sids.
Jeremy.

When creating a group via a script, don't let winbind do it as well.

Volker

Also set alias info for builtins.

Volker

Don't try to show entries that could not be listed.

Volker

In samr_lookup_names Windows does not return WKN_GRP(5) but ALIAS(4) when
you search in BUILTIN. Match that.

Volker

If there are no alias members, don't ask for their sids.

Volker

Cosmetic fix: use sid_is_in_our_domain.

Volker

Fix success message for net groupmap modify

Volker

Sync with openldap schema.  Add munged dial and config attrs/ocs.

Typo (missed SAFE_FREE).,
Jeremy.

Fix eta calculation when resuming

Added Andrew Bartlett's patch to use an allocated buffer for count_chars.
Jeremy.

Fixup the 'multiple-vuids' bugs.
Jeremy.

Apply the multiple login stuff to smbclient ...

Fix a few comments/formatting issues to make it more in line with
3.0 and reduce diff output.

merge old change from 2.2 branch to give legend in heading

Fix format error for generating SELECT, since unknown_3 is now gone

abartlet's pdb_set/changed flag fix for NULL passwords

merges from 3.0

Fix typo: fileds -> fields

A couple of versioning adjustments.  Remove unused version #define, and
define the INFO/version string in _only_ one place.

More sync between HEAD and SAMBA_3_0

- passdb.h: replace UNKNOWN_3 with FIELDS_PRESENT

- parse_samr.c: put comment back in...seems like we shouldn't lose info like this

Start of merging changes with SAMBA_3_0.  Fix up some comments so that
they line up.  Also change lockout_time to bad_password_time, since this
is actually what is replicated.

tdbsam & smb_panic merge from 3.0 (initial tests are ok but I'll setup a HEAD PDC tomorrow)

More paranoia checks.
Jeremy.

Paranoia fixes :-).
Jeremy.

BUG 900: TAKEDA yasuma <yasuma@miraclelinux.com; fix token processing in cmd_symlink, cmd_link, cmd_chown, cmd_chmod functions

BUG 1055; patch from SATOH Fumiyasu  <fumiya@miraclelinux.com>; formatting fixes for 'net share'