r25156: Update coding guiding wrt to primitive data types
(This used to be commit 01d42744903d7ef9044fa4d2ad1cc64594717573)

r25154: move winbindd code into winbindd/

metze
(This used to be commit 3ac7566ae14c48ff9b0f6b232e0ec4b2f73df558)

r25152: fix headers used in wbinfo.c

metze
(This used to be commit 2a794db3c548330d477497aa66245845486d9888)

r25150: don't recursivly call call dump_core()
as currently seen in 'make test' localy and in the farm

metze
(This used to be commit f41efe01b5a22889f5b18916df9a2f924263e7f9)

r25148: Adapt to coding conventions.

Guenther
(This used to be commit c3b423c52a2bf3f50870158d8c7ffd314c8ac935)

r25146: rename winbindd_nss.h => winbind_struct_protocol.h

as this header has nothing todo with winbindd nor nss
and it contains the definitions for the struct based
protocol

metze
(This used to be commit e9e03aac2221c44eec89cc957b8bb3be721103bd)

r25143: rename public functions from winbind_client.h

init_request => winbindd_init_request
free_response => winbindd_free_response
read_reply => winbindd_read_reply

write_sock => winbind_write_sock
read_sock => winbind_read_sock
close_sock => winbind_close_sock(void)

metze
(This used to be commit 8a95d7a7edcfa5e45bccc6eda5c45d9c308cb95d)

r25142: Panic if setting the group list fails while switching security
contexts. Patch from Tim Prouty <tim.prouty@isilon.com>.
(This used to be commit a136de663f122603e8d34e06027896ff39b35e11)

r25141: More pstring removal.
Jeremy.
(This used to be commit cfcf7cf03e1be34e6839c1a659c4e8c1b5358c37)

r25140: Less red bars to hurt my eyes...
(This used to be commit f935d21200294c8d29ce527300f605e721cb3620)

r25139: Avoid code duplication: let regval_ctr_copyvalue() call regval_ctr_addvalue().
This also corrects regval_ctr_copyvalue() in that it cannot create (invalid)
regval containers with dupliacte entries...

Michael
(This used to be commit 2daaaaa835078c543fa12cd1819e8a3d86cf6e5e)

r25138: More pstring elimination. Add a TALLOC_CTX parameter
to unix_convert().
Jeremy.
(This used to be commit 39c211a702e91c34c1a5a689e1b0c4530ea8a1ac)

r25136: When tallocing a string to uppercase remember the terminating
'\0' in size calculations.
Jeremy.
(This used to be commit 54c658867db3b6c602080d60936abab3af26e112)

r25135: Remove one END_PROFILE(SMBntcreateX) from call_nt_transact_create() which
appears to be a leftover. Should fix the build with profiling enabled.

Jeremy, please check.

Guenther
(This used to be commit 98f2e10e3f7a86b1b30619a25ee15a489ad10d43)

r25134: Fix Bug #4968 and make dns updates work with heimdal as well (again,
gss_import_name() needs to follow the same logic as in the LDAP sasl wrapping
(see -r25133).

Tested with MIT 1.2.7, 1.3.6, 1.4.3, 1.5.1, 1.6.1 and Heimdal 0.7.2, 1.0,
1.0.1.

Guenther
(This used to be commit 913fb138aa90a6627ee49b2e92d1bbc73e11f9ea)

r25133: Fix sasl wrapping (for ldap sign&seal).

