1
0
mirror of https://github.com/samba-team/samba.git synced 2025-01-25 06:04:04 +03:00

16924 Commits

Author SHA1 Message Date
Jeremy Allison
ae02c05bfc 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
2007-10-10 11:19:02 -05:00
Jeremy Allison
16b0617cb0 r16587: Fix Klocwork #1987. Memleak on reusing x_dbf.
Jeremy.
2007-10-10 11:19:01 -05:00
Jeremy Allison
a912ef0d97 r16585: Fix Klocwork #1976. Possible null deref.
Jeremy.
2007-10-10 11:19:01 -05:00
Jeremy Allison
ce14daf51c r16582: Fix Klocwork #1997 and all generic class of problems
where we don't correctly check the return from memdup.
Jeremy.
2007-10-10 11:19:01 -05:00
Jeremy Allison
6967fd4cef r16581: Fix Klocwork #2017. Possible null deref.
Jeremy.
2007-10-10 11:19:01 -05:00
Jeremy Allison
f6d5bae4a1 r16579: Fix Klocwork #2016. Possible null deref.
Jeremy.
2007-10-10 11:19:01 -05:00
Jeremy Allison
9cbfaf62a3 r16576: Fix Klocwork #2015. Possible null deref.
Jeremy.
2007-10-10 11:19:01 -05:00
Christopher R. Hertel
c65ebeb028 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.
2007-10-10 11:19:00 -05:00
Jeremy Allison
b1fc2d8b99 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.
2007-10-10 11:19:00 -05:00
Derrell Lipman
d90061aa93 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
2007-10-10 11:19:00 -05:00
Derrell Lipman
c1b4c51053 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
2007-10-10 11:18:59 -05:00
Jeremy Allison
2b8d72f09c r16548: Fix bug #3867 reported by jason@ncac.gwu.edu.
Jeremy.
2007-10-10 11:18:59 -05:00
Jeremy Allison
be888a3cda r16546: Fix bug #3865 reported by jason@ncac.gwu.edu.
Jeremy.
2007-10-10 11:18:59 -05:00
Jeremy Allison
16e42b446b r16544: Fix bug #3864 reported by jason@ncac.gwu.edu.
Jeremy.
2007-10-10 11:18:59 -05:00
Jeremy Allison
cde8323fdc r16542: Fix #3863, reported by jason@ncac.gwu.edu
Jeremy.
2007-10-10 11:18:59 -05:00
Jeremy Allison
09e11dcb23 r16541: Fix #3862 reported by jason@ncac.gwu.edu.
Jeremy.
2007-10-10 11:18:58 -05:00
Jeremy Allison
cd048cb775 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.
2007-10-10 11:18:57 -05:00
Gerald Carter
f3d2bbf0d2 r16528: Self-confidence is plummeting. How could one person goof
the same change 3 times......

Fix pam_winbind compilation again.
2007-10-10 11:18:57 -05:00
Gerald Carter
4490fbdfc2 r16510: fix from Nicholas Brealey <nick@brealey.org> for building pam_winbind.so 2007-10-10 11:18:57 -05:00
Volker Lendecke
8cf364e602 r16490: Fix a memleak and two typos 2007-10-10 11:18:57 -05:00
James Peach
09fb20f4fe r16484: Bugzilla 3805. Only define _ABS if it is not already defined. 2007-10-10 11:18:57 -05:00
James Peach
a22bf28bca r16483: Turn off some useless MIPSPro warnings be default. 2007-10-10 11:18:56 -05:00
James Peach
2131ddd7a1 r16482: Fix bugzilla 3801 by forcing MIPSPro warning 1035 to be an error. 2007-10-10 11:18:56 -05:00
Günther Deschner
5853525f11 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
2007-10-10 11:18:56 -05:00
Günther Deschner
098a87f492 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
2007-10-10 11:18:56 -05:00
Gerald Carter
0fca41efde r16478: small cleanup to make pam_winbind.so build and iniparser
include directory like other parts of the build files.
2007-10-10 11:18:56 -05:00
Günther Deschner
d6b52e8181 r16475: destroy talloc ctx when we weren't able to collect onlinestatus
messages.

Guenther
2007-10-10 11:18:56 -05:00
Günther Deschner
dfbe555c69 r16474: There is no point in figuring out lockout policies if we do not allow
offline logons at all.

Guenther
2007-10-10 11:18:55 -05:00
Günther Deschner
520777f794 r16473: There is no point in calling set_dc_type_and_flags() before each
pam_auth login (when using kerberos).

Guenther
2007-10-10 11:18:55 -05:00
Gerald Carter
ce7846d6f1 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.
2007-10-10 11:18:55 -05:00
Gerald Carter
8421d1b080 r16470: remove ntlm_auth_proto.h and passwd_proto.h in 'make clean 2007-10-10 11:18:55 -05:00
Günther Deschner
840ac23ec0 r16458: Increase debuglevel of cli_rpc_pipe_close().
Guenther
2007-10-10 11:18:55 -05:00
Günther Deschner
49fb1a3ebc r16453: Fix another memleak.
Guenther
2007-10-10 11:18:55 -05:00
Günther Deschner
479dec6845 r16452: Fix memleak in the CLDAP processing (found by valgrind).
Guenther
2007-10-10 11:18:54 -05:00
Jeremy Allison
d929323d6f 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.
2007-10-10 11:18:54 -05:00
Jeremy Allison
490c7c8467 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.
2007-10-10 11:18:54 -05:00
Jeremy Allison
9017547ccc r16429: Fix final 4 Klocwork bugs we're going to fix before
release - #785, #786, #787, #788.
Jeremy.
2007-10-10 11:18:53 -05:00
Volker Lendecke
adf2dcce09 r16427: Fix bug # 3848. Thanks to Wilco Baan Hofman for testing the release candidate!
Volker
2007-10-10 11:18:53 -05:00
Jeremy Allison
e71cc6647a r16426: Klocwork #1544, #1545, #1546, #1549, #1550, #1552, #1553, #1554
Jeremy.
2007-10-10 11:18:53 -05:00
Jeremy Allison
4fbeae1a3a 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.
2007-10-10 11:18:52 -05:00
Jeremy Allison
e83c3e0a65 r16422: winbindd_demote_client isn't used and generates
a Klocwork issue (#1844). Remove it
Jeremy.
2007-10-10 11:18:52 -05:00
Jeremy Allison
f5dddf339e r16420: Fix Klocwork #1674. Null deref.
Jeremy.
2007-10-10 11:18:52 -05:00
Günther Deschner
4e1816855a r16415: Also delete object files from iniparser.
Guenther
2007-10-10 11:18:52 -05:00
Volker Lendecke
601643a460 r16411: Fix compilation of vfs_afsacl, thanks to Greszler Szilard for trying 2007-10-10 11:18:52 -05:00
Volker Lendecke
d6547d12b1 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
2007-10-10 11:18:52 -05:00
Jeremy Allison
d357f8b335 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.
2007-10-10 11:18:52 -05:00
Jeremy Allison
220627ab70 r16396: Klocwork #1170. Null deref.
Jeremy.
2007-10-10 11:18:51 -05:00
Jeremy Allison
c9a4ce51ed r16393: Klocwork #1164. Null deref.
Jeremy.
2007-10-10 11:18:51 -05:00
Jeremy Allison
cab256d72a r16392: Klockwork #1168. Protect against null deref.
Jeremy.
2007-10-10 11:18:51 -05:00
Jeremy Allison
e99cedfb0c r16390: Klocwork #604 - ensure if NULL pointer we assert
if len != 0.
Jeremy.
2007-10-10 11:18:51 -05:00