Fix a signed/unsigned warning.

More notes added.

Add mutex protection around auth calls.
Jeremy.

Add mutex protection around auth calls.
Jeremy.

Merge of holding pattern stuff from HEAD.

s/WIFSIGNALLED/WIFSIGNALED/g

s/WIFSIGNALLED/WIFSIGNALED/g

missed a file

do not use RCS variables in file

do not use RCS variables in file

syncing docs with HEAD

Remove debugging stmt.

add ntlogon directory and files from 3.0

Sync up compiler warning fixes.

Removed level 0 DEBUG leftover from testing.

and another comment.

dded further comment.

Remove a wayward file.

Adding Win2K profile info.

Adding more notes.

Missed auth_ntlmssp.c in last night's checkin.  Also keep track of the current
challenge in the NTLMSSP context.

Andrew Bartlett

Added note about Win2Pro RK.

playing janitor for tpot; remove holding pattern

remove torture tools on make clean

*lots of small merges form HEAD
*sync up configure.in
*don't build torture tools in make all
*make sure to remove torture tools as part of make clean

small merges from SAMBA_3_0; mostly typos, renames, etc...

sync up ignored files from HEAD

winbindd merges from HEAD

merging some rpcclient and net functionality from HEAD

initialize acct_ctrl before using it
remove ldap_msgfree(result); as result is unitialized at this point

initialize acct_ctrl variable before it is used
remove ldap_msgfree(result); as result is unitialized at this point

more unused files

remving more unused files

[merge]
* removed unused variable from rpcclient code
* added container option to net command (patch from SuSE)
* Makefile patch for examples/VFS from SuSE

First set of Profile Manipulation Notes.

* removed unused variable from rpcclient code
* added container option to net command (patch from SuSE)
* Makefile patch for examples/VFS from SuSE

removing unused files from release branch

This commit was manufactured by cvs2svn to create branch 'SAMBA_3_0'.

Refactor the NTLMSSP code again - this time we use function pointers to
eliminate the dependency on the auth subsystem.  The next step is to add
the required code to 'ntlm_auth', for export to Squid etc.

Andrew Bartlett

Doing a malloc(strlen(s)) then a pstrpcp(y, s) is just silly, make it a strdup
instead.

Andrew Bartlett

added cli_lsa_enum_account_rights() call. Note that this is in
principal similar to the existing cli_lsa_enum_privsaccount() call,
except that cli_lsa_enum_account_rights() doesn't require a call to
open_account first. There is also the minor matter that
cli_lsa_enum_account_rights() works whereas
cli_lsa_enum_privsaccount() doesn't!

this call can be used to find what privileges an account or group
has. This is a first step towards proper privileges support in Samba.

Fix warnings by getting the function into a shape the autoprototyper will
read, and add a static.

Andrew Bartlett

Crash fixes:
 - fix a crash when a second NTLMSSP session tried to free the first
 - fix a crash due to some NULL pointers in the Add Printer Wizard (or read
   printer code too it appears).  As far as I can tell it's just that the
   GUID just might not exist.

Andrew Bartlett

One more from waitpid() audit: Correctly interpret waitstatus when a
child fails.  Possibly this was causing smbmount to exit(0) when it
really should have indicated a failure.

Run autoheader

Fix bug where the existence of config.cache would cause functions like
yp_get_default_domain to be misdetected.

According to the autoconf manual we need to check for libraries before
looking for functions in them.

IRIX uses libns_winbind.so instead of libnss_winbind.so

IRIX uses libns_winbind.so instead of libnss_winbind.so

[merge] removing unused variable and unused file

removing unused variable and unused file

Fix some debug lines, and add a bit more info to help track down ldap
connectivity problems.

Andrew Bartlett

merge: disable [u|g]idPool objects

make -i flag work lik eit did in 2.2

make -i flag work lik eit did in 2.2

Merge from HEAD - patch by metze to fill in 'workgroup' etc in testparm output,
and probably SWAT too..

Andrew Bartlett

Merge from HEAD:

Keep all the const warnings in one place, by adding a utility function to
make the TDB_DATA.

Merge from HEAD:
 - fstring/pstring mixups
 - the detection code that found them (disabled)
 - a bit of whitespace
 - a static

Andrew Bartlett

Merge from HEAD:
 - remove useless #else
 - signed/unsigned fixes
 - use an fstring for LM hash buffer.

Andrew Bartlett

Merge from HEAD - eliminated unused arguments.

Andrew Bartlett

Merge indirection, signed/unsigned and uninitialiased-value fixes from HEAD.

Andrew Bartlett

Commit my (disabled) fstring/pstring mixup detection code.

We need to fix some 'overmalloc' cases before it can be enabled by default.

Andrew Bartlett

The last of the fstring/pstring mixups, and an extra 'static'.

A couple more pstcpy/fstrcpy mixups - doing an fstrcpy into a pstring is
harmless, but breaks my automated check.

Added comment about a SMB_ASSERT()

Remove wrapper function _get_trust_account_password() as it doesn't
seem to do anything useful anymore other than call
secrets_fetch_trust_account_password().

Added a comment in case anyone is thinking of doing a SMB_ASSERT in winbindd.

