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)

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

Volker
(This used to be commit d3f5acb16e14ec394f1af41fa2f9e27fdca937db)

r16326: Klocwork #509. Always check return allocs.
Jeremy.
(This used to be commit 7e397b534a5ca5809facf5aa84acbfb0b8c9a5b4)

r16324: Klocwork #499. Allways check results from alloc.
Jeremy.
(This used to be commit 2b69d436da7b2902ea419f3bcc45c7b5a5c571fb)

r16322: Klocwork #481., Don't deref null on malloc fail.
Jeremy.
(This used to be commit dd31f3fc0e044fdae139aefcb21773249c30eb74)

r16320: Ensure variable is not null before calling fclose.
Klocwork #412.
Jeremy.
(This used to be commit fed55d653a73d09550a90a4904023580dd46ae76)

r16317: Klocwork #300. Check for error return on malloc.
Jeremy.
(This used to be commit aa0d739834a286aeee88f76048dc347dd2852d09)

r16313: Not a problem - but ensure Klocwork is quiet (#872).
Jeremy.
(This used to be commit 22a345deed6caa5750e2bb233a20422ad7b90d94)

r16311: Check for NULL return. Klocwork #998.
Jeremy.
(This used to be commit c430730f5a0dc00626b9b924e88adfb84cd48869)

r16309: Fix Klocwork #1596. Check for NULL on talloc.
Jeremy.
(This used to be commit 34b6b6723b1c87e19b0a51ef785a614d86bd5cef)

r16307: Make sure we know we must pass a valid pointer here.
Klocwork #1129.
Jeremy.
(This used to be commit e8d86362ba8762a5e4180e7320f5ac8bb37c203d)

r16306: Error handling in this asn1 code *sucks*. Fix a generic
class of memory leak bugs on error found by Klocwork (#123).
Many of these functions didn't free allocated memory on
error exit.
Jeremy.
(This used to be commit 8ef11a7c6de74024b7d535d959db2d462662a86f)

r16301: Fix a memleak
(This used to be commit 21aaede518503e6722ba5ccfdb2c77007d12ddee)

r16299: Fix another "dereferencing type-punned pointer will break
strict-aliasing rules" gcc4 warning.
Jeremy.
(This used to be commit ee77dde9b6375423f95f31455d06cef7303400be)

r16298: On request of jiri sasek - Sun Microsystems - Prague Czech Republic <Jiri.Sasek@Sun.COM>
change priv_op and priv_info to names that don't
conflict with the solaris namespace.
Jeremy.
(This used to be commit db5b4e3f13bceeb2f078f412bdd4666516eedc5c)

r16295: Fix gcc4 "warning: dereferencing type-punned pointer will break
strict-aliasing rules" warning.
Jeremy.
(This used to be commit 7ac5aee88cae3400fb6400b3628544da2e474560)

r16294: Fix syntax error.

Credits to Andreas Schwab from SUSE Linux for this fix.
(This used to be commit da4c5c3b75e6fd4f9e661cbcbe7d38024a02d344)

r16290: Fix Coverity bugs 298 and 298
(This used to be commit b96808bb623c01f2515fdbdede8b4e9edebff23b)

r16289: Fix smbclient test hang. This is caused by readline refusing to believe
the end of file it receives when reading from a pipe after we force
interactive mode.
(This used to be commit 3eee12645883df21efa5175b10854a965fc90422)

r16287: Use intptr_t to return an integer of an unknown type cast
to void *.
Jeremy.
(This used to be commit e24361ecddef8a48a42a356775b93ce5c4027fae)

r16285: On a 64-bit box, size_t != uint32. Ensure we use
the right parameter type.
Jeremy.
(This used to be commit 938545f5352161b4fe195c2a826a26db5236f851)

r16284: Start fixing up gcc4 -O6 warnings on an x86_64 box. size_t != unsigned
int
in a format string.
Jeremy.
(This used to be commit face01ef01e1a3c96eae17c56cadf01020d4cb46)

r16281: Start fixing up gcc4 -O6 warnings on an x86_64 box. size_t != unsigned
int
in a format string.
Jeremy.
(This used to be commit c49ad9200bf3fdd8b9404d93182c2e04e662190a)

r16280: Start fixing up gcc4 -O6 warnings on an x86_64 box. size_t != unsigned
int
in a format string.
Jeremy.
(This used to be commit 45d5cad8c2438c032d9f11c62a50a04d0637bf6c)

r16277: Start fixing up gcc4 -O6 warnings on an x86_64 box. size_t != unsigned int
in a format string.
Jeremy.
(This used to be commit 3a15fff21988b426a514ad28060eaf0ddc17ef6b)

r16275: Start fixing up gcc4 -O6 warnings on an x86_64 box. size_t != unsigned int
in a format string.
Jeremy.
(This used to be commit 952547471f03bd2da1eda0247471b17aa1ff15de)

r16274: Fix the smbclient prompting behaviour for both systems that have
libreadline and those that don't. We always use the built-in readline
replacement for non-interactive mode. Interactive prompts are always
emitted to stdout and non-interactive mode never prompts at all.

Introduce x_fdup to avoid spuriously closing stdout when a logfile is
specified on the command line and setup_logging is called a second time.
(This used to be commit 848ac756f651a4be231e5635580c0fd5f3d3fa0e)

r16273: Add smbclient tests to verify that
        in interactive mode, smbclient always prompts on stdout
        in non-interactive mode, smbclient never prompts
(This used to be commit dff1eba81ed4bdd6dbf2df6e7c28b0da74a39615)

r16272: Fix memleak.

Guenther
(This used to be commit afdb1189029e01a132f16fea48624126ec65cd77)

r16270: Fix Klocwork #706 - ensure sscanf has correct format
specifier.
Jeremy.
(This used to be commit dc53d35b0a1491da94e231943a81547be4c75631)

r16269: Fix the build.

Guenther
(This used to be commit 546710d58c07acdaa175caa48cec4d3f2bc657ad)

r16268: Add TCP fallback for our implementation of the CHANGEPW kpasswd calls.

This patch is mainly based on the work of Todd Stecher
<tstecher@isilon.com> and has been reviewed by Jeremy.

I sucessfully tested and valgrinded it with MIT 1.4.3, 1.3.5, Heimdal
0.7.2 and 0.6.1rc3.

Guenther
(This used to be commit 535d03cbe8b021e9aa6d74b62d81b867c494c957)

r16267: Fix Klocwork #401, #402 - ensure format specifier
limited. Fix memleak in printing gencache contents.
Jeremy.
(This used to be commit 81731e1f68cdf4af80733338238aeae0a7d108c0)

r16261: Smaller fixes for net ads password.

Guenther
(This used to be commit 689ae22c80a890278610d9ada1eb4fa5e37bd5ce)

r16255: Fix 'net ads join' when the workgroup is set incorrectly
in smb.conf.  This did work before the join rewrite.
Samba will have problems if you try to run any of the daemons
with an incorrect workgroup but it should not fail to join.

The summary is that a member server should always use it's
own machine name when setting up schannel since that is
the only account it has.  Thanks to Volker for the discussion.
(This used to be commit 95763b94f709fe1ad9e381dbc6b364c2f3759024)

r16253: Fix another host/ UPN case in (the currently unused)
cli_rpc_pipe_open_krb5.

Guenther
(This used to be commit fa19099112490daa085bb310f2f4ed877bb22b40)

r16252: Fix Klocwork ID 1119, 1121.

Volker
(This used to be commit 678bbcf06109b276d3e4514c3788a9fb31348de0)

r16251: for i in `seq 1 1000`
do
echo "I will always compile before commit :-)"
done

Also fix Klokwork ID 806.

Volker
(This used to be commit 4974c598c00abc20cfb73eee12a7c49c279e0f54)

r16250: Fix Klokwork IDs 148, 151, 152, 154.

Volker
(This used to be commit d7a75ee94db009085165c062f73b68162a8b6da8)

r16249: Fix Klokwork ID 130
(This used to be commit 09586824f6568fb3305e3e59ba6bc8f5f632fb56)

r16248: Fix Coverity ID 297
(This used to be commit e56e3c19e1244e2b7409d57a030ca8b7ec446932)

r16247: Fix Coverity ID 296
(This used to be commit b82c95cb438b57bb8910e26657f8ffb590fe02df)

r16245: Cope with string being zero len. Klocwork bug #410.
Jeremy.
(This used to be commit afe2d4c49333309b940cec8408ccd37ed3747cb7)

r16243: Fix Klocwork bugs #581 and #706, ensure we check
the end of array first in the loop. Reformat to
modern standards.
Jeremy.
(This used to be commit 66275bd3bc56e67759dbabe77cb2ba019c6f4887)

r16241: Fix Klocwork #106 and others like it.
Make 2 important changes. pdb_get_methods()
returning NULL is a *fatal* error. Don't try
and cope with it just call smb_panic. This
removes a *lot* of pointless "if (!pdb)" handling
code. Secondly, ensure that if samu_init()
fails we *always* back out of a function. That
way we are never in a situation where the pdb_XXX()
functions need to start with a "if (sampass)"
test - this was just bad design, not defensive
programming.
Jeremy.
(This used to be commit a0d368197d6ae6777b7c2c3c6e970ab8ae7ca2ae)

r16230: Fix Klocwork #861 and others. localtime and asctime
can return NULL. Ensure we check all returns correctly.
Jeremy.
(This used to be commit 6c61dc8ed6d84f310ef391fb7700e93ef42c4afc)

r16222: Fix DEBUG statements.

Guenther
(This used to be commit 5ecfaf7d505e6acc23a06dd64d00f5e6fb8efe6f)

r16221: No need for friednly error messages at log level 10.

Guenther
(This used to be commit 58a7c0900325065cc969eb4f2f4c85d41e27bc89)

r16219: BUG 3836, 3837, 3004: compile warning fixes from Jason Mader.
(This used to be commit 6c1f1c091f5e87bf9464fe8ad7eb2cb683819a62)

r16216: Add debug messages to make it possible to try and
debug why a job pause or resume command is not being
done.
Jeremy.
(This used to be commit e6aacb1426bd04c4006f7be66228f9f8d9a7065d)

r16213: Fix Klocwork #852. iface_n_ip can potentially
return NULL. Ensure we don't deref.
Jeremy.
(This used to be commit c2f0ea2ff2f6ad925cee9c85110c6ad828ffb7a9)

r16209: Klocwork bug #66, ensure no null deref.
Jeremy.
(This used to be commit 79e693798cf322071ea64a4014a01ad9eaba73e8)

r16207: Ensure we don't allocate an OID string unless
we know we don't have an error. Klocwork #6.
Jeremy.
(This used to be commit 2c1a2d7b40e7ef353461f97f5c69c2079b5670ab)

r16206: Don Meyer <dlmeyer@uiuc.edu> pointed out that RHEL4 rpms use 'Epoch: 0' so work around that
(This used to be commit 304b7913cd25a024269fecdd541af06836816f68)

r16205: Fix 'make install' when srcdir != builddir
(This used to be commit 44d4bab991d5fc88005a542d2490ae3254a0f5b5)

r16204: Fix Klocwork # 14

localtime() can return NULL.

Volker
(This used to be commit 07c5dcb8633e6fadb596dc5a22d8d31b2e16a3ef)

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)