The gss_import_name() broke as we switched from the internal MIT OID
"gss_nt_krb5_principal" to "GSS_KRB5_NT_PRINCIPAL_NAME" and didn't switch from
passing the krb5_principal (or better: a pointer to that, see MIT's "*HORRIBLE*
bug") to pass the string principal directly.

Jerry, Jeremy, neither I could figure out the need of passing in a
krb5_principal at all nor could I reproduce the crash you were seeing.

I sucessfully tested the code (now importing a string) with MIT 1.2.7, 1.3.6,
1.4.3, 1.5.1, 1.6.1 and Heimdal 0.7.2, 1.0, 1.0.1.

Guenther
(This used to be commit cb2dc715e33467c8b588161e816e72a948f6860c)

r25130: make use only of base types which are provided by libreplace
in winbind client and nss/pam stuff

metze
(This used to be commit 2e13e05fa91788bd128e6940bccc0d2cc7140986)

r25128: _XOPEN_SOURCE_EXTENDED and socklen_t are always provided by libreplace

metze
(This used to be commit b3ee9adf28ee8136528d0236a3a2c894c2223053)

r25127: Add ol-schema-migrate.pl to the repo.
This script is useful for migrating OpenLDAP schema files to FDS/RHDS
lidf schema files.

License kindly updated to GPLv3+ at our request.

Simo.
(This used to be commit ab7770b34b3202a5836cfa098187eeed1bd16be3)

r25125: create prototypes for 'bool ' functions

metze
(This used to be commit 5afa66970ee831a8be887c9e7eeeaceead6049db)

r25121: Remove pstring limits from much of our string handling function.
Still a few left (mainly the substitute ones).
Jeremy.
(This used to be commit 6552e52979a7c6954faa8b2c9c0f08b3779835c4)

r25120: One more warning.
Jeremy.
(This used to be commit 9cc70e3ccff9c3b20076572235d9dcbfd23e97e5)

r25119: Fix a couple of warnings.
Jeremy.
(This used to be commit 0e655ac94653128566a8764ea804f37c4c7b300f)

r25118: More pstring elimination.
Jeremy.
(This used to be commit 7632f8fb4003657591778d2b55f546d1737859d1)

r25117: The mega-patch Jerry was waiting for. Remove all pstrings from
the main server code paths. We should now be able to cope with
paths up to PATH_MAX length now.
Final job will be to add the TALLOC_CTX * parameter to
unix_convert to make it explicit (for Volker).
Jeremy.
(This used to be commit 7f0db75fb0f24873577dcb758a2ecee74fdc4297)

r25116: Fix talloc_asprintf_append to do the right thing with
truncated strings.
Jeremy.
(This used to be commit 93c42fd9b52d9135d8c8b52e97cf342c1c5b5010)

r25111: Move to talloced pathnames on most code paths.
There are now ony 17 pstrings left in reply.c,
and these will be easy to remove (and I'll be
doing that shortly). Had to fix an interesting
bug in pull_ucs2_base_talloc() when a source
string is not null terminated :-).
Jeremy.
(This used to be commit 0c9a8c4dff10974dbffd2a302ae982896122fcc0)

r25109: Remove obsolete argument from ads_guess_service_principal().

Guenther
(This used to be commit 2dea9464bba76af4315a8207ccd3e564ec19d146)

r25108: Make ifdef labyrinth in sasl code a bit more readable.

Guenther
(This used to be commit f31949ec3456134de474a0219a8cd5dcd15adea6)

r25107: Fix const warning caused by the way I reformatted this
(This used to be commit 6ce01bf41339b4c48aa458bb0667664727d3dac3)

r25104: Fix extra lines added by mistake.
Jeremy.
(This used to be commit a28a35f9cdddb9da1b2adb9e2e6c86d78cf07ea6)

r25103: Ensure we don't return unwritten memory (valgrind caught).
Jeremy.
(This used to be commit b3f0d39d11fa18b7bfef6cec88efaf4a2be2d6e0)

r25102: Rewrite msdfs code to use talloced filenames. Passes make test
and make valgrindtest. Final step will be to change srvstr_get_path()
to return talloced memory in the major codepaths.
Jeremy.
(This used to be commit cf6b6f9c3a38b68d2671c753f412772344506742)

r25099: Commit fix for CVE-2007-4138 from 3.0.26 release.
(This used to be commit aa768a4a20930833d6921ed112601873b228a64d)

r25092: Add support for storing trusted domain passwords in LDAP for
passdb backend = ldapsam.

Along with reproducing the functionality of the secrets.tdb
code, I have prepared the handling of the previous trust password
(in case we are contacting a dc which does not yet know of a recent
password change). This information has still to be propagated
to the outside, but this requires a change of the api and also
a change of the secrets.tdb code.

Michael
(This used to be commit 6c3c20e6c4a2b04de8111f2c79b431f0775c2a0f)

r25091: Start adding support for storing trusted domain passwords in LDAP
(for passdb backen = ldapsam). At a first step, add the hooks,
calling the secrets_ functions.

Michael
(This used to be commit 9c03cdf3a449149c50451a44deb420341e65af34)

r25090: Fix a syntax error just introduced into the LDAP schema.

