s3:auth:check_sam_security: introduce a bool var to control pad_pw_count incrementation

This is a preparatory patch for the last part in fixing bug #4347 .

Michael

s3:passdb: store the plain nt passwords hashes in history, not salted md5

This is in order to be able to do challenge response with the history,
so that this can be checked when an invalid password was entered:
If the given password is wrong but in the history, then the bad password
count should not be updated...

The "lucky" bit here is that the md5 has and the nt hash (md4) both are
16 bytes long.

This is part of the fix for bug #4347 .

Michael

s3:smbd:password_in_history: treat entry with 0 salt as 0 + plain nt hash

This is to introduce a new format of the password history, maintaining backwards
compatibility: The old format was 16 byte hash + 16 byte md5(salt + nt hash).
The new format is 16 zero bytes and 16 bytes nt hash.

This will allow us to respect the last X entries of the nt password history
when deciding whether to increment the bad password count.

This is part of the fix for bug #4347 .

Michael

s3: Remove some code that has become unnecessary

The code I just removed was checked in with e5466fffc286a99f as a bug fix for
https://bugzilla.samba.org/show_bug.cgi?id=3319. With the changes to
is_visible_file made with 9e8b8f8c16612 these lines have become unnecessary,
even with "hide unreadable = yes" dead msdfs symlinks show. This is because we
can not stat(2) them and default to showing them.

Why this change? I have a user who wants to use "hide unreadable" on msdfs
links. Because you can't edit acls on symlinks themselves, the user created the
targets as bogus, empty files that just exist as acl placeholders. With the
code in place that this patch removes, we never allow this to work.

Jeremy, please check! :-)

Thanks,

Volker

s3-docs: fix eventlogadm manpage typo.

Guenther

s3: Lock down some srvsvc calls according to what w2k3 seems to do

s3:auth:check_sam_security: improve calling and logging of pdb_update_sam_account

Log what went wrongl, and also call pdb_update_sam_account inside
become_root/unbecome_root: do the logging outside.

Michael

s3:auth:check_sam_security: fix a leading tab/ws mixup

Michael

s3:auth:check_sam_security: create (and use) a common exit point

for use after sam_password_ok() has been called.

Michael

s3:auth:check_sam_security: null out sampass after it has been stolen.

So that a later talloc_free would not harm. I could have used
talloc_move instead of talloc steal in make_server_info_sam(),
but this would have required a change of the signature.

Michael

s3:auth:sam_password_ok: take username, acct_ctrl and nt/lm hashes, not sampass

This is in preparation to extending check_sam_security to also check
against the password history before updating the bad password count.
This way, sam_password_ok can more easily be reused for that purpose.

Michael

s3:auth: use data_blob_null instead of data_blob(NULL, 0) in sam_password_ok()

This way it is more explicit that there is no allocated data here
that may leak.

Michael

s3:auth:sam_password_ok: fix allocation of a data blob.

data_blob(mem_ctx, 16) does not use mem_ctx as a talloc ctx but
copies 16 bytes from mem_ctx into the newly allocated data blob.
This can not have been intentional. A blank uint8_t array of
length 16 is allocated by passing NULL instead of mem_ctx.
And using data_blob_talloc(mem_ctx, NULL, 16) adds the allocated
blank 16 byte array to mem_ctx - so this is what must have been
intended.

Michael

s3:auth:sam_password_ok: enhance readability (imho) by adding some pointers

and removing bool variables and several checks.

Michael

s3:check_sam_security: untangle assignment from statement

Michael

s3: Factor password_in_history() out of check_passwd_history()

Simplify E_md5hash a bit

s3: Simplify pdb_set_plaintext_passwd: pwhistory==NULL can not happen anymore

s3: Simplify pdb_set_plaintext_passwd: pwHistLen==0 was checked above

s3: Add a paranoia check to pdb_set_plaintext_passwd()

s3: Simplify pdb_set_plaintext_passwd() by removing a redundant condition

if (current_history_len != pwHistLen) {
     if (current_history_len < pwHistLen) {
     }
}

The second "if" is a bit pointless here

s3: Simplify pdb_set_plaintext_passwd: memcpy deals fine with 0 bytes

s3: Simplify pdb_set_plaintext_passwd by using talloc_zero_array

s3: Make use of talloc_array in pdb_set_plaintext_passwd()

