r16628: Fix bug #3880, reported by jason@ncac.gwu.edu
by ensuring we return the correct enum for sid
type, not a uint32.
Jeremy.
(This used to be commit 98a5e20ff4ceacda65dcc0ce5498ed4ffde520f8)

r16626: Fix bug #3878. Reported by jason@ncac.gwu.edu.
Jeremy.
(This used to be commit 4c3019eb99d0a18a33ef1fa90d01b9c99c0b25c3)

r16624: Fix bug #3877, reported by jason@ncac.gwu.edu
Jeremy.
(This used to be commit 1f52b8b40619679242da663f5e5e7836d547f0a2)

r16616: Klocwork #2025. Stop null deref. I actually don't
think this can happen in real life but the code is
too complicated to be sure....
Jerry please merge this for 3.0.23.
Jeremy.
(This used to be commit 1e5042d4c0d1a0d0a5cfbcb0d47815e1510ee52a)

r16614: Klocwork #2012. memleak on error path.
Jeremy.
(This used to be commit 58b9adb849854610e7167e8aa02a02bd15b0bf00)

r16612: Klocwork fix #2011. memleak on error path.
Jeremy.
(This used to be commit b4e9475d2ac65f72cab0d5c8276da27cf1aeb791)

r16610: Subtle one from Klocwork #2076. If multiple flags
are set in a winbindd request it might overwrite existing
state->response.extra_data.data values without freeing.
Jeremy.
(This used to be commit 4e7262c81ad2945048cb8d0789af032a05008988)

r16608: Klocwork #2035. Null deref (bit of a stretch... :-).
Jeremy.
(This used to be commit 7d7a98208b1b514da60486fcbbb7b6d04df4ffac)

r16606: Klocwork #1990. Malloc the correct size.
Jeremy.
(This used to be commit d1a1c4e092877a6ea0f98eed2a37a96d42c36323)

r16603: Klockwork #2028. Fix null deref on error path.
Jeremy.
(This used to be commit 067feef34388e93ded8db7fd3b4a6a4b752a3059)

r16601: Klocwork #2038. Fix memleak on error path.
Jeremy.
(This used to be commit 934dddb2fa9fb60a87c0b0be81db97f2b59c7cb0)

r16599: Make it clear to Klocwork we're not dereferencing. Issue #2026.
Jeremy.
(This used to be commit 9402bf0d4cc6d04283ed69a6dedac7767df84626)

r16597: Klocwork #2006. Fix possible null deref.
Jeremy.
(This used to be commit 9b73385d6b90c7806d8ccfc1f2354ede761fad61)

r16595: Klocwork #2067. Fix possible memleak on error exit.
Jeremy.
(This used to be commit 1d21a3dec9ea061ce900ad1223f7c2a43c064600)

r16593: Make the invarient explicit to Klocwork. Bug #2023.
Jeremy.
(This used to be commit 720a9172050beb158a25fbb826676ba19cdcef20)

r16591: Belt and braces approach to shut Klocwork up - bug #2001.
Jeremy.
(This used to be commit d5c1028498de0346b7a35cc132b8081e04e639cc)

r16589: Fix Klocwork #1999. Although it should be impossible to
get duplicate OID's returned in the oids_out list it is
still good programming practice to clear out a malloc'ed
string before re-writing it (especially in a loop).
Jeremy
(This used to be commit ae02c05bfca46eb6a8ba25b124c18a358a759cb5)

r16587: Fix Klocwork #1987. Memleak on reusing x_dbf.
Jeremy.
(This used to be commit 16b0617cb0f91fbe1ce53dcb601fe5ed9e51bddf)

r16585: Fix Klocwork #1976. Possible null deref.
Jeremy.
(This used to be commit a912ef0d9724373966e77f9e8928be462de8df4a)

r16582: Fix Klocwork #1997 and all generic class of problems
where we don't correctly check the return from memdup.
Jeremy.
(This used to be commit ce14daf51c7ee2f9c68c77f7f4674e6f0e35c9ca)

r16581: Fix Klocwork #2017. Possible null deref.
Jeremy.
(This used to be commit 6967fd4cefa84a7b7b5e14467bfa8152907d55c9)

r16579: Fix Klocwork #2016. Possible null deref.
Jeremy.
(This used to be commit f6d5bae4a105eee1b1d5b1aaa70a675705345d9e)

