r16203: Fix potentially writing one result of strtok beyond the end of alt_path.

Found by Klocwork, ID 653.

Volker
(This used to be commit 4cb8cf221f986b15de3ac608b173310c14248533)

r16202: Fix Klocwork #3. Strange - was already fixed in HEAD.
Jeremy.
(This used to be commit 319f80bbf0455cfaf80eab51313a56db4ed04ac5)

r16201: Fix Klocwork 439
(This used to be commit b369d0891afe8b777b837eaac317131232568ca7)

r16199: Fix Klocwork #1 - ensure we test the first
strtok for NULL.
Jeremy.
(This used to be commit 98751e8190317416de56b4a19a489c5f4b7d6bc9)

r16198: Fix Klocwork 1023
(This used to be commit 3a7d01afbebe5c4c36935b6128998e1327c8f856)

r16197: Fix Klocwork id 1338
(This used to be commit 963ae5bd144b90e71331a88029698a1a6dc52190)

r16196: A bit of defensive programming:

Klocwork ID 1773 complained about oldest being dereferenced in line 2275 where
it could be NULL. I think you can construct extreme racy conditions where this
actually could happen.

Volker
(This used to be commit b5602cc4f1d77ed48ddca0f7f42b28706160c923)

r16195: starting development for 3.0.24pre1
(This used to be commit 1b64f9a48bbb5a88c2029585e208865b41f2f423)

r16193: fix packaging dependency on RH platforms for the Unicode::MapUTF8 perl module (caused by upgarde in smbldap-tools)
(This used to be commit abca3b12e78814a395b806a9969a8f12b3230dfd)

r16192: Fix timeformats in the winbind response struct.

(pam_winbind users were forced to change a password inappropriately)

Guenther
(This used to be commit 65643d31725a4e3fe157d66e9ecad03a65a484e2)

r16190: Fix more memleaks.

Guenther
(This used to be commit dfebcc8e19bee06b7c03f88845314e9cfd6f398a)

r16187: Fix memleak.

Guenther
(This used to be commit e7d2b84aba2f2f5d844ba6a5fdcce35c3750d0b2)

r16156: Fix storing NULL in the wrong place. Klocwork id's 127 and 128.

Volker
(This used to be commit 7674a4f8361d3f3b649245118b82d8a074a2760e)

r16155: Janitor for jelmer.
Fix typo in DEBUG()
Jeremy.
(This used to be commit da768bf9c2761884cc97f690133d6897ed353e15)

r16154: Fix winbind function table typo.

Guenther
(This used to be commit aeff1f0c47992ce3941e27e63f9b1516c4918963)

r16153: Fix possible NULL dereference found by Klocwork # 252
(This used to be commit 128260527b90d77ca3dfc900e012018ef00ba9e0)

r16150: Fix possible NULL dereference found by Klocwork ID # 17
(This used to be commit 3159bd3a4e3ad70c60fea4cacc892be9f1d71ab9)

r16141: Dummy commit to make the build farm re-test against Samba4 16140
(This used to be commit a1fcacf75683e4c08236bb4cc4164678ea1a1ce4)

r16138: By default, rootsbindir relies on SBINDIR so import it here
(This used to be commit a18af427d7ea7b5a04d7006d10299d9e43317ae4)

r16136: By default, rootsbindir relies on SBINDIR so import it here
(This used to be commit 5ffb424641bfc71ce396800a6e426015b11fd6ba)

r16131: fix RedHat 9 spec files
(This used to be commit 1ffb0f06adeb1658fa1f5eed747841c6ff1e5665)

r16130: fix a shell portability issue for Solaris sh in installman.sh
(This used to be commit 9be3e07884e7f8121bc64766e7d036495cd94a15)

r16126: Janitor for Volker..... This looks correct to me.
Jeremy.

Fix a parsing error that became apparent in 'make test': If we have no
group
rids in the info3 we set the array buffer pointer to "1" but fail to
actually
ship the array.

Volker.
(This used to be commit ee1b9207d3119c2b3e7c1c4e59250dbd323eae6a)

r16122: As we use 'inetOrgPerson' as structural objectclass for new accounts for
eDir, we already add 'sn' as required attribute on LDAP add
operations.

When we modify an entry, we need to request 'sn' as well in our
attribute lists, so that we don't try to add it a second time.

Guenther
(This used to be commit e018ea3d1d70e68e6802843f88b3245dc09244c9)

r16121: Fix a eDir related memory leak.

Guenther
(This used to be commit 322f1664df553d95fcdfc24f19bd7f34ce9b834b)

r16117: Make winbindd work again in security=ads.