s3: Simplify pdb_set_plaintext_passwd() a bit

Remove an indentation by the early return in

+       if (pwHistLen == 0) {
+               /* Set the history length to zero. */
+               pdb_set_pw_history(sampass, NULL, 0, PDB_CHANGED);
+               return true;
+       }

s3: Simplify pdb_set_plaintext_passwd() slightly

No functional change, this just removes an indentation level by the early
"return True;" in

+       if ((pdb_get_acct_ctrl(sampass) & ACB_NORMAL) == 0) {
+               /*
+                * No password history for non-user accounts
+                */
+               return true;
+       }

Volker

s3: Fix a typo

s3: Avoid a memset(, 0, ) call

s3:pdb_set_pw_history: free the old history before setting the new.

This is not strictly necessary, since this only leaks into the
struct samu, and this is not so long-lived in the code path that
changes the password, but it definitely correct and does not harm.

Michael

s3:pdb_ldap:init_sam_from_ldap: untangle an assignment from the check

to enhance readability and denbuggability.

Michael

s4/torture: fix small bug in lock test

Cleanup path should unlock, not cancel existing locked range.

s4/torture: add more lock cancellation tests

* Test the SMB1 behavior when multiple lock ranges are requested, pend,
  and then are cancelled.  The entire LockingAndX request fails.

s3:lib/time: remove TIME_T_MIN/MAX defines

we already get them from lib/util/time.h

lib/util: move TIME_T_MIN/MAX defines into header file

ѕ3:lib/time: replace make_dos_ and put_dos_ functions with those from lib/util/

s3:lib/time: remoce null_mtime() - use null_time()

s3:lib/time: remove unused nt_time_equals

we have nt_time_equal doing the same in lib/util/

Second part of the fix for bug #7020 - smbd using 2G memory.

There was a second leak in the processing of the out_data.frag
prs_struct. It needs freeing once the current pdu has been returned
asynchronously.

Jeremy.

s4-smbtorture: add test_ChangeID to RPC-SPOOLSS.

This tests ChangeID consistency between spoolss_GetPrinterData,
spoolss_GetPrinterDataEx and spoolss_GetPrinter (level 0).

Guenther

s4-smbtorture: add test_SetPrinter to RPC-SPOOLSS.

Guenther

s4-smbtorture: refactor test_GetPrinter in RPC-SPOOLSS.

Guenther

ѕ3:configure: use gettimeofday check from lib/util/

vfs_commit: print warning when no fsync support is there

this one was part of an old patch from jpeach.

tdb: fix standalone 'make installdocs'

metze

tdb: create symbol links to shared libraries see https://bugzilla.samba.org/show_bug.cgi?id=6991 for details

Signed-off-by: Stefan Metzmacher <metze@samba.org>

talloc: create symbol links to shared libraries see https://bugzilla.samba.org/show_bug.cgi?id=6991 for details

Signed-off-by: Stefan Metzmacher <metze@samba.org>

tevent: create symbol links to shared libraries see https://bugzilla.samba.org/show_bug.cgi?id=6991 for details

Signed-off-by: Stefan Metzmacher <metze@samba.org>

s4: Fix result check for getaddrinfo()

I think this completes commit 50feca550eed7828198b7c0fc5f0e5ddc863313d.
Now result should be handled correctly both for systems that
support EAI_NODATA but returns EAI_NONAME (as my Ubuntu 9.x)
and systems that doesn't support EAI_NODATA at all.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

Fix bug #7020 - smbd using 2G memory.

When deferring an async pipe writeX and readX transfer
the outstanding request struct onto the conn struct. This
needs freeing after the packet is finally processed.

Jeremy.

s4:SAMLDB LDB module - Fix trailing whitespaces

s4:SAMLDB LDB module - Rework to allow checks for wrong "defaultObjectCategory" attributes

This allows a stricter check for "defaultObjectCategory" attributes which is
performed when the "relax" control isn't specified.
Additional note: As the added comment points out this isn't complete.
And I personally think that moving this at some point to the "objectclass"
module is a better idea to make this fully work (since there we have direct
access to the schema). If someone has a good idea how to do this please inform
me.

Anyway, the SAMLDB module does require some restructure since at the moment it's
very overloaded and therefore a bit a mess. In the meantime I started to work
on a new approach to realise it in a better way.

s4:schema_load.c - Typo