r16576: Fix Klocwork #2015. Possible null deref.
Jeremy.
(This used to be commit 9cbfaf62a3c4bb7d2e594e412449506ab0af4063)

r16570: Corrected the copyright notice.  I had requested and received permission
from my management at the University to release under my own copyright.
My mistake for entering the wrong info.
(This used to be commit c65ebeb02810fb4039555c55779ec62a4a8de564)

r16554: Sorry, just had to change this. Don't use int when
you're passing a BOOL parameter, don't use "clever"
code in while statement - make things easier and
clearer to understand when triggering something
with an if.
Jeremy.
(This used to be commit b1fc2d8b99e0402c0e8fe954d9f9563dc4dc2812)

r16552: Fix bug 3849.

Added a next_token_no_ltrim() function which does not strip leading separator
characters.  The new function is used only where really necessary, even though
it could reasonably be used in many more places, to avoid superfluous code
changes.

Derrell
(This used to be commit d90061aa933f7d8c81973918657dd72cbc88bab5)

r16550: Fix bug 3866.  Thanks for the report!

Although I've never met a computer or compiler that produced pointers to
functions which are a different size than pointers to data, I suppose they
probably exist.  Assigning a pointer to a function is technically illegal in C
anyway.

Change casts of the option_value based on the option_name to use of variable
argument lists.

For binary compatibility, I've maintained but deprecated the old behavior of
debug_stderr (which expected to be passed a NULL or non-NULL pointer) and
added a new option debug_to_stderr which properly expects a boolean (int)
parameter.

Derrell
(This used to be commit c1b4c510530ca3118d1eccb9615a8cad732c7373)

r16548: Fix bug #3867 reported by jason@ncac.gwu.edu.
Jeremy.
(This used to be commit 2b8d72f09c6d79d5d41421545b538da573adcd9d)

r16546: Fix bug #3865 reported by jason@ncac.gwu.edu.
Jeremy.
(This used to be commit be888a3cda637149c3183418c6e721c6fef1b652)

r16544: Fix bug #3864 reported by jason@ncac.gwu.edu.
Jeremy.
(This used to be commit 16e42b446bea171c3ad848aefaa92c7404aade42)

r16542: Fix #3863, reported by jason@ncac.gwu.edu
Jeremy.
(This used to be commit cde8323fdc4d4ddaa30e8c59bec89dc130fe26a6)

r16541: Fix #3862 reported by jason@ncac.gwu.edu.
Jeremy.
(This used to be commit 09e11dcb2304eec9656e76c24921c82f4a870914)

r16537: Fix for bug #3858, all files in a directory not
being deleted when hide unreadable set to true.

Here's the scoop.

This one is really interesting. The pattern of deleting a directory is to do a
findfirst to get the first part of the list, then for each name returned it
does a open/set delete on close/close -> thus deleting the file. Then it does a
findnext with the last file name THAT IT JUST DELETED ! Now we can handle this
in the findnext in the case where hide unreadable is set to false as we look
back in our cache of names and just seek to the right point. The bug is
actually fixed in the first hunk of this patch - the one that removes the
is_visible_file() check after SearchDir returns false. We don't actually need
it and in this case it's causing the delete to be aborted because it can't find
the name (doh ! it was just deleted). We don't need it as SearchDir is only
ever called from findnext, and findnext should only ever be returning names we
gave it.

The rest of the patch are the debugs I used to find
the problem but they're generically useful.

Phew - that one took a while to track down.....

Jerry, please merge for 3.0.23 final.

Jeremy.
(This used to be commit cd048cb775f0a8525fc19aa463db07c477521f5b)

r16528: Self-confidence is plummeting.  How could one person goof
the same change 3 times......

Fix pam_winbind compilation again.
(This used to be commit f3d2bbf0d222794bbb39b51a0f19167c6e8add1a)

r16521: Fix different extended_dn handling for win2k/2k3 (thanks to Frederic
Brin at Novell).

Guenther
(This used to be commit f19f781e15da98cfaa94c71fc455fe095dac7b4d)

r16510: fix from Nicholas Brealey <nick@brealey.org> for building pam_winbind.so
(This used to be commit 4490fbdfc27eab02bf0140d3ffb70402858423e1)

r16490: Fix a memleak and two typos
(This used to be commit 8cf364e602eea408fd9cd6acd12f2b72971361ae)

r16486: fix RedHat init scripts to rely on OS provided killproc() function
(This used to be commit 164cdf69f395236ffef9ecc202543d53b5eda2da)

