summaryrefslogtreecommitdiffstats
path: root/source3
Commit message (Collapse)AuthorAgeFilesLines
* r13975: Re-fix Coverity #156 - I had left the hidden arg. inconsistentJeremy Allison2007-10-103-5/+6
| | | | | | between Realloc and realloc_array. Jeremy. (This used to be commit 841c9b1847ae12656b827e3d35b8bf0c3f68b8b4)
* r13971: Fix Coverity bugs #209 - #213 - it detected mistakesJeremy Allison2007-10-101-56/+67
| | | | | | | | in the pointer aliasing once realloc could change a pointer. This was in the bugzilla.samba.org database as #687 but we never figured out what it was ! Jeremy. (This used to be commit 8d183441403524fe39e79af843d6cfe65898f7d3)
* r13968: fix typo, caught by GuentherJim McDonough2007-10-101-1/+1
| | | | (This used to be commit 217d3fbe7923115ae610a39e586ceb93df7683f5)
* r13965: Make sure we always reset the userAccountControl bits when re-joiningGünther Deschner2007-10-101-1/+1
| | | | | | | with an existing account. Guenther (This used to be commit e4c12ab167ee83772a2bdd1946b8d73613fc0d7e)
* r13958: Fix Coverity Bug # 141Volker Lendecke2007-10-101-1/+1
| | | | (This used to be commit 598513d1d3e23cc71ea0fd53230d393b6724b534)
* r13957: Based on patch from Richard Renard <richard.renard@idealx.com>:Jim McDonough2007-10-101-15/+18
| | | | | | Fix machine accounts (should not have valid shells) and users with no home directory (were getting previous user's directory). (This used to be commit f629f8a7b972f09fe959c68843b9cd5a03abfc76)
* r13955: Fix Coverity ID 139.Volker Lendecke2007-10-101-1/+1
| | | | | | | | Not a bug in the strictest sense, more a clarification. This whole routine assumes new_gid != NULL anyway, so there's no point in checking. Volker (This used to be commit dfbf09c772b9588271e2d8e053c7494bb087c544)
* r13951: Fix Coverity Bug #163.Volker Lendecke2007-10-101-28/+0
| | | | | | | This code was not used anyway :-) Volker (This used to be commit bbfb20569380529d60e3c61cd0be63a09eecfd17)
* r13950: Fix Coverity bug #168Volker Lendecke2007-10-101-0/+1
| | | | (This used to be commit cbf894c0e37964df57bd6a91ac10dfff571b1b3c)
* r13948: Fix the build. Remove rpcclient.h include.Jim McDonough2007-10-101-1/+0
| | | | (This used to be commit 6bf879bee3c59ba54b1b4c465c777e3dd0043f83)
* r13947: Use tabs instead of spaces for indention.Lars Müller2007-10-101-3/+3
| | | | (This used to be commit fb76390c6ae5928a2a222d61cbadf825611999ef)
* r13946: Link pam_smbpass with the required object files. Fix bug #3565.Lars Müller2007-10-101-1/+2
| | | | (This used to be commit 426c8fe0bdacbf218d4cf4a10af789afbc0e53f2)
* r13945: Move display_sec.c to lib/ (as suggested by Volker).Günther Deschner2007-10-102-4/+4
| | | | | Guenther (This used to be commit 7e80d5358eb181c3515acb732a3594e80391261b)
* r13916: Fix Coverity bug #29. Looks like my code. I wonder how much there is ↵Volker Lendecke2007-10-101-0/+1
| | | | | | | | | still lurking... Volker (This used to be commit 1345a52794f4f55173ed677af3d0714e88bf17c6)
* r13915: Fixed a very interesting class of realloc() bugs found by Coverity.Jeremy Allison2007-10-1055-481/+562
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | realloc can return NULL in one of two cases - (1) the realloc failed, (2) realloc succeeded but the new size requested was zero, in which case this is identical to a free() call. The error paths dealing with these two cases should be different, but mostly weren't. Secondly the standard idiom for dealing with realloc when you know the new size is non-zero is the following : tmp = realloc(p, size); if (!tmp) { SAFE_FREE(p); return error; } else { p = tmp; } However, there were *many* *many* places in Samba where we were using the old (broken) idiom of : p = realloc(p, size) if (!p) { return error; } which will leak the memory pointed to by p on realloc fail. This commit (hopefully) fixes all these cases by moving to a standard idiom of : p = SMB_REALLOC(p, size) if (!p) { return error; } Where if the realloc returns null due to the realloc failing or size == 0 we *guarentee* that the storage pointed to by p has been freed. This allows me to remove a lot of code that was dealing with the standard (more verbose) method that required a tmp pointer. This is almost always what you want. When a realloc fails you never usually want the old memory, you want to free it and get into your error processing asap. For the 11 remaining cases where we really do need to keep the old pointer I have invented the new macro SMB_REALLOC_KEEP_OLD_ON_ERROR, which can be used as follows : tmp = SMB_REALLOC_KEEP_OLD_ON_ERROR(p, size); if (!tmp) { SAFE_FREE(p); return error; } else { p = tmp; } SMB_REALLOC_KEEP_OLD_ON_ERROR guarentees never to free the pointer p, even on size == 0 or realloc fail. All this is done by a hidden extra argument to Realloc(), BOOL free_old_on_error which is set appropriately by the SMB_REALLOC and SMB_REALLOC_KEEP_OLD_ON_ERROR macros (and their array counterparts). It remains to be seen what this will do to our Coverity bug count :-). Jeremy. (This used to be commit 1d710d06a214f3f1740e80e0bffd6aab44aac2b0)
* r13914: Fix Coverity bug #151.Volker Lendecke2007-10-101-1/+1
| | | | | | | | I think this is actually a false warning, but as I've seen it with high gcc warning levels, lets fix it :-) Volker (This used to be commit 3f671033bca7a025f9639728a0a0a0adede6ed35)
* r13895: As agreed upon with gd on the phone, remove ↵Volker Lendecke2007-10-102-15/+3
| | | | | | | | | | | WBFLAG_PAM_CONTACT_TRUSTDOM. This can not work for NTLM auth, where we only have a workstation account for our own domain. For the PAM Kerberos login we need to find a better way to do this, probably using Dsr_GetDCName and some winbind-crafted krb5.conf. Volker (This used to be commit bf7c608147bcbbedd89b3dcd24a929ea3e601bc8)
* r13893: Fix for Coverity issue CID #164. The first one that I don'tJeremy Allison2007-10-101-0/+2
| | | | | | think is a direct bug, but some code that needs clarification :-). Jeremy. (This used to be commit 61901a3f10de64a72b655d9aa884424a4fc88a44)
* r13892: Doh ! My bugfix had a bug :-). Spotted by Willi Mann <willi@wm1.at>,Jeremy Allison2007-10-101-2/+2
| | | | | | if rrec can be null make sure we *never* deref it. Jeremy. (This used to be commit d6d7a5ac62b6ee08e365c5982302b1d8dc69a78f)
* r13889: Fix resource leak on error path. Coverity bug CID #73.Jeremy Allison2007-10-101-0/+1
| | | | | Jeremy. (This used to be commit 46e10980927f1dfa4a1995e778df880cf823cbdb)
* r13887: Fix coverity bug CID #94. mem leak on error codepath.Jeremy Allison2007-10-101-0/+2
| | | | | Jeremy. (This used to be commit dd47e0ef1175a57ec2e9b797ac942cb79f4a5d05)
* r13884: Fix coverity CID #95. Resource leak on error path.Jeremy Allison2007-10-101-0/+2
| | | | | Jeremy. (This used to be commit f4bf550b5757024b41062784b185b52a1a0e11f4)
* r13882: Fix coverity CID bug #96. Missing free on errorJeremy Allison2007-10-101-0/+1
| | | | | | exit path. Jeremy. (This used to be commit 95ef857c89a330ef4012ba3c10d2bbbbab112b34)
* r13880: Fix coverity bug CID #97, mem leak on error path.Jeremy Allison2007-10-101-4/+4
| | | | | Jeremy. (This used to be commit 0dc37dd2d85d59e7287cebcb7019194cf6754074)
* r13878: move PORT_DATA_1 to use static sized UNICODE strings as per MSDNGerald Carter2007-10-103-29/+30
| | | | (This used to be commit c803e1b2afdfc5bd983f046c976c01adebcfa1ad)
* r13875: Fix coverity bug #148. Deref of rrec before NULL check.Jeremy Allison2007-10-101-2/+7
| | | | | Jeremy. (This used to be commit 0f1dffb2f2ce5ace1b3216f578ab115c976624c7)
* r13873: I think this is the longstanding wins server crash bug, notJeremy Allison2007-10-101-17/+16
| | | | | | | part of the changes I made but something that's been there a while.... Coverity bugid #41. Jeremy. (This used to be commit 2f6cf810eae124820a073258ffe62aace7a92d9c)
* r13864: Some cleanup and the samr set security object function client-side.Günther Deschner2007-10-107-140/+108
| | | | | Guenther (This used to be commit 0ae3fddf95a95ec8a2f4d52e1276c1721b33ddfd)
* r13861: Avoid "net rpc join" segfaulting when storing the servername in theGünther Deschner2007-10-101-1/+1
| | | | | | | affinity cache. Guenther (This used to be commit b8c07babbd22832132da8f70026aa1816983bc38)
* r13846: Take care of system that do not have LDAP librariesSimo Sorce2007-10-101-0/+5
| | | | (This used to be commit ab62c8d93acb432678e301e57aeb86887913ebe6)
* r13843: Merge in net sam provision and some pdb_ldap fixesSimo Sorce2007-10-102-82/+516
| | | | (This used to be commit 705d8118081784e9907648fd1daaaa5ec0285972)
* r13841: Fix an uninitialized variable warning.Volker Lendecke2007-10-101-1/+1
| | | | | | | | Jerry, this just fixes the warning. This routine does not seem to cope well with !UNMARSHALLING. You might want to look... Volker (This used to be commit 2c0c40dfb5da9e901e099295f43032a745e71253)
* r13829: From the "It's not pretty but it works" categoryGerald Carter2007-10-105-13/+244
| | | | | | | | | | | * Finish prototype of the "add port command" implementation Format is "addportcommand portname deviceURI" * DeviceURI is either - socket://hostname:port/ - lpr://hostname/queue depending on what the client sent in the request (This used to be commit 6d74de7a676b71e83a3c3714743e6380c04e4425)
* r13824: * add api table for Xcv TCPMON and LOCALMON calls startingGerald Carter2007-10-103-15/+118
| | | | | | | | with the "MonitorUI" call * Fix some parsing errors This gets us to the Add Port Wizard dialog. (This used to be commit a444aa7f0088fb71ff89df8c280209188b33ec3d)
* r13821: replacing some strings with macrosGerald Carter2007-10-101-9/+11
| | | | (This used to be commit a34ab5c827630a5517e4c706877a172e6063f227)
* r13820: * Start fleshing out the XcvDataPort() server implementationGerald Carter2007-10-103-40/+107
| | | | | * Add support for the "Local Port" monitor as well through this API (This used to be commit ba9cdd88a0abf90a9c04959e554d7e4f10d17ff7)
* r13819: Remove accidently with rev 13713 submitted and never used MY_FLAGSLars Müller2007-10-101-1/+1
| | | | | variable. (This used to be commit 8f48c6d1e54b4f085f0db879ef548efecba7b42d)
* r13816: Volunteering :-)Volker Lendecke2007-10-104-23/+23
| | | | | | | | | | | | | | > for the svn log: > > - Solaris' /bin/sh doesn't know "test -e" - let's use "test -f" instead > > Some volunteer wanna check this in? :) > > Cheers > Bjoern Volker (This used to be commit 58d5f2031ac6018417ecd3c2306c120b5c7d1e43)
* r13815: "Into the blind world let us now descend,"Gerald Carter2007-10-106-138/+161
| | | | | | | | | | | | | | | | | Began the poet, his face as pale as death. "I will go first, and you will follow me." --- Adding XcvDataPort() to the spoolss code for remotely add ports. The design is to allow an intuitive means of creating a new CUPS print queue from the Windows 2000/XP APW without hacks like specifying the deviceURI in the location field of the printer properties dialog. Also set 'default devmode = yes' as the new default since it causes no harm and only is executed when you have a NULL devmode anyways. (This used to be commit 123e478ce5b5f63a61d00197332b847e83722468)
* r13802: I *knew* ASU on sparc had to be good for *something* ! :-).Jeremy Allison2007-10-102-5/+5
| | | | | | | Fix incorrect size understanding of sid name type (yes it's already correct in the Samba4 IDL :-). Jeremy. (This used to be commit 305a774d2880a57d1ebdf2ecf2d7e0b519695a33)
* r13799: Make locktest debug a little easier to read.Jeremy Allison2007-10-101-3/+17
| | | | | Jeremy. (This used to be commit 13de4d000e04ffca41814554bdee8291ddb1efe2)
* r13796: Another load_case_tables...Jeremy Allison2007-10-101-0/+2
| | | | | Jeremy. (This used to be commit 36df6ba3ad8b93744577ad92df0fe0b9bf9a4300)
* r13794: If you are going to go, go big. That's what I always say.Gerald Carter2007-10-101-2/+2
| | | | | | * disable winbind enum {users,groups} by default after further conversations with Volker. (This used to be commit d640d815405ce226c51577de5524daf63515d0a7)
* r13792: Merged Simo's fixes for tdbtraverse.Jeremy Allison2007-10-101-2/+2
| | | | | Jeremy. (This used to be commit 9ed3bd431c8b8073e177df463e254cf45529bed6)
* r13791: Having S-1-1-0 show up in winbind lookupsid does not really make sense.Volker Lendecke2007-10-102-3/+18
| | | | | Volker (This used to be commit ae9614ce019e25fb29dad8429d93f3140c2f84ad)
* r13778: When deleting machine accounts it's the SeMachineAccountPrivilegeJeremy Allison2007-10-101-1/+9
| | | | | | that counts. Jeremy. (This used to be commit aa85ba4f3799ffbe5c6f84f768f03a4c68d879dc)
* r13776: Merge in the editposix ldapsam optimizationSimo Sorce2007-10-102-109/+892
| | | | (This used to be commit a374546c7e8dfc17eb2346c518d1d89f28c32feb)
* r13772: More default changesGerald Carter2007-10-101-2/+3
| | | | | | | * winbind nested groups = yes * host msdfs = ye * msdfs root = yes (This used to be commit b5f01559e1f75c427e59646ee79e18433806213e)
* r13771: BUG 3534: ignore lines in the username map file with no right hand ↵Gerald Carter2007-10-101-2/+4
| | | | | | list rather than bailing out (This used to be commit acff5163ca7be59e01438f7cf63faef9ed54b820)
* r13766: Patch from Arek Glabek <aglabek@centeris.com>:Gerald Carter2007-10-101-6/+3
| | | | | | * Fix parsing error in eventlogadm caused by log entries with no DAT: line. (This used to be commit f0a8f438793a806e8cf73e1e695b09e540a4239e)