s4:ldap_schema.py - add an additional check for validity of "defaultObjectCategory"

s4:ldap_schema.py - Move generated attributes check

Make more clear that they're created before the "schemaUpdateNow".

s3: List trusted domains from wcache when domain is offline.

Signed-off-by: Bo Yang <boyang@samba.org>

s3: Make winbindd_cache.c aware of domain offline to avoid unnecessary backend query.

Signed-off-by: Bo Yang <boyang@samba.org>

s3: Fix infinite loop in NCACN_IP_TCP asa there is no timeout. Assume lsa_pipe_tcp is ok but network is down, then send request is ok, but select() on writeable fds loops forever since there is no response.

Signed-off-by: Bo Yang <boyang@samba.org>

s3 net: Fix up a share type translation and translate some more strings

s3 net: Add a few missing gettext calls.

Many many thanks to Adi Roiban from #ubuntu-translators for helping me find
and fix my problems with the translations.

Fix the merged build. Probably not strictly correct but allows us to "make test".
Jeremy.

s4:dsdb/repl: convert dreplsrv_out_drsuapi_send/recv to tevent_req

metze

blackbox/test_export_keytab.sh: correctly remove temporary files

metze

blackbox/test_export_keytab.sh: use VALGRIND for samba4kinit

metze

s4: Happy New Year 2010

metze

s3: Happy New Year 2010

metze

s3: Trim down smbcontrol a bit

s3: Fix a shadowed variable warning

s3: Fix an uninitialized variable warning

s3: Convert cli_ulogoff to the async API

s3: Convert cli_tdis to the async API

s3: Slightly simplify winbindd_dual_ccache_ntlm_auth

data_blob_const can't fail

s3: Fix some nonempty blank lines

libwbclient: Remove a pointless check

We have dereferenced "blobs" before

s3: Remove some unused code

s3: Convert cli_sesssetup_ntlmssp to the async API

s3: NT_STATUS_MORE_PROCESSING_REQUIRED is a valid sesssetup return value

s3: Convert cli_session_setup_kerberos to the async API

This is still cheated, acquiring the ticket is not async yet, but the SMB
part is

s3: Fix a typo

s3: simplify find_root_domain, find_our_domain() never fails

s3: Use global_sid_Builtin in find_builtin_domain

s3: Avoid adding a domain twice

If we found a match with sid==NULL, we ended up adding the domain twice

s3: Make free_domain_list() static

s3: Adapt sid_dup_talloc to README.Coding

s3: Introduce domain_is_forest_root() helper function

Hopefully this makes the flag tests a bit more understandable

s4-dsdb: force REVISION_ADS for new and updated ACLs in dsdb

w2k8-r2 gives a "schema mismatch" error if the revision is not set to
REVISION_ADS and you replicate the ntsecuritydescriptor using DRS.

Nadya, please check this!

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-drs: don't give an error on repsTo delete if add is also specified

w2k8-r2 in dcpromo asks for a delete+add during its initial join.

s4-sddl: DRS replication needs REVISION_ADS for SDs

DRS replication with w2k8-r2 fails with a schema mismatch error if we
set the revision to NT4

ldb: Fix the standalone ldb build.

libreplace: some systems don't have memmem()

added rep_memmem() and a testsuite

s4-dsdb: switched to using RMD_FLAGS instead of DELETED in extended DNs

This allows for more flags in the future

s4-kcc: added a preiodic task to remove deleted objects

we check for deleted objects in each partition every 10 minutes, using
onelevel searches

s4-dsdb: fixed several memory leaks

need to be careful with those temporary contexts

s4-dsdb: fixed samdb_create_foreign_security_principal() to use the wellknown GUID

This also fixes a memory leak

s4-ldb: fixed valgrind error: ares can be freed by callback

s4-dsdb: make sure 'whenChanged' is set on modify

We also should preserve (and then replace) whenChanged on delete

s4-dsdb: added dsdb_tombstone_lifetime()

s4-dsdb: allow system to remove deleted objects

This will be used by a periodic job to remove tombstoned objects

s4-ldaptest: need to use MessageElement for modify messages

Without MessageElement() the flags are not set, which is invalid

s4-ldb: show an error string, as well as error message

This makes it easier to track down error mismatches from the test
suite

s4-drs: make sure the DNs we put in the db have a extended GUID