r16484: Bugzilla 3805. Only define _ABS if it is not already defined.
(This used to be commit 09fb20f4fe26a5ec82e2f296bf033ab0864af669)

r16483: Turn off some useless MIPSPro warnings be default.
(This used to be commit a22bf28bcab6ef3c4addaf57fdeb2020625ead0e)

r16482: Fix bugzilla 3801 by forcing MIPSPro warning 1035 to be an error.
(This used to be commit 2131ddd7a13f078be471d327c8d73300facf8def)

r16480: (Ugly) workaround before the set_dc_type_flags & friends cleanup:

When trying to login using krb5 with a trusted domain account, we
need to make sure that our and the remote domain are AD.

Guenther
(This used to be commit 5853525f111c0ab6a97b081d5964f778e7c36565)

r16479: When dcip_to_name failed to get the name of the ip in saf_servername we
cannot put saf_name in the failed conn cache as it's uninitialized.
Store saf_servername (the ip) in that case.

Volker, please check.

Guenther
(This used to be commit 098a87f492f69caeb523478a7ebcd0e3f636497d)

r16478: small cleanup to make pam_winbind.so build and iniparser
include directory like other parts of the build files.
(This used to be commit 0fca41efde36d535a274b983e032c1def33f081e)

r16475: destroy talloc ctx when we weren't able to collect onlinestatus
messages.

Guenther
(This used to be commit d6b52e818109e6eb5a3df1bbc127c333e819141d)

r16474: There is no point in figuring out lockout policies if we do not allow
offline logons at all.

Guenther
(This used to be commit dfbe555c69b3272bcff1d76a699aae2bdb85bdaf)

r16473: There is no point in calling set_dc_type_and_flags() before each
pam_auth login (when using kerberos).

Guenther
(This used to be commit 520777f7946e55b1437df138e529fdc053362d16)

r16471: Bug reported by Vitaly Protsko <villy@sft.ru> in 3.0.23rc1.
Add missing automatic add of the Administrators SID in the absence
of winbindd and precense of Domain Admins SID in the user's token.
(This used to be commit ce7846d6f19f63ca99179b75e6f2195cc593795f)

r16470: remove ntlm_auth_proto.h and passwd_proto.h in 'make clean
(This used to be commit 8421d1b080dd65d1459cc66f0bd4e20fed5d0385)

r16458: Increase debuglevel of cli_rpc_pipe_close().

Guenther
(This used to be commit 840ac23ec007df445892d851144d6458c4e06a6b)

r16453: Fix another memleak.

Guenther
(This used to be commit 49fb1a3ebc44602302c347195752891bf28c7037)

r16452: Fix memleak in the CLDAP processing (found by valgrind).

Guenther
(This used to be commit 479dec68459df606ff566ac86eb3b4bbbd2ca77a)

r16435: Add in the uid info that Jerry needs into the
share_mode struct. Allows us to know the unix
uid of the opener of the file/directory. Needed
for info level queries on open files.
Jeremy.
(This used to be commit d929323d6f513902381369d77bcd7b714346d713)

r16433: Fix Coverity #300 (triggered by a Klockwork bugfix
I think). If a alloc fails just return NT_STATUS_NO_MEMORY,
don't go to "done" label and deref pointers.
Jeremy.
(This used to be commit 490c7c84674860ecd9daa24341edb427b9fe0aa5)

r16429: Fix final 4 Klocwork bugs we're going to fix before
release - #785, #786, #787, #788.
Jeremy.
(This used to be commit 9017547cccadeecb80f3db58a43838dc656fce2f)

r16427: Fix bug # 3848. Thanks to Wilco Baan Hofman for testing the release candidate!

Volker
(This used to be commit adf2dcce09ae29a8c1677b25e1cd5f022b804d01)

r16426: Klocwork #1544, #1545, #1546, #1549, #1550, #1552, #1553, #1554
Jeremy.
(This used to be commit e71cc6647a2eaba0eac95b6abb40745e45db72a4)

r16424: Fix possible null deref and a memory leak found by
examining Klockwork #1519. get_printer_subkeys()
could return zero without initializing it's return
pointer arg. Fixed this. Added free of subkey pointer
return in registry/reg_printing.c (interesting that
neithe Coverity or Klocwork found this one).
Jeremy.
(This used to be commit 4fbeae1a3ac3499e5d9f566655cbafccd9d691cb)