Michael
(This used to be commit 7dc68ce0fb4490761326eaaca516fde3bf642ed5)

r25088: Change the objectclass sambaTrustedDomainPassword to
have the current and possibly the previous trust password
stored as clear text passwords. (Previous use of NTPassword
was a mistake - this is a hash value.)

Michael
(This used to be commit 0beae52ff469903adbfefdffd93a34bb7ad7d68d)

r25087: Reformatting: Get rid of an abundance of leading tabs...
(This used to be commit d8676f30d5b8168ff0c82993fb51cff18bea1f96)

r25086: Fix interdomain trusts (this povides the fix expected in r22709):

Fix winbindd on a Samba DC talking to a trusted domain DC by
making it use the trusted domain password...

Michael

I hope this does not brake any other setup.
(This used to be commit 2322fe5cd76c5b7214c66f8a403debe1e36a82d6)

r25080: Once we decrypted the packet but have timing problems (closkew, tkt not yet or
no longer valid) there is no point to bother the keytab routines.

Guenther
(This used to be commit 7e4dcf8e7ecfd35668e86e22bed5a9280ae83959)

r25074: as all requests in the winbindd child are sync, we can use talloc_tos()

metze
(This used to be commit 6e238fe5b458fc6b9184144fa124f497c81eec8d)

r25068: Older samba3 DCs will return DCERPC_FAULT_OP_RNG_ERROR for every opcode on the
LSARPC_DS pipe, continue with no_lsarpc_ds mode here as well to get
domain->initialized set to True. This avoids permanent scanning of Samba3 DCs
in winbindd. Thanks Michael, for pointing this out.

Guenther
(This used to be commit 9b85f7ca43896ddd40dd8cb4a52969e94914db43)

r25064: We use BOOL/True/False not bool/true/false
(This used to be commit f624a269bf242630e9bda7a89b9b0e8e12646982)

r25063: Fix segfault in smbldp_set_creds when we want to use anonymous, the
code was not passing in the "anon" flag correctly and was passing
NULL pointers.
(This used to be commit 6316a9c14d8bc467c84c1604248a9e30abaacd2f)

r25062: Reformat, remove trailing spaces and fit lines into 80 columns
(This used to be commit 1850d1c10f1fe77402e0ed79d6980d55d90637fb)

r25061: Pro-actively shut up Coverity :-)
(This used to be commit f2ce4a803cfd04fa993d2d87720b4b6f67fc46db)

r25060: Fix formatting, remove trailing spaces and cut lines longer than 80 chars
(This used to be commit 7a4d4658904a601ae42ec1804db06cda734553d2)

r25059: Apply to 3.2 as well
(This used to be commit 17a7d33b3bede85d6d64ecadfa065e71739517be)

r25057: Add a schema file ready to be used in Fedora/RedHat Directory Server
(This used to be commit 501e8b6c79afdefb9955b92e5c25dbe1033ab518)

r25055: Add file_id_string_tos

This removes file_id_string_static and file_id_string_static2
(This used to be commit 638c848c9afe374feb30e34c494f89b2a6c64f7b)

r25049: Set new, more secure defaults for Samba 3.2.

Andrew Bartlett
(This used to be commit 87d39f61b498c1327c2b3a818e41bb6451fe7518)

r25040: Add "net sam rights"

Not strictly in the SAM, but close enough. This command acts directly on
the local tdb, no running smbd required

This also changes the root-only check to a warning
(This used to be commit 0c5657b5eff60e3c52de8fbb4ce9346d0341854c)

r25036: Fix a become/unbecome_root() pair
(This used to be commit 9e53f50e9489a5f6cfa40eed59be39e8a294db7d)

r25032: Contact an off site DC if non is available on site.
(This used to be commit 50879e6de5101b6c5ab8b3fb954f1d2a48530716)

r25030: ip_srv_nonsite and count_nonsite are initialized in get_kdc_list() in any
case.
(This used to be commit 287604a1c7dc7dede4b278de92ad8233f597d0b6)

r25025: Remove an unused function
(This used to be commit 858f7b7d786ca352ffebb56b9fda40c636acef74)

r25024: Fix a whole bunch of Coverity bugs

The callers of get_domain_group_from_sid() with some justification
expected map->gid to be initialized when get_domain_group_from_sid
returned True.
(This used to be commit bc8b74dbfec965ede7bf45118d1a863b28d000fd)

