summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2011-03-27tdb: Fix Coverity ID 2192: NO_EFFECTVolker Lendecke1-1/+1
(ret < 0) can never be true
2011-03-27s3: Fix Coverity ID 2195: NO_EFFECTVolker Lendecke1-1/+1
level is unsigned
2011-03-27librpc: Add some error checking to dcerpc_floor_pack_rhs_if_version_dataVolker Lendecke1-1/+5
2011-03-27librpc: Fix Coverity ID 2198: NULL_RETURNSVolker Lendecke1-3/+14
2011-03-27s3: Fix Coverity ID 2200: NULL_RETURNS (cut&paste error)Volker Lendecke1-1/+1
2011-03-27s3: Fix Coverity ID 2228: RESOURCE_LEAKVolker Lendecke1-0/+1
Holger, please check!
2011-03-27s3: Fix Coverity ID 2226: RESOURCE_LEAKVolker Lendecke1-0/+1
Holger, please check!
2011-03-27s3: Fix Coverity ID 976: BAD_SIZEOFVolker Lendecke1-1/+1
2011-03-27s3: Fix Coverity ID 958: BAD_SIZEOFVolker Lendecke1-1/+1
This is supposed to wipe out the md5 context, not only the first bytes of it. Others, please check!
2011-03-27s3: Fix Coverity ID 682: NEGATIVE_RETURNSVolker Lendecke1-1/+4
2011-03-27s3: Fix Coverity ID 590: DEADCODEVolker Lendecke1-4/+0
2011-03-27s3: Fix Coverity ID 585: NEGATIVE_RETURNSVolker Lendecke1-1/+1
2011-03-27s3: Fix Coverity ID 513: UNINITVolker Lendecke1-1/+1
2011-03-27s3: Fix Coverity ID 83: RESOURCE_LEAKVolker Lendecke1-0/+3
2011-03-27s3: Convert cli_raw_tcon to cli_smbVolker Lendecke2-26/+24
Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Sun Mar 27 19:50:21 CEST 2011 on sn-devel-104
2011-03-27s3: Fix a memleakVolker Lendecke1-0/+2
2011-03-27s3: Replace a use of cli_errstrVolker Lendecke1-1/+1
2011-03-27s3: Replace cli_negprot_sendsync() by cli_negprot_send()Volker Lendecke3-38/+22
2011-03-27s3: Remove a pointless staticVolker Lendecke1-1/+1
2011-03-27s3: Remove a use of cli_send_smbVolker Lendecke3-39/+50
2011-03-27s3: Remove two calls to cli_errstrVolker Lendecke1-4/+8
Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Sun Mar 27 13:17:47 CEST 2011 on sn-devel-104
2011-03-27s3: Fix Coverity ID 2296: UNUSED_VALUEVolker Lendecke1-3/+0
Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Sun Mar 27 12:12:35 CEST 2011 on sn-devel-104
2011-03-27s3: Fix Coverity ID 2327: FORWARD_NULLVolker Lendecke1-0/+2
Make check_ft_info robust against an invalid type. Günther, please check!
2011-03-27s3: Fix Coverity ID 2328: FORWARD_NULLVolker Lendecke1-0/+3
Make startsmbfilepwent robust against an invalid open type
2011-03-27s3: Fix Coverity ID 2329: RESOURCE_LEAKVolker Lendecke1-0/+1
2011-03-27s3: Fix Coverity ID 2330: RESOURCE_LEAKVolker Lendecke1-0/+1
2011-03-27s3: Fix Coverity ID 2331: RESOURCE_LEAKVolker Lendecke1-0/+1
2011-03-26s3: Fix an uninitialized variable readVolker Lendecke1-1/+2
The "is_valid_policy_handle" in the exit path reads uninitialized data Lets see if this fixes Coverity ID 2172 which complains about h being dereferenced in the dcerpc_winreg_CloseKey. Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Sat Mar 26 14:24:28 CET 2011 on sn-devel-104
2011-03-26s3: Fix Coverity ID 2326, CHECKED_RETURNVolker Lendecke1-1/+1
2011-03-26Revert "Fix bug 8021 - Incorrect string termination in volume/volume_name ↵Jeremy Allison1-1/+1
for TRANS2-QUERY_FS_INFO/Info Volume." This reverts commit 042aafb87df6c05877b8fc7ef0d44877689d860a. I need to test against OS/2 to see if this is something Win28K breaks for older clients before allowing this. Jeremy. Autobuild-User: Jeremy Allison <jra@samba.org> Autobuild-Date: Sat Mar 26 01:19:55 CET 2011 on sn-devel-104
2011-03-25Fix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup ↵Jeremy Allison1-2/+23
is configured. As discovered by David Disseldorp <ddiss@suse.de>, convert_string_talloc() doesn't always return consistent results for a zero length string. The API states an incoming string must *always* contain the terminating null, but unfotunately too much code expects passing in a zero source length to return a null terminated string, so at least ensure we return a correct null string in the required character set and return the correct length. Also ensure we cannot return a zero length for a converted string (we ensure that the returned buffer is always allocated and zero terminated anyway) as calling code depends on the fact that returning true from this function will *always* return a non-zero length (as it must include the terminating null). Jeremy. Autobuild-User: Jeremy Allison <jra@samba.org> Autobuild-Date: Fri Mar 25 23:25:40 CET 2011 on sn-devel-104
2011-03-25s3-netapi: fix memoryleak while not using talloc_tos() in ↵Günther Deschner2-6/+6
cli_get_session_key() usage. Thanks Volker! Guenther Autobuild-User: Günther Deschner <gd@samba.org> Autobuild-Date: Fri Mar 25 18:21:18 CET 2011 on sn-devel-104
2011-03-25s3: Compile IDL files in autogen, some configure tests need thisVolker Lendecke2-0/+12
Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Fri Mar 25 16:35:30 CET 2011 on sn-devel-104
2011-03-25Revert "s3-includes: fixed CONST_DISCARD() to actually discard const"Volker Lendecke1-1/+1
This reverts commit 37261928bec145f4cbd85f3e409063563c00ac9e. In source3 this creates a warning of the form passdb/pdb_ldap.c:4565: warning: passing argument 2 of ‘pull_utf8_talloc’ from incompatible pointer type include/proto.h:444: note: expected ‘char **’ but argument is of type ‘char ***’ for every use of CONST_DISCARD. Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Fri Mar 25 13:16:35 CET 2011 on sn-devel-104
2011-03-25charcnv: ensure convert_string_error doesn't conflict with s3 functionAndrew Tridgell1-5/+5
use _handle for handle based functions Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org> Autobuild-User: Andrew Tridgell <tridge@samba.org> Autobuild-Date: Fri Mar 25 05:22:24 CET 2011 on sn-devel-104
2011-03-25s3-charcnv: fixed converted_size return in fast pathsAndrew Tridgell1-9/+20
2011-03-25s3-charcnv: removed unused functionAndrew Tridgell2-8/+0
lp_failed_convert_char() is not needed any more
2011-03-25lib/util/charset rename iconv_convenience to iconv_handleAndrew Bartlett15-108/+108
This better reflects what this structure is Andrew Bartlett
2011-03-25s3-charcnv: convert_string_internal() should not display errorsAndrew Tridgell1-31/+4
debug error display happens in the convert_string() outer function
2011-03-25s3-string: removed the conv_silent globalAndrew Tridgell1-29/+17
use convert_string_error() instead Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25s3-string: removed last use of conv_silentAndrew Tridgell1-31/+11
This replaces the push_ascii_nstring() implementation with a call to convert_string_error() Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25s3-charcnv Add convert_string_error()Andrew Bartlett2-18/+75
This function returns errors rather than printing them. Andrew Bartlett
2011-03-25charcnv: added convert_string_error()Andrew Tridgell1-33/+57
this will allow us to remove conv_silent Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25s3-includes: fixed CONST_DISCARD() to actually discard constAndrew Tridgell1-1/+1
a cast is not enough Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25unistr: moved some UCS2 macros into util_unistrAndrew Tridgell2-9/+9
we need to move towards eliminating smb_ucs2_t. This moves a couple of the related macros into the only file they are used in Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25libcli: created smb_constants.hAndrew Tridgell5-27/+44
this starts the (long!) process of moving some of the SMB constants into common files. This just moves the FLAGS2_ defines, which are needed for common string routines (for FLAGS2_UNICODE_STRINGS)
2011-03-25tdb: rename convert_string() to tdb_convert_string()Andrew Tridgell1-5/+5
this prevents a conflict with the convert_string() in samba
2011-03-25s3-libsmb: work our correct length for EA stringsAndrew Tridgell1-1/+1
2011-03-25s3-string: sec_len==-1 support is no longer neededAndrew Tridgell1-41/+20
Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>
2011-03-25s3-dfs: removed the last user of sec_len==-1 in pull_*_base_talloc()Andrew Tridgell1-1/+2
hopefully there aren't any others lurking in the code