r16422: winbindd_demote_client isn't used and generates
a Klocwork issue (#1844). Remove it
Jeremy.
(This used to be commit e83c3e0a65edeb423d964488e219e30d023b13e8)

r16420: Fix Klocwork #1674. Null deref.
Jeremy.
(This used to be commit f5dddf339ee3a867e21f34a81bd0b33195b7397d)

r16415: Also delete object files from iniparser.

Guenther
(This used to be commit 4e1816855a8f86d1228c44b39126fbfb3cb893c0)

r16411: Fix compilation of vfs_afsacl, thanks to Greszler Szilard for trying
(This used to be commit 601643a4608cdccf33277cef8900c4da61712268)

r16409: Fix Klocwork ID's.

1177

In reg_perfcount.c: 1200 1202 1203 1204
In regfio.c: 1243 1245 1246 1247 1251

Jerry, the reg_perfcount and regfio.c ones, can you take a look please? This
is really your code, and I'm not sure I did the right thing to return an
error.

smbcacls.c: 1377
srv_eventlog_nt.c: 1415 1416 1417
srv_lsa_nt.c: 1420 1421
srv_netlog_nt.c: 1429
srv_samr_nt: 1458 1459 1460

Volker

Volker
(This used to be commit d6547d12b1c9f9454876665a5bdb010f46b9f5ff)

r16397: Fix Klocwork #11767 and drasticly simplify the
logic in smbd/process.c. All interested (Volker,
Jerry, James etc). PLEASE REVIEW THIS CHANGE.
The logic should be identical but *much* easier
to follow and change (and shouldn't confuse Klockwork :-).
Jeremy.
(This used to be commit d357f8b33594472ffa78d0a112accccc2a8b1fe7)

r16396: Klocwork #1170. Null deref.
Jeremy.
(This used to be commit 220627ab70977a9a26dc0e010f73195c2c87ec96)

r16393: Klocwork #1164. Null deref.
Jeremy.
(This used to be commit c9a4ce51ed85686a35f350f4681b95d68adebbd6)

r16392: Klockwork #1168. Protect against null deref.
Jeremy.
(This used to be commit cab256d72a91f59432e28e9623c10384d9ca2b5e)

r16390: Klocwork #604 - ensure if NULL pointer we assert
if len != 0.
Jeremy.
(This used to be commit e99cedfb0cabe3863797c8bd4594ee0826022d2e)

r16388: Klocwork #320. Null deref.
Jeremy.
(This used to be commit ceea8e21006bd6bae9e203a672f82e4d066bba28)

r16386: Klockwork #lots. Ensure talloc_get_type_abort aborts
if handed a NULL pointer, not returns NULL.
Jeremy.
(This used to be commit d47ec4dc25bffa6f605c0f6fa1d9c046dbc520a7)

r16383: Klocwork #1086. No null deref.
Jeremy.
(This used to be commit 7b68a2acfc583d1559c02461cab13419185bcd74)

r16381: Now samu can't be null don't check it on set. Klockwork
related fix.
Jeremy.
(This used to be commit 2e5a88147826113f78ee0d8845c704692942cebf)

r16378: Klocwork #1079. Allow BUFFER5 to cope with a NULL
dependentfiles pointer.
Jeremy.
(This used to be commit 05c50d99a4b1d2bc11a83e07902082227d7c074a)

r16377: Klocwork #1077.
Jeremy.
(This used to be commit 72709a8465caf5b7d95911250234f203b0ff4675)

r16375: Klocwork #1670.
Jeremy.
(This used to be commit 99605ce296663b7697d737fd521f0e4d8436d1f2)

r16373: Fix Klocwork #1071.
Jeremy
(This used to be commit 080464b93d965a583b2b5f248a2bb8a7fb516e6f)

r16371: Fix Klocwork #1048.
Jeremy.
(This used to be commit 5057e90db32adf4395fc2a3fce26a7f64c780828)

r16369: Klocwork #1035.
Jeremy.
(This used to be commit 6924d1cd443e4f7eb7b634c97c92effd6373ff12)

r16367: Missed one, Klocwork #915.
Jeremy.
(This used to be commit b581fee9824712b9385e9975739a8ddbb297bca8)

r16365: Fix Klocwork #895, #898, #899, #915, #932, #938 and a
few other problems Klocwork missed.
Jeremy.
(This used to be commit fe05769a1a85f924c67be7e5dcee4871a86948d7)

r16363: Fix Klocwork ID 981 1652

Volker
(This used to be commit ce1d8423ef7cd86fc64200002fde707bca621d44)

r16362: That's the best way to fix a Klocwork bug: Remove the (unused...;-) code
altogether. This was essentially a second implementation if string_to_sid.

Klocwork ID 1038

Volker
(This used to be commit e35f901067b702f5e61733a6edabc610a7dd5aab)

r16361: Fix Klocwork ID 1731 1770 1771 1775 1796

Volker
(This used to be commit 8a5cebc19e4709399976efe9e3ba3bf29249620a)

r16360: Fix Klocwork ID 136 520 521 522 523 542 574 575 576 607

in net_rpc.c: 715 716 732 734 735 736 737 738 739 749

in net_rpc_audit.c: 754 755 756

in net_rpc_join.c: 757

in net_rpc_registry: 766 767

in net_rpc_samsync.c: 771 773

in net_sam.c: 797 798

Volker
(This used to be commit 3df0bf7d6050fd7c9ace72487d4f74d92e30a584)

r16358: ALWAYS compile this stuff on a 64-bit box before
checking in. size_t != uint32 on a 64-bit machine.
Jeremy.
(This used to be commit 09c89732869eae0d8c8971ac78235d34e4dcecb9)

r16356: Helping derrell out. Jeremy.

Only set the DFS capability flag if the share is a DFS root.  Fixes bug 3814.
(This used to be commit 6193f1170819096ea8a646e5a456f627df83872d)

r16355: Remove derrell-test created by mistake.
Jeremy.
(This used to be commit 1dea2ab8562696e4435de2de46877d821b3eb232)

r16354:  r14595@cabra:  derrell | 2006-05-09 00:00:35 -0400
 Made a trivial change to the file so I can have one more check-in
(This used to be commit 9b562e919a4a369d7767a444229487a83ae9f6f5)

r16353:  r14594@cabra:  derrell | 2006-05-08 23:59:42 -0400
 Added first draft of README file, testing SVK's --patch facility
(This used to be commit d11238ecf8ef63b967648ce54e98e57858be95f7)

r16350: Fix the build.

GUenther
(This used to be commit 3203ce3b49e6f21ed690e9d7393e98419de54c27)

r16349: Another fix to make winbind more robust in large domains:

We may only feed rpc_useraliases with chunks of 1024 entries.  This is
important as the token generation otherwise fails when a user is member
of more then 1024 groups.

Volker, please check.

Guenther
(This used to be commit d8fd94648f965eb043f957b154ce63b245a90328)

r16346: Allow to exit a "net rpc shell" with 'quit' or 'exit'.

Guenther
(This used to be commit 232566e1cb1eaaa6ae09e9de8a8a82d91acdfda7)

r16345: Improve the chance that our users can discover one of the coolest 'net'
features.

Guenther
(This used to be commit 446d79a0007d3d99c73eb758216f18f64036f11d)

r16344: Allow to set passwords directly when creating users via "net rpc user
add" (as the documentation says, and currently onle "net ads user add"
did). Fixes #3843.

Guenther
(This used to be commit 5d776d5fabded9b713080789aefc6058510b51f6)

r16340: Commit Bjoern's fix for the AIX PIC suffix. 'make test' still does the first
tests, and vfstest is still able to load modules. Looks like this is ok :-)

Volker
(This used to be commit c08bfdf87a1368a5d6c9a58ddb8ba379d969941d)

r16339: Fix Klocwork ID

277 278     (cmd_*)

485 487 488 (ldap.c)

Volker
(This used to be commit 5b1eba76b3ec5cb9b896a9a5641b4d83bdbdd4cf)

r16338: Fix Klocwork ID 150
(This used to be commit ded2952e00fec712ce612cf6aaabf85c1a6f4488)

r16337: Add configure fixes by Bjoern Jacke <samba@j3e.de>.

Volker
(This used to be commit 1b3a24732df19addb8c2dfaae99f5fbf5bbccc44)

r16336: Fix Klocwork ID's 1087, 1095, 1096, 1098, 1099, 1101, 1102, 1105, 1107, 1109,
1111

Volker
(This used to be commit def075baf873b24c522d09ddc0141d84431a1fae)

r16335: Fix Klocwork IDs 107 108 109 111 112 113 114 116 117 118 121
(This used to be commit 555984ea772730a5752905f1130e0bf6ec48207f)