r25023: Coverity #455. Not actually a bug, but this clarifies
the code a lot.
Jeremy.
(This used to be commit 5ba12eefbed209a8a6b474da479efd190f91638e)

r25022: Fix coverity #395. Resource leak on error.
Jeremy.
(This used to be commit 6a237600b825f4df2241277bdd80e98669f69685)

r25021: Fix coverity #435. Use of -1.
Jeremy.
(This used to be commit f789186086b55a81c52e05d1f8c97c33b69131bd)

r25020: Fix coverity #374, possible null deref.
Jeremy.
(This used to be commit 82b3656b4b26ff63d894eed975d73ef3c8b5d2b2)

r25019: Fix coverity bug #105, run #332. Use of uninitialized variable.
Jeremy.
(This used to be commit a58de8cee51c1396a2607ee743c92d58d7703547)

r25011: We don't use mangle_map any more.
Jeremy.
(This used to be commit 2e0999c1df33c192fee5f739070a0e4c3d22dff8)

r25009: Large patch discussed with Volker. Move unix_convert to a talloc-based
interface. More development will come on top of this. Remove the
"mangled map" parameter.
Jeremy.
(This used to be commit dee8beba7a92b8a3f68bbcc59fd0a827f68c7736)

r25002: Refactor out code duplication created by the previous
unification of r24998.

Michael
(This used to be commit 6026d132537f2fb2a963fa54377a926fa99eeb35)

r24999: Use the new regval_compose() function in regval_ctr_addvalue().

Michael
(This used to be commit dab9ffe602eaca478a73f3b882f543629ce3001e)

r24998: Add a function regval_compose() to compose a REGISTRY_VALUE from
input data. Use this function in a first step to refactor
the canonicalization code of smbconf_store_values().

Michael
(This used to be commit f4caa2d7d412e2b9bb2a1ce71514551569ccf373)

r24996: Eliminate more trailing spaces...
(This used to be commit 1ee1b75113c7035c8882fe53981dba7cfc3aa187)

r24993: Apply some const
(This used to be commit 613b9fcd18bcc29bf5313e2287b53b8de430d17e)

r24991: Kill all those lots of trailing whitespaces from secrets.c.
These red bars in vi really hurt my eyes... :-o

Michael
(This used to be commit 2e99e141c3254fe072756697b8db3cbd4e4f1db4)

r24990: Kill an incredible amount of trailing whitespaces...
Further reformat get_trust_pw to conform to coding rules.

Michael
(This used to be commit b9e76a479e933084b1ee081ef5d8bd6bdbd7fadf)

r24981: - Use the formal syntax for calling functions through pointers.  I've wanted
  to make this change for ages, but now with the issue of "open" requiring it,
  this is the time to just do all of them.

Derrell
(This used to be commit e746aaaf4db7099252ef048da7857bd488cb681f)

r24978: move domain_init_recv() near the functions that uses it

metze
(This used to be commit 18e830a935d4712bfbff54f94b531e30cea8a1f9)

r24977: Ensure negative caching for name2sid, sid2name and rids2names
mappings.

rafal
(This used to be commit 3a9b5eabf97e892e761184da37465d850b5d774c)

r24974: catch SIGINT and SIGQUIT like SIGTERM

metze
(This used to be commit ab15f5c3e72df6272db9a4431f720f604fcf40f2)

r24970: Polish some comments and add my (C).

Michael
(This used to be commit aa043bb446ec8ab90a8e61c590be8d33b6a2d42d)

r24969: Fwd port "open" patch
(This used to be commit 113d62682ae8b045ff0132a743a32f3bc4856d54)

r24955: Reformat some object lists in Makefile.in to minimize the diff.
(This used to be commit 5d94c3dab2dfab68f170e18e6af9374026f6f194)

r24954: Fix the build without krb5.

Guenther
(This used to be commit 9df961914c3119eb6b20ecdc05d963480dea3521)

r24952: Set the kdc locator env vars. This makes the krb5 locator plugin fully
operational (from within winbindd and outside).

Guenther
(This used to be commit 800645d5388d7df1f8cf31b4218bfc7dd5b4d12a)

r24951: Warn if "os level" > 255

Thanks to Karolin Seeger <ks@sernet.de>
(This used to be commit 3abf06efc1286090caf380eb685e90ba7f412748)

