summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2007-10-10r14023: My last bug fix still left a potential null deref.Jeremy Allison1-8/+9
C- "must try harder" :-). Jeremy. (This used to be commit 9c55bf74ca28530045c5cb3cbfffad242039ca75)
2007-10-10r14022: Fix Coverity bug # 92Volker Lendecke1-0/+1
(This used to be commit b824245c4e04353f0d3fd0ccf6bc5776a601daed)
2007-10-10r14020: Coverity bug CID #66. Missing free on error exit.Jeremy Allison1-1/+4
Jeremy. (This used to be commit b9980bddf5ee74b2887196d6d1a0cf393720ba3a)
2007-10-10r14018: Coverity error CID #27. Missing return -1 on error condition.Jeremy Allison1-0/+1
Jeremy. (This used to be commit 94e869d9c6d1a1c1df0d072530bf8d4452bd10bb)
2007-10-10r14017: Okay, okay, I have turned a double-free into dead code :-)Volker Lendecke1-2/+0
Fix Coverity # 214. Volker (This used to be commit 4a75edf9deca2be18670d7f9f2e383ed0898512c)
2007-10-10r14014: Coverity paranoia. Shut it up by making the guarenteeJeremy Allison1-2/+7
in the code explicit - but this was a false positive (CID #16). Jeremy. (This used to be commit 43a0e869f2aee9b4e22d0d7fc92051e82f7536ad)
2007-10-10r14009: Remove last const warning (have to use CONST_DISCARD).Jeremy Allison1-1/+3
Jeremy. (This used to be commit af0ade470f2fac3509a44207b4572e279ba30e34)
2007-10-10r14007: Coverity bug CID #197. Don't compare against 0, weJeremy Allison1-1/+1
mean the ttl instead. Jeremy. (This used to be commit ccb2a52e29de7e2847ccd93aa99236f63202a4af)
2007-10-10r14006: Fix a couple of irritating warnings.Jeremy Allison1-2/+2
Jeremy. (This used to be commit ead13ca522d7b8cbb47d660d3cb73c3582088985)
2007-10-10r14003: Clarify code that lead to Coverity report #13.Jeremy Allison2-13/+16
Not a bug, but better to remove false positives. Jeremy. (This used to be commit f9a75d76546bc4618736f0d48646e77d7572db25)
2007-10-10r14002: Quick fix for Coverity CID #12.Jeremy Allison1-0/+4
Jeremy (This used to be commit 7c6e274cd578521192a0b0c4e6a4fb5dc7d722ac)
2007-10-10r13999: Quick fix for Coverity bug #11.Jeremy Allison1-1/+7
Jeremy. (This used to be commit b9de6c926953b3321fa3850d501c807c6eabf230)
2007-10-10r13997: Prefix $prefix by default even to rootsbindir as we else run in troubleLars Müller1-2/+2
if we use ./configure --prefix=dir && make install Vendors might use ./configure --with-rootsbindir=/sbin && make DESTDIR=/var/tmp/samba/ install if they use a buildroot. Thanks to Björn Jacke for pointing me at the stderr output of the build farm. (This used to be commit 75c95d149a22eba4a5afedcec7ba5e2a28cccf01)
2007-10-10r13994: Belt and braces - ensure RPC_BUFFER is valid.Jeremy Allison1-44/+75
Jeremy. (This used to be commit d993797191865878ebfd2ff9028d341017605cd6)
2007-10-10r13991: Fix Coverity bug # 69Volker Lendecke1-0/+1
(This used to be commit 6dc79e6b12e221e9af85a1edf487b5fb5aae222b)
2007-10-10r13989: Fix for Coverity bug #45 and associated spoolss RPC_BUFFERJeremy Allison1-7/+22
problems. Ensure that if the parse succeeds on UNMARSHALL we have a valid (although possibly empty) RPC_BUFFER returned. Jeremy. (This used to be commit d319cc9c08bfa865a6431a8631a9c609f589be1f)
2007-10-10r13987: Fix Coverity bug # 74. This tool is good...Volker Lendecke1-0/+1
Thanks, Volker (This used to be commit 86f62484dd7db43e036d9edf29e459b8bd0e5fbe)
2007-10-10r13984: Fix Coverity bug # 98Volker Lendecke1-1/+3
(This used to be commit 0a2aa3a48bd5fd7e5a9aa06068ddd621b19c1dbe)
2007-10-10r13983: Fix Coverity bug # 111Volker Lendecke1-0/+1
(This used to be commit 08d7fd31ab250bc6ba3922b36aa7b0cfef1e5bf1)
2007-10-10r13981: Fix Coverity bug # 138Volker Lendecke1-1/+1
(This used to be commit 303067ba3bdf34ab501f0d99e386cfdb6ab10233)
2007-10-10r13980: Fix Coverity bug # 63. Jeremy, could you take a look at this?Volker Lendecke1-5/+16
Thanks, Volker (This used to be commit 49043a756dfa6a973d5605f42068df351a4a9ab5)
2007-10-10r13979: We've dereferenced my_methods already, so there's no point in ↵Volker Lendecke1-1/+1
checking for != NULL. Coverity #149. Volker (This used to be commit d38e05329a77650d8fbb8611ca148964f62c9ba4)
2007-10-10r13978: Here is why it's essential to use SAFE_FREE instead of free.Jeremy Allison1-6/+6
If we use free(data.dptr) and then the subsequent tdb_open fails in _reg_perfcount_get_counter_data() then data.dptr is left as a non-zero pointer that has been freed. This would cause it to be reused later on. Coverity bug #162. Jeremy. (This used to be commit 053efc20981e0280c6af0ebb9e17cea07da85fe8)
2007-10-10r13975: Re-fix Coverity #156 - I had left the hidden arg. inconsistentJeremy Allison3-5/+6
between Realloc and realloc_array. Jeremy. (This used to be commit 841c9b1847ae12656b827e3d35b8bf0c3f68b8b4)
2007-10-10r13971: Fix Coverity bugs #209 - #213 - it detected mistakesJeremy Allison1-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)
2007-10-10r13968: fix typo, caught by GuentherJim McDonough1-1/+1
(This used to be commit 217d3fbe7923115ae610a39e586ceb93df7683f5)
2007-10-10r13965: Make sure we always reset the userAccountControl bits when re-joiningGünther Deschner1-1/+1
with an existing account. Guenther (This used to be commit e4c12ab167ee83772a2bdd1946b8d73613fc0d7e)
2007-10-10r13958: Fix Coverity Bug # 141Volker Lendecke1-1/+1
(This used to be commit 598513d1d3e23cc71ea0fd53230d393b6724b534)
2007-10-10r13957: Based on patch from Richard Renard <richard.renard@idealx.com>:Jim McDonough1-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)
2007-10-10r13955: Fix Coverity ID 139.Volker Lendecke1-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)
2007-10-10r13951: Fix Coverity Bug #163.Volker Lendecke1-28/+0
This code was not used anyway :-) Volker (This used to be commit bbfb20569380529d60e3c61cd0be63a09eecfd17)
2007-10-10r13950: Fix Coverity bug #168Volker Lendecke1-0/+1
(This used to be commit cbf894c0e37964df57bd6a91ac10dfff571b1b3c)
2007-10-10r13948: Fix the build. Remove rpcclient.h include.Jim McDonough1-1/+0
(This used to be commit 6bf879bee3c59ba54b1b4c465c777e3dd0043f83)
2007-10-10r13947: Use tabs instead of spaces for indention.Lars Müller1-3/+3
(This used to be commit fb76390c6ae5928a2a222d61cbadf825611999ef)
2007-10-10r13946: Link pam_smbpass with the required object files. Fix bug #3565.Lars Müller1-1/+2
(This used to be commit 426c8fe0bdacbf218d4cf4a10af789afbc0e53f2)
2007-10-10r13945: Move display_sec.c to lib/ (as suggested by Volker).Günther Deschner2-4/+4
Guenther (This used to be commit 7e80d5358eb181c3515acb732a3594e80391261b)
2007-10-10r13916: Fix Coverity bug #29. Looks like my code. I wonder how much there is ↵Volker Lendecke1-0/+1
still lurking... Volker (This used to be commit 1345a52794f4f55173ed677af3d0714e88bf17c6)
2007-10-10r13915: Fixed a very interesting class of realloc() bugs found by Coverity.Jeremy Allison55-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)
2007-10-10r13914: Fix Coverity bug #151.Volker Lendecke1-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)
2007-10-10r13895: As agreed upon with gd on the phone, remove ↵Volker Lendecke2-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)
2007-10-10r13893: Fix for Coverity issue CID #164. The first one that I don'tJeremy Allison1-0/+2
think is a direct bug, but some code that needs clarification :-). Jeremy. (This used to be commit 61901a3f10de64a72b655d9aa884424a4fc88a44)
2007-10-10r13892: Doh ! My bugfix had a bug :-). Spotted by Willi Mann <willi@wm1.at>,Jeremy Allison1-2/+2
if rrec can be null make sure we *never* deref it. Jeremy. (This used to be commit d6d7a5ac62b6ee08e365c5982302b1d8dc69a78f)
2007-10-10r13889: Fix resource leak on error path. Coverity bug CID #73.Jeremy Allison1-0/+1
Jeremy. (This used to be commit 46e10980927f1dfa4a1995e778df880cf823cbdb)
2007-10-10r13887: Fix coverity bug CID #94. mem leak on error codepath.Jeremy Allison1-0/+2
Jeremy. (This used to be commit dd47e0ef1175a57ec2e9b797ac942cb79f4a5d05)
2007-10-10r13884: Fix coverity CID #95. Resource leak on error path.Jeremy Allison1-0/+2
Jeremy. (This used to be commit f4bf550b5757024b41062784b185b52a1a0e11f4)
2007-10-10r13882: Fix coverity CID bug #96. Missing free on errorJeremy Allison1-0/+1
exit path. Jeremy. (This used to be commit 95ef857c89a330ef4012ba3c10d2bbbbab112b34)
2007-10-10r13880: Fix coverity bug CID #97, mem leak on error path.Jeremy Allison1-4/+4
Jeremy. (This used to be commit 0dc37dd2d85d59e7287cebcb7019194cf6754074)
2007-10-10r13878: move PORT_DATA_1 to use static sized UNICODE strings as per MSDNGerald Carter3-29/+30
(This used to be commit c803e1b2afdfc5bd983f046c976c01adebcfa1ad)
2007-10-10r13875: Fix coverity bug #148. Deref of rrec before NULL check.Jeremy Allison1-2/+7
Jeremy. (This used to be commit 0f1dffb2f2ce5ace1b3216f578ab115c976624c7)
2007-10-10r13873: I think this is the longstanding wins server crash bug, notJeremy Allison1-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)