merge alt_name patch from head

query_alt_name takes a forced unicode string in win2000. It is not
null terminated.

Thanks to Metze for finding this.

clearer debug message when the user is already in the ldap db

add help text for 'net ads lookup'

Don't touch data after a free.
Jeremy.

Missing indirect in final free.
Jeremy.

Merge of indirection fixes from HEAD.
Jeremy

Always initialise this variable - and don't set the 'must change now' if it was
last changed at '0'.

We need to actually change this password sometime...

Andrew Bartlett

Fix typo, and the build.

More janitorial duties. Pulling these in from samba-head.

Make sure that those cleanups actually went in.

Now that I am running config.developer, I decided to get rif of some warnings:

1. reboot in parse_reg and cli_reg was shadowing a definition on FreeBSD
   4.3 from system includes.

2. Added a bit of const to places.

3. Made sure internal functions were declared where needed.

Patch from metze to add what he feels is the correct semantics for a Domain
Controller.  As we have had a number of attempts at this over the last little
while, I need to get my test rig going, and give this whole area a poke...

Meanwhile, if you want to use this, just adjust your 'auth methods' line to use
samstrict_dc...

Andrew Bartlett

Patch from metze to to make testparm show values for 'workgroup', 'netbios
name' and 'netbios scope'.  Probably has a similar effect on SWAT.

Also adds '-V' to testparm.

Andrew Bartlett

Patch from ab, to make findsmb omit -r most of the time.

(original mail follows)
--
From: Alexander Bokovoy <a.bokovoy@sam-solutions.net>
Greetings!

I propose to make an optional argument '-r' for findsmb utility to pass
'-r' option to nmblookup optionally and default to omit it. The reason is
to have findsmb working more user-friendly in contemporary office
environments where Windows 95/98 boxes are rare and also to allow usage of
the utility under non-priviledged accounts when Windows95/98 boxes are in
minority or even absent in network neighbourhood.

An attached patch provides this change for HEAD together with
documentation changes.

--
/ Alexander Bokovoy

Updates to our NTLMSSP code:

This tries to extract our server-side code out of sessetup.c, and into a more
general lib.  I hope this is only a temporay resting place - I indend to
refactor it again into an auth-subsystem independent lib, using callbacks.

Move some of our our NTLMSSP #defines into a new file, and add two that I found
in the COMsource docs - we seem to have a double-up, but I've verified from
traces that the NTLMSSP_TARGET_TYPE_{DOMAIN,SERVER} is real.

This code also copes with ASCII clients - not that we will ever see any here,
but I hope to use this for HTTP, were we can get them.  Win2k authenticates
fine under forced ASCII, btw.

Tested with Win2k, NTLMv2 and Samba's smbclient.

Andrew Bartlett

Accessing data after it's been free()ed really is a no-no...

Andrew Bartlett

Fix to debian bug #171071 - we had the wrong dereference on the pointer to be
Realloc()ed, causing it to fail.

Big thanks to Sandor Sonfeld <sonf@linuxmail.org> for the debug, stack and
valgrind traces!

Andrew Bartlett

Don't force the DOS password into a 14 char space, as this would imply null
termination - the password will not be null terminated before hashing if
len >= 14.

related to debian bug #157432

Andrew Bartlett

Ensure we do not pass uninitialised data to the kernel.  (Picked up by
valgrind).

Andrew Bartlett

Fix typo in comment.

Merge ... remove "winbindd holding pattern" from appliance.  It's a bit
simpler as the rescanning of trusted domains helps us out a bit.

Keep all the const warnings in one place, by adding a utility function to
make the TDB_DATA.

Andrew Bartlett

Oops, this is the change to use an fstring for the incoming service buffer -
the commit to reply.c just matches a pstrcpy for the pstring.  (harmless, fixes
it for the automated test).

Andrew Bartlett

Make the 'service' in make_connection() use an fstrcpy(), and an fstring,
becouse that is what it's input (reply_tcon_and_x) uses, and becouse we really
don't want supprises for service names.

Also remove a legacy #define, in favor of the lp_ equiv.

Andrew Bartlett

While the usage is most bisarre, this fixes up another fstring/pstring mixup.
(In this case, an fstrcpy() into a pstring, ie harmless).

Andrew Bartlett

Use size_t for the counter vars, to match the type they are assigned from
(signed/unsigned mixup).

Andrew Bartlett

A couple more fstring/pstring issues - and move from sizeof(fstring) to
sizeof(var) in slprintf statements, so as to ensure we don't get mixups there.

Also remove fstrcpy() into a variable that is about to be printed into anyway.

Andrew Bartlett

Remove an unused paramter for our old LM-only password change code, and fix a
(harmless) fstring/pstring mixup.

Andrew Bartlett

Fix a number of client-side fstring/pstring mixups.

Andrew Bartlett

Patch from Nik Conwell <nik@bu.edu>.  Don't reference free()ed data when trying
to figure out if we have got our ticket yet.

Andrew Bartlett

Return the 'freindly' NT error message if at all possible.

Andrew Bartlett

No point having this for both parts of the 'if' statement.

Andrew Bartlett

[merge] make sure to update print queue cache during timeout_processing() to send notify events; CR 1491