r24949: Remove some static buffers
(This used to be commit df648d47ff3c4e24f439fda839653bda98323100)

r24943: Some stackframes
(This used to be commit cddb9f11d5fafcd3797cb242775c37f0c04d4f15)

r24920: Reformatting.

Guenther
(This used to be commit 62426a055513c309a381d0d96d60edc07e6dfcdf)

r24903: One more tick in #if 0 code
(This used to be commit e0b4fb55e3176785a85d66efece15d0a366a35f1)

r24902: DEBUG might use talloc_tos() itself...
(This used to be commit 1b39374a173bf6b40426000d010d092e94afc4d7)

r24879: Activate the winbindd cache-validation message handler.
Now the winbindd cache can be checked at runtime by
calling "smbcontrol winbindd validate-cache".

For the execution of the validation code, I fork a child
and in the child restore the default SIGCHLD handler in
order for the fork/waitpid mechanism of tdb_validate to work.

Michael
(This used to be commit f379a5c47d5004a5a66b6c12ec119c739b9e146d)

r24877: Don't panic in tdb validation code when the fork or waitpid fails.
Return error instead.

Michael
(This used to be commit 9b25355dd66519afd548fecbf2da7608b024a35b)

r24864: - Correct failure of libsmbclient against a version of Windows found on a NAS
  device.  The device resets a NBT connection on port 139 when it receives a
  NetBIOS keepalive request.  That request should be supported when NetBIOS is
  in use; Windows is behaving badly.

  libsmbclient needs a way to determine if a connection is still alive, and
  was using a NetBIOS keepalive request if port 139 was in use (on the
  assumption that it was probably NBT), and getpeername() when port 139 was
  not being used (assuming naked transport).

  This patch simplifies the code by exclusively using getpeername() to check
  whether a connection is still alive.  The NetBIOS keepalive request is
  optional anyway (with preference being given to using TCP mechanisms for the
  same purpose), so this should be both simpler and more reliable.

Derrell
(This used to be commit 1f122352b02e3f4be9ac2d638b18807dafd05429)

r24853: Rename function as Jerry asked.
s/net_use_upn_machine_account/net_use_krb_machine_account/

rafal
(This used to be commit 86af9fedad71697f22cc739518340f7753b8f9da)

r24848: Make tdb_validate() take an open tdb handle instead of a file name.
A new wrapper tdb_validate_open() takes a filename an opens and closes
the tdb before and after calling tdb_validate() respectively.

winbindd_validata_cache_nobackup() now dynamically calls one of
the above functions depending on whether the cache tdb has already
been opened or not.

Michael
(This used to be commit dc0b08e6590caa4974fd4d9f34f39f261d1f1dee)

r24847: Change standard failure return code of tdb_validate from "-1" to "1".
(This is more safely used with casts from int to uint8, e.g.)

Michael
(This used to be commit cb3583b39212688a627a29bbddb2e98497714d42)

r24845: Fix a segfault in smbcontrol when called with parameters but no extra args.

Michael
(This used to be commit f1bbf9bc18fc615a5660674f8346b4a7bed50044)

r24843: Add a "validate-cache" control message to winbindd.
So there is a new subcommand "smbcontrol winbindd validate-cache" now.

This change provides the infrastructure:
The function currently returns "true" unconditionally.
The call of a real cache validation function will be incorporated
in subsequent changes.

Michael
(This used to be commit ef92d505c04397614cb0dd5ede967e9017a5e302)

r24842: Fix build warning.

Guenther
(This used to be commit 205ef6ab3e9332a2a4a4f775c2cdddbcd3ec739d)

r24841: The locator has no dependencies to ldap libs anymore. Also fix the build.

Guenther
(This used to be commit ce7c816083db6fc844b936a14dd593fe1ef07374)

r24838: Now for real: build the locator, if we can.

Guenther
(This used to be commit 28f9b8d98dabd9ba8e812ca01bcf4fc192fb5e44)

r24836: Initialize some uninitialized variables.
This prevents a segfault when get_kdc_ip_string() is called
with sitename == NULL.

Michael
(This used to be commit 58d31e057b57bc69a96e63aabba9aa1da5418d83)

r24833: Move locator to nsswitch (does not belong to libads anymore).

Guenther
(This used to be commit af90c6949c929c82d4390b2f87a420bd598275dd)