We still used the old HOST/* UPN to get e.g. users, now we need
samaccountname$@REA.LM.

Guenther
(This used to be commit f6516a799aec2db819f79b9a1e641637422a9b4c)

r16116: Hoist the slow CLOCK_REALTIME message inside the branch so we never
confuse an uninitialised __profile_clock with CLOCK_REALTIME. Flip the
condition argument to SMB_WARN around so that it's correct (though
completely non-intuitive).
(This used to be commit 60b5f9618b77afb397f96c1ff406c8cd3a4648f3)

r16115: Make "net ads changetrustpw" work again.

(adapt to the new UPN/SPN scheme).

Guenther
(This used to be commit 8fc70d0df0c93c29b49f924bac9ff5d9857cfd9d)

r16114: Make winbindd's group enumeration (set|get|endgrent) work again (when
enabled).

Do not bail out when a group just has 0 members.

Jeremy, please check, this has been removed with r13915.

Guenther
(This used to be commit 3a738a855d335e44e167351e6396bf3fe81a03af)

r16112: Added Volker's fix for extra odd byte at the end
of UNIX_BASIC infolevel. Checked client implementations
(cifsfs and libsmb) and they ignore it. Thanks Volker.
Jeremy.
(This used to be commit 6cd1cb5c9deba2aa0d7a3251c8142834cbaba6d1)

r16111: Patch from Björn JACKE <samba@j3e.de>.

This fixes a problem where the clock definition for clock_gettime() is
present at compile time, but is not available on the running system. In
this case, we fall back to less-preferred clocks until we find one that
we can use.
(This used to be commit fc6ed6a1aa2225ccde04c4ecaf0777dc0de4f1cb)

r16103: Properly rebuild timelimit on systems with executable extensions.

Paul
(This used to be commit f5be3e12e2dc42a2092b71c9e4cab6f5d0ef8860)

r16080: Re-add accidentially excluded in-forest domain trusts (fixes bug #3823).

Guenther
(This used to be commit 8759a00fedfe5d8d789c8b707c924d8116da1102)

r16079: How embarassing. :-(  Broke the build.

I will *always* compile before checking in...
I will *always* compile before checking in...
I will *always* compile before checking in...
I will *always* compile before checking in...
I will *always* compile before checking in...
(This used to be commit f3bd1c8c7de93917787e7341f759699c8399f006)

r16078: Patch from Aleksey Fedoseev <fedoseev@ru.ibm.com> to correctly
set the type of 'acl compatibility from string to integer
after the changes in r14207.
(This used to be commit b01f289f1a788fe8f62a213b67a73a9bfee01bb1)

r16076: Fix for machine password timeout overflow from Shlomi Yaakobovich
<Shlomi@exanet.com>.
Jeremy.
(This used to be commit 5cd234a1fff1e9d025eea6600649e56c997eafc2)

r16065: Re-add a strlower_m(account) in samr_create_user that was dropped for no
reason but to increase fidelity with W2k3. Tom Bork has raised valid concerns
that Unix scripts might rely on the account names being lower-case, so keep
that. We might later decide to only lower-case the unix name passed to
'add [user|group] script' but keep the passdb entry upper-case. But there are
enough user-visible changes in 3_0 already so that we should push this off to
a later date.

Tom, waiting for more bug reports from you ;-))

Thanks for insisting!

Volker
(This used to be commit bc78cca290559c5ca7623b9f6d9933e32668b9c4)

r16064: Bug fix for another one Tom Bork has reported:

'valid users = +unixgroup' failed with smbpasswd if 'unixgroup' has a
(non-algorithmic) group mapping.

Thanks a lot!

People out there listening, please test current code, this release is

   **BIG**

:-)

Volker
(This used to be commit 8f9ba5f96c9b506623ef97b7ed3d84f39d914a3c)

r16060: This is one of the more dirty patches I've put in lately. Parse enough of
SetUserInfo level 25 to survive the join method XP uses if the user did not
exist before. For good taste this contains way too much cut&paste, but for a
real fix there is just not enough time.

Up to 3.0.22 we completely ignored that a full level 21 is being sent together
with level 25, but we got away with that because on creation we did not set
the "disabled" flag on the workstation account. Now we correctly follow W2k3
in this regard, and we end up with a disabled workstation after join.

Man, I hate rpc_parse/. The correct fix would be to import PIDL generated samr
parsing, but this is would probably be a bit too much for .23...

Thanks to Tom Bork for finding this one.

Volker
(This used to be commit 5a37aba10551456042266443cc0a92f28f8c3d0d)

r16057: Coalesce the DMAPI configure tests into a single macro. Add
a more specific probe to try and eliminate old, incompatible
DMAPI implementations provided by IRIX 6.4 and AIX 4.3.
(This used to be commit aafd4db457ce8a60c628d54a3ace3b97c8885dca)

r16054: Janitor for Volker ? Volker - some reason you didn't
fix this in 3.0 ?
Jeremy.

We had no way to return NT_STATUS_OK from the netlogon serverpwset,
although
we successfully set the machine password...

One thing the samba3 join test found.

Volker
(This used to be commit e5b7acc9b5cb6e8cf3d03c9d392fad06e0d282d9)

r16047: Remove unnecessary line, as this value is set in either branch of the
'if' below.  Spotted by Aleksey Fedoseev.
(This used to be commit 00eb42791abf3e34dc42a2067f07b8549ea31abb)

r16046: Hmmm. I don't fully understand popt, but this is necessary for smbclient to
show a prompt again :-)

James, could you check that this is ok? I think this was part of your Revision
15848.

Thanks,

Volker
(This used to be commit 91534146b4718206f0e2a7f456154a0497277dfd)

r16025: make sure the samba4's smbtorture can be executed

metze
(This used to be commit bed220afb816138a3e806c5d20e812667f6994ae)

r16019: This should not be a level zero message - it's harmless
and can happen though misconfiguration.
Jeremy.
(This used to be commit 4b9cf399a691ba4a7392caca558d0e98b4d19104)

r16017: Add Samba4 replacement for timegm to work
on Solaris.
Jeremy.
(This used to be commit b65c6468651282879a39fa8029c190f2d1a91a28)

r16016: Add debug to be symetrical with reading from
cache.
Jeremy.
(This used to be commit da26565a2e85dc36b283f6b81378a706f3ae5f26)

r16014: Correctly set the group RID in init_sam_from_buffer.

BIG THANKS to Tom Bork for reporting that Bug!

Volker
(This used to be commit 40339fdcced67d62e449ba6f19329d89c808e139)

r15997: Fix bug in OS/2 Warp - it doesn't set the ff_last
offset correctly when doing info level 1 directory
scans. Thanks to Guenter Kukkukk <Guenter.Kukkukk@kukkukk.com>
for reporting this problem and testing the fix.
Jeremy.
(This used to be commit 65d4dfbd6045a4e3f9eaf520c70ef29ff7ddee82)

r15995: Make smbldap obey config tests. Patch from "John E. Malmberg"
<wb8tyw@qsl.net>.
Jeremy.
(This used to be commit 93dd476a9108adde42de422e1886bbaee75b952b)

r15989: try to make timelimit.c more portable

Patch from Bjoern Jacke (Thanks!)

metze
(This used to be commit b9d1440f46f443851607300e31b123d6afa81c06)

r15985: Adding "own-domain" switch to wbinfo which is handy from time to time.

Guenther
(This used to be commit 3c9416c2bedeec7f075e94d45d08f37ae6dd41d1)

r15984: Correctly handle the case when there is no configuration file for
pam_winbind.

Guenther
(This used to be commit 29758ea1c4e1b9b57d27765d539306058299fcd1)

r15983: Honour the krb5 principal name change (of the new ads join code) in the
kerberized winbind pam_auth.

Guenther
(This used to be commit 216125fe132fa6b886b99139e38988725beb88f0)

r15982: Fix confusing order of DEBUG statements in winbindds pam_auth.

Guenther
(This used to be commit 3f5a2e49c108bfe8f8b875af9e69d5ad3b0567ee)

r15980: Correctly destroy talloc_ctx when the LDAP posix attribute query has
failed. Noticed by Bob Gautier.

Guenther
(This used to be commit 7327f94546a90df25c688dcafd42e0993133057a)

r15977: Fillup the password_policy method in winbindd for winbindd_passdb. This
should make pam_winbind work again on a Samba PDC (and fix Bug #3800).

Guenther
(This used to be commit 4addabd054a2627133d3fff71234db18cf2c822c)

r15976: Set our internal domains to "online" by default in winbindd.

Guenther
(This used to be commit 2678582c6cc7fb100cb3bfd867816878461ae7b4)

r15975: Only call the printer publishing calls if 'security = ads'
(prevent a segv)
(This used to be commit a2ef525d9e3b4f050cb4e02fad67808d3e916373)

r15971: Obey the manpage description and make changesecretpw accept a password via stdin
(This used to be commit 60d4aabc3205aa80f8d49c3c2db95927c61a81a5)

r15967: enable RAW-OPLOCK in samba3 make test

metze
(This used to be commit 6d35268639a7796583f5cc7bee12cf884a207af2)

r15965: Pam modules install fix, thanks to Ryan Novosielski for spotting this one
(This used to be commit 8f5b198acaca5ce14ab1098d86f34df9a67619d4)

r15960: Don't double-free (ensure we always free file_fsp in
the same place on error - I missed the extra free in
delay_for_oplocks).
Jeremy.
(This used to be commit 146a2648156563b41642d42c46aff401e435a42b)

r15959: Ooops. Use the right file_free call...
Jeremy.
(This used to be commit 1117a327bd4fb425d25ea63df782095701fb83bb)

r15958: Make us pass RAW-OPLOCK with kernel oplocks off.
This allows a requestor to set FORCE_OPLOCK_BREAK_TO_NONE
to ensure we don't break to level 2. Fixed a couple
of resource leaks in error paths in open_file_ntcreatex.
Jeremy.
(This used to be commit c7c9adcce7f13d01445f31b07fb28a76f0a1d6df)

r15951: oplock_request must be an int, not a BOOL. We
were getting away with mixing types. Not cool.
Jeremy.
(This used to be commit ad3bc112a21afd3a04449ec3f604e017d753c224)

r15949: Patch for bug #3308 to stop us returning duplicate
mid replies on path based set-eof trans2 calls.
Needs modification for HEAD (as in head open_file_ntcreateX
properly returns NTSTATUS - I'll fix this tomorrow my
time). Secondly it still fails the Samba4 RAW-OPLOCK
smbtorture because of an interesting case. Our oplock
code always returns "break to level 2" if it can.
In this case (path-based set-eof or set-allocation size
on an exclusive oplocked file) W2K3 always sends a
break-to-none. We send the break to none (from level2)
after we've done the write for eof or allocation size.
I need to work out some way of telling our break code
to always break to none (might need to extend the message
field).
Jeremy.
(This used to be commit ad9895c654f400e242adcd4099f7cd004521ee92)

r15943: Update misleading DEBUG statements and comments that refer to
open_file_shared.
(This used to be commit 784126edff942d1c2f79f7c9feb59980426f331e)

r15936: Fix memory leak on error exit path. Trying to
track down #3308.
Jeremy.
(This used to be commit e39c11c91153c1ed547bc635d02769a1cdf3ecc0)

r15928: When we search for clock_gettime, we might find it in librt, but librt
might pull in libpthread. This is quite bad, firstly because it can
cause oplock signals on Linux to go wonky, and secondly because merely
linking with pthreads can cause performance degradations due to implicit
locking requirements.

The solution is to only search for clock_gettime if --with-profiling-data
was specified. If we do end up searching for it, then we test whether
linking with librt pulled in libpthread, and we only allow the definition
for clock_gettime to succeed if libpthread was NOT linked in.

Problem reported by Thomas Bork and diagnosed by Volker Lendecke.
(This used to be commit 5712644fcc00939b9101b2e3143674d2cbdf1e12)

r15924: Ok, the Solaris fix broke AIX. Next try :-)

Volker
(This used to be commit 53bd3fa30ad1f8bada4a49e4c79ea7ff70188174)

r15918: Attempt to bring Solaris into make test. The Solaris shell does not like the
isolated !.

Metze, can you check this?

Thanks,

Volker
(This used to be commit c9542c64ce2a21671b8f4e30780e8a6e07ab6632)

r15917: Activate RPC-AUTHCONTEXT in "make test"
(This used to be commit 17f701ab4f46502ca7a957d3e79b005a0acebdc1)

r15911: Make us survive rpc-authcontext committed next
(This used to be commit c24bfdce625782637b5f4d11a5117ef795ddfc2f)

r15910: vfs_full_audit does not need current_user
(This used to be commit 09f3c7a86f931371dc6f886edccc15d3f5437d34)

r15909: Implement recycle:subdir_mode
(This used to be commit 4dd8694a250e4d064a790fe8f422c965ab533880)

r15906: smbpasswd help text for -W option (patch from Aruna Prabakar <aruna.prabakar@hp.com>
(This used to be commit 0a81af4fef5b794fea257d9e3e11b16c3a4de12f)

r15904: This does two things:

Fix more potential segfaults when something on our way to a DC connection
fails.

We can not continue if dcip_to_name() fails. With

192.168.234.100 nt4pdc
192.168.234.100 windows#1c
192.168.234.100 windows#1b

in the lmhosts file when nt4pdc is rebooted, we do find the DC's IP address,
we can connect to TCP 139 while it is booting but anything else fails. So we
fall back to put the IP address into domain->dcname. When the DC is fully up
later on we try to do the auth2 against \\192.168.234.100 which gives
INVALID_COMPUTER_NAME. And we never get out of this loop again.

Fix this.

Jerry, maybe you can take a look.

Thanks,

Volker
(This used to be commit b1244e79068af9e287252b2dfbb8d612e717674a)

r15895: Ensure all new rid allocation goes through
the same function (deals with races).
Jeremy.
(This used to be commit 4962548dfe8ec2854e209217066556f339d3186e)

r15890: Use correct enum type (bug #3722) from Jason Mader <jason@ncac.gwu.edu>.
Jeremy.
(This used to be commit a8eb1186a12b44178a28b013373f6f8a4356d9ec)

r15888: Fix bug #3804 from jason@ncac.gwu.edu
Invalid comparisons.
Jeremy.
(This used to be commit 9890a31c5f4a8911b0f56eee67cfbcc46f15ee43)

r15887: Ensure we use sys_write so we're not interrupted.
Jeremy.
(This used to be commit c66620770d2154543a6ec99d369771b339df5463)

r15884: Fix bug #3803 from jason@ncac.gwu.edu - bad comparison.
Jeremy.
(This used to be commit 1d9dbe3b662ea757224ca4d8c991ae73bb92ae8d)

r15873: Use short signal names to placate ksh trap.  bash seems to accept either
and even uses them on the manpage.  this should now enable make test on
AIX.
(This used to be commit de836d2077e53c07a50c51d1580f8e93700eb5bd)

r15872: Take one step toward getting this working on AIX.  the [-z ] stuff
doesn't work there.
(This used to be commit 0ee78d96d9bd039084a270172316d80df4c00248)

r15864: build fixes for IRIX 6.4 in the build farm; only enable the C++ reserved word check when we selecte --enable-developer
(This used to be commit ece5fe3e78da460fb1c0eca3185da7f80c5cfdc4)

r15863: trying to fix a compile issue on HP-UX 11.x caused by conflicts of int16 and int32 definitions in internal and system headers
(This used to be commit 094443ed6c0fdd06f459211dbca50feed58463ef)

r15848: Introduce commandline options to set the remainder of the parameters in
dynconfig.c. This is mainly useful for test harness scripts, hence the
lack of short options.
(This used to be commit bf3b71c84595608d71e0f15a6158adacb295518e)

r15845: Ok. This was a tough one. If for some reason the tconX fails towards a domain
controller the next time we connect this child ran into a segfault because it
tried to reference a half-baked connection.

Volker
(This used to be commit c8a8204c744cf7aa1a1a6992a3433d99b6bb73a1)

r15842: patch from volker to instruct winbindd to find a trusted DC on its own when runing on a Samba DC (since we don't implement the getdcname() call that well
(This used to be commit 39f7ff75a7a21b85b54cba954f1c5552e562be5c)

r15838: Back-port tridge's talloc fixes (r15824, r15828) from Samba4.
Jeremy.
(This used to be commit f6c110ddb8cfaa1a57dea52818e7611134c2dcfe)

r15822: Add suggestion made by Ralf Haferkamp.
(This used to be commit 7c375fd540fa54ac8ae71c42ed07e01c593044b3)

r15817: Remove some unused code
(This used to be commit 72f103708d17aa86e09fa7a02699f969f7ee9190)

r15755: Fix Coverity bug # 294. Apparently password can be NULL, but cli_session_setup
derefences it.

Volker
(This used to be commit b013b6908d22cfd38fcc56a9cb2ca675d75996d1)

r15735: also export PREFIX as testit from test_functions.sh needs it

metze
(This used to be commit 8709bba24a2359aff14920b1dab6b51f21401149)

r15732: samba3 now passes BASE-DENY1

metze
(This used to be commit e5a8553a57e0b1463a421ba69631f234184404b0)

r15729: Second part of Aleksey Fedoseev <fedoseev@ru.ibm.com> patch.
Jeremy.
(This used to be commit 5137006233a0c2de94ce3da7340ff0032ef7426d)

r15727: Fix msgtest - noticed by Aleksey Fedoseev <fedoseev@ru.ibm.com>.
Jeremy.
(This used to be commit 9f5c95c41872deac5e029b69daa5004d5c0fb4cc)

r15717: BUG 3711: patch from    Leonid Kabanov <lkabanov@mail.ru> for some solaris sheel scripting bugs with 'make test'
(This used to be commit 94e0bc50135646039a2ebcf5c922a82cc0c4a2e7)

r15712: BUG 3435: patch from volker to fix 'msdfs root = yes' in [homes]
(This used to be commit 466478f07e6233b89f442660ad42ef7ee870ad48)