samba.git
13 years agos3-spoolssd: Start the spoolss service correctly.
Andreas Schneider [Tue, 29 Mar 2011 08:52:32 +0000 (10:52 +0200)]
s3-spoolssd: Start the spoolss service correctly.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-spoolssd: Register spoolssd endpoints.
Andreas Schneider [Mon, 28 Mar 2011 12:18:23 +0000 (14:18 +0200)]
s3-spoolssd: Register spoolssd endpoints.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-epmap: Make rpc_ep_setup_register an internal function.
Andreas Schneider [Mon, 28 Mar 2011 12:18:00 +0000 (14:18 +0200)]
s3-epmap: Make rpc_ep_setup_register an internal function.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-spoolssd: Fixed logfile creation.
Andreas Schneider [Mon, 28 Mar 2011 12:30:29 +0000 (14:30 +0200)]
s3-spoolssd: Fixed logfile creation.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-spoolssd: Added missing include.
Andreas Schneider [Thu, 24 Mar 2011 09:22:57 +0000 (10:22 +0100)]
s3-spoolssd: Added missing include.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-spoolssd: Pass down event and messanging context.
Andreas Schneider [Thu, 20 Jan 2011 17:59:40 +0000 (18:59 +0100)]
s3-spoolssd: Pass down event and messanging context.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3-spoolssd: Fixed reopening of logs.
Andreas Schneider [Wed, 25 Aug 2010 08:15:47 +0000 (10:15 +0200)]
s3-spoolssd: Fixed reopening of logs.

Everytime the log was reopened a .spoolssd has been appended which
resulted in long file names.

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd Start spoolss from smbd
Simo Sorce [Thu, 20 May 2010 12:33:14 +0000 (08:33 -0400)]
s3:spoolssd Start spoolss from smbd

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd Initialize the spoolss rpc interfaces
Simo Sorce [Tue, 15 Jun 2010 22:39:27 +0000 (18:39 -0400)]
s3:spoolssd Initialize the spoolss rpc interfaces

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd Add spoolss own signal handlers
Simo Sorce [Wed, 16 Jun 2010 17:03:32 +0000 (13:03 -0400)]
s3:spoolssd Add spoolss own signal handlers

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd handle smb.conf reloads directly
Simo Sorce [Wed, 16 Jun 2010 19:36:06 +0000 (15:36 -0400)]
s3:spoolssd handle smb.conf reloads directly

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd Create our own log file
Simo Sorce [Tue, 15 Jun 2010 22:38:44 +0000 (18:38 -0400)]
s3:spoolssd Create our own log file

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:spoolssd Add skeleton for spoolss daemon
Simo Sorce [Thu, 20 May 2010 12:48:18 +0000 (08:48 -0400)]
s3:spoolssd Add skeleton for spoolss daemon

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agos3:printing Make print_queue_receive public
Simo Sorce [Thu, 20 May 2010 12:49:50 +0000 (08:49 -0400)]
s3:printing Make print_queue_receive public

Signed-off-by: Günther Deschner <gd@samba.org>
13 years agoldb: detect eof on ldif files
Andrew Tridgell [Tue, 29 Mar 2011 05:31:17 +0000 (16:31 +1100)]
ldb: detect eof on ldif files

use feof() to detect parsing errors in ldif files

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Mar 29 08:24:04 CEST 2011 on sn-devel-104

13 years agos4-dsdb: allow modification of linked attribute targets with relax
Andrew Tridgell [Tue, 29 Mar 2011 05:14:18 +0000 (16:14 +1100)]
s4-dsdb: allow modification of linked attribute targets with relax

this is used to help recover a corrupt database.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agodebug: ignore debug_set_logfile() with a blank string
Andrew Tridgell [Tue, 29 Mar 2011 03:41:58 +0000 (14:41 +1100)]
debug: ignore debug_set_logfile() with a blank string

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Mar 29 07:03:58 CEST 2011 on sn-devel-104

13 years agos4-cmdline: moved loadparm init after fault setup
Andrew Tridgell [Tue, 29 Mar 2011 03:30:24 +0000 (14:30 +1100)]
s4-cmdline: moved loadparm init after fault setup

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-param: set a default logfile
Andrew Tridgell [Tue, 29 Mar 2011 03:29:54 +0000 (14:29 +1100)]
s4-param: set a default logfile

setup a logfile for when one is not specified in smb.conf

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agolib: removed a spurios declaration of 'logfile'
Andrew Tridgell [Tue, 29 Mar 2011 03:29:22 +0000 (14:29 +1100)]
lib: removed a spurios declaration of 'logfile'

we don't have a global logfile variable any more

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agoRemove two unused labels.
Jeremy Allison [Mon, 28 Mar 2011 23:07:13 +0000 (16:07 -0700)]
Remove two unused labels.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Mar 29 02:23:02 CEST 2011 on sn-devel-104

13 years ago=?UTF-8?q?alpha=5Fstrcpy()=20is=20a=20utility=20function=20which=20reportedly:
David Disseldorp [Mon, 28 Mar 2011 22:59:32 +0000 (15:59 -0700)]
=?UTF-8?q?alpha=5Fstrcpy()=20is=20a=20utility=20function=20which=20reportedly:

=20Strips=20out=20all=20but=20'a-Z0-9'=20and=20the=20character=20in=20other=5Fsafe=5Fchars=20and
=20replaces=20with=20'=5F'.
=20This=20statement=20does=20not=20currently=20hold=20true=20in=20all=20cases=20(e.g.=20src=20=3D
=20"=D0=A2=D0=90=D0=9D=D0=A6=D0=95=D0=92=D0=90=D0=A2=D0=AC").?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

13 years agos3:rpc_client: fix argument order of call to lsa_open_policy2 in rpccli_lsa_open_policy2
Michael Adam [Mon, 28 Mar 2011 22:01:40 +0000 (00:01 +0200)]
s3:rpc_client: fix argument order of call to lsa_open_policy2 in rpccli_lsa_open_policy2

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Tue Mar 29 01:34:28 CEST 2011 on sn-devel-104

13 years agos3-selftest Fix test_smbclient_auth.sh
Andrew Bartlett [Mon, 28 Mar 2011 11:06:40 +0000 (22:06 +1100)]
s3-selftest Fix test_smbclient_auth.sh

The -N in the command line meant that all the logins were as the
guest, not as the user specified.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Mar 29 00:47:50 CEST 2011 on sn-devel-104

13 years agos3-selftest Allow LM passwords and turn of NTLMv2 for security=share test
Andrew Bartlett [Mon, 28 Mar 2011 11:20:23 +0000 (22:20 +1100)]
s3-selftest Allow LM passwords and turn of NTLMv2 for security=share test

13 years agos3-selftest Add tests for security=server
Andrew Bartlett [Mon, 28 Mar 2011 09:42:55 +0000 (20:42 +1100)]
s3-selftest Add tests for security=server

Andrew Bartlett

13 years agos3-selftest unconditionaly include subunit.sh
Andrew Bartlett [Mon, 28 Mar 2011 11:05:27 +0000 (22:05 +1100)]
s3-selftest unconditionaly include subunit.sh

we no longer . this file into other shell scripts, so we don't need
this check any more.

Andrew Bartlett

13 years agowintest New snapshot for my wintest VMs
Andrew Bartlett [Mon, 28 Mar 2011 09:44:10 +0000 (20:44 +1100)]
wintest New snapshot for my wintest VMs

13 years agos4-fsmo: say which role is being transferred
Andrew Tridgell [Mon, 28 Mar 2011 05:48:46 +0000 (16:48 +1100)]
s4-fsmo: say which role is being transferred

this is particularly useful for "samba-tool fsmo transfer --role=all"

13 years agos4-fsmo: samba-tool fsmo transfer now gets full error code
Andrew Tridgell [Mon, 28 Mar 2011 05:45:22 +0000 (16:45 +1100)]
s4-fsmo: samba-tool fsmo transfer now gets full error code

We now wait for the transfer to succeed or fail

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-dsdb: perform FSMO transfers asynchronously
Andrew Tridgell [Mon, 28 Mar 2011 05:44:32 +0000 (16:44 +1100)]
s4-dsdb: perform FSMO transfers asynchronously

this gives the administrator a proper error message on the command
line

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-fsmo: make rootDSE modify for FSMO transfer async
Andrew Tridgell [Mon, 28 Mar 2011 05:20:47 +0000 (16:20 +1100)]
s4-fsmo: make rootDSE modify for FSMO transfer async

this gives the ldap client the error code from the transfer

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-fsmo: nicer error messages on failed FSMO transfers
Andrew Tridgell [Mon, 28 Mar 2011 04:32:41 +0000 (15:32 +1100)]
s4-fsmo: nicer error messages on failed FSMO transfers

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-drs: default hostname in samba-tool drs command
Andrew Tridgell [Mon, 28 Mar 2011 04:32:15 +0000 (15:32 +1100)]
s4-drs: default hostname in samba-tool drs command

default to our hostname

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-dsdb: only allow administrators to trigger FSMO role transfers
Andrew Tridgell [Mon, 28 Mar 2011 03:41:36 +0000 (14:41 +1100)]
s4-dsdb: only allow administrators to trigger FSMO role transfers

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos4-fsmo: samba-tool fsmo takes a URL, not a hostname
Andrew Tridgell [Mon, 28 Mar 2011 03:17:45 +0000 (14:17 +1100)]
s4-fsmo: samba-tool fsmo takes a URL, not a hostname

better to call the option --url

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agowintest: added a "ipconfig /flushdns"
Andrew Tridgell [Mon, 28 Mar 2011 01:50:24 +0000 (12:50 +1100)]
wintest: added a "ipconfig /flushdns"

w2k3 doesn't flush its DNS cache on a dynamic update

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agowintest: fixed syntax highlighting with emacs
Andrew Tridgell [Mon, 28 Mar 2011 01:27:33 +0000 (12:27 +1100)]
wintest: fixed syntax highlighting with emacs

13 years agowintest: ensure we startup the w2k3 DC VM
Andrew Tridgell [Mon, 28 Mar 2011 01:27:01 +0000 (12:27 +1100)]
wintest: ensure we startup the w2k3 DC VM

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agoBe a little clearer about when and when not to set this option.
Jeremy Allison [Mon, 28 Mar 2011 21:12:36 +0000 (14:12 -0700)]
Be a little clearer about when and when not to set this option.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Mar 28 23:59:47 CEST 2011 on sn-devel-104

13 years agoFix inspired by work done by David Disseldorp for bug #8040 - smbclient segfaults...
Jeremy Allison [Mon, 28 Mar 2011 20:26:27 +0000 (13:26 -0700)]
Fix inspired by work done by David Disseldorp for bug #8040 - smbclient segfaults when a Cyrillic netbios name or workgroup is configured.

Change msrpc_gen to return NTSTATUS and ensure everywhere this is
used it is correctly checked to return that status.

Jeremy.

13 years agos3: Fix Coverity ID 2143: DEADCODE
Volker Lendecke [Mon, 28 Mar 2011 09:12:32 +0000 (11:12 +0200)]
s3: Fix Coverity ID 2143: DEADCODE

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Mar 28 12:06:19 CEST 2011 on sn-devel-104

13 years agos3: Fix Coverity ID 2143: DEADCODE
Volker Lendecke [Mon, 28 Mar 2011 09:12:32 +0000 (11:12 +0200)]
s3: Fix Coverity ID 2143: DEADCODE

13 years agos3: Fix Coverity ID 2173: FORWARD_NULL
Volker Lendecke [Mon, 28 Mar 2011 08:52:51 +0000 (10:52 +0200)]
s3: Fix Coverity ID 2173: FORWARD_NULL

13 years agoRevert "Change strict allocate to default to true.
Stefan Metzmacher [Mon, 28 Mar 2011 06:11:01 +0000 (08:11 +0200)]
Revert "Change strict allocate to default to true.

This reverts commit ecf48af135e4c1ebc5aafe4b3dad785162f5949a.

This makes Samba unusable on systems without Linux and
a modern Filesystem.

This was discussed with Jeremy on IRC:
http://irclog.samba.org/2011/01/20110126-Wed.log

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Mar 28 09:00:09 CEST 2011 on sn-devel-104

13 years agos3: Fix Coverity ID 2188: MISSING_BREAK
Volker Lendecke [Sun, 27 Mar 2011 20:06:46 +0000 (22:06 +0200)]
s3: Fix Coverity ID 2188: MISSING_BREAK

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 23:11:10 CEST 2011 on sn-devel-104

13 years agos3: Fix Coverity ID 2189: MISSING_BREAK
Volker Lendecke [Sun, 27 Mar 2011 20:06:46 +0000 (22:06 +0200)]
s3: Fix Coverity ID 2189: MISSING_BREAK

13 years agotdb: Fix Coverity ID 2192: NO_EFFECT
Volker Lendecke [Sun, 27 Mar 2011 19:43:53 +0000 (21:43 +0200)]
tdb: Fix Coverity ID 2192: NO_EFFECT

(ret < 0) can never be true

13 years agos3: Fix Coverity ID 2195: NO_EFFECT
Volker Lendecke [Sun, 27 Mar 2011 19:31:44 +0000 (21:31 +0200)]
s3: Fix Coverity ID 2195: NO_EFFECT

level is unsigned

13 years agolibrpc: Add some error checking to dcerpc_floor_pack_rhs_if_version_data
Volker Lendecke [Sun, 27 Mar 2011 19:19:37 +0000 (21:19 +0200)]
librpc: Add some error checking to dcerpc_floor_pack_rhs_if_version_data

13 years agolibrpc: Fix Coverity ID 2198: NULL_RETURNS
Volker Lendecke [Sun, 27 Mar 2011 19:17:53 +0000 (21:17 +0200)]
librpc: Fix Coverity ID 2198: NULL_RETURNS

13 years agos3: Fix Coverity ID 2200: NULL_RETURNS (cut&paste error)
Volker Lendecke [Sun, 27 Mar 2011 18:57:45 +0000 (20:57 +0200)]
s3: Fix Coverity ID 2200: NULL_RETURNS (cut&paste error)

13 years agos3: Fix Coverity ID 2228: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 18:44:01 +0000 (20:44 +0200)]
s3: Fix Coverity ID 2228: RESOURCE_LEAK

Holger, please check!

13 years agos3: Fix Coverity ID 2226: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 18:38:52 +0000 (20:38 +0200)]
s3: Fix Coverity ID 2226: RESOURCE_LEAK

Holger, please check!

13 years agos3: Fix Coverity ID 976: BAD_SIZEOF
Volker Lendecke [Sun, 27 Mar 2011 18:06:19 +0000 (20:06 +0200)]
s3: Fix Coverity ID 976: BAD_SIZEOF

13 years agos3: Fix Coverity ID 958: BAD_SIZEOF
Volker Lendecke [Sun, 27 Mar 2011 18:03:23 +0000 (20:03 +0200)]
s3: Fix Coverity ID 958: BAD_SIZEOF

This is supposed to wipe out the md5 context, not only the first bytes of it.

Others, please check!

13 years agos3: Fix Coverity ID 682: NEGATIVE_RETURNS
Volker Lendecke [Sun, 27 Mar 2011 17:41:34 +0000 (19:41 +0200)]
s3: Fix Coverity ID 682: NEGATIVE_RETURNS

13 years agos3: Fix Coverity ID 590: DEADCODE
Volker Lendecke [Sun, 27 Mar 2011 17:29:42 +0000 (19:29 +0200)]
s3: Fix Coverity ID 590: DEADCODE

13 years agos3: Fix Coverity ID 585: NEGATIVE_RETURNS
Volker Lendecke [Sun, 27 Mar 2011 17:27:26 +0000 (19:27 +0200)]
s3: Fix Coverity ID 585: NEGATIVE_RETURNS

13 years agos3: Fix Coverity ID 513: UNINIT
Volker Lendecke [Sun, 27 Mar 2011 17:11:44 +0000 (19:11 +0200)]
s3: Fix Coverity ID 513: UNINIT

13 years agos3: Fix Coverity ID 83: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 17:08:10 +0000 (19:08 +0200)]
s3: Fix Coverity ID 83: RESOURCE_LEAK

13 years agos3: Convert cli_raw_tcon to cli_smb
Volker Lendecke [Sun, 27 Mar 2011 16:03:24 +0000 (18:03 +0200)]
s3: Convert cli_raw_tcon to cli_smb

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 19:50:21 CEST 2011 on sn-devel-104

13 years agos3: Fix a memleak
Volker Lendecke [Sun, 27 Mar 2011 15:43:11 +0000 (17:43 +0200)]
s3: Fix a memleak

13 years agos3: Replace a use of cli_errstr
Volker Lendecke [Sun, 27 Mar 2011 15:37:24 +0000 (17:37 +0200)]
s3: Replace a use of cli_errstr

13 years agos3: Replace cli_negprot_sendsync() by cli_negprot_send()
Volker Lendecke [Sun, 27 Mar 2011 15:34:36 +0000 (17:34 +0200)]
s3: Replace cli_negprot_sendsync() by cli_negprot_send()

13 years agos3: Remove a pointless static
Volker Lendecke [Sun, 27 Mar 2011 15:20:28 +0000 (17:20 +0200)]
s3: Remove a pointless static

13 years agos3: Remove a use of cli_send_smb
Volker Lendecke [Sun, 27 Mar 2011 15:04:56 +0000 (17:04 +0200)]
s3: Remove a use of cli_send_smb

13 years agos3: Remove two calls to cli_errstr
Volker Lendecke [Sun, 27 Mar 2011 09:50:55 +0000 (11:50 +0200)]
s3: Remove two calls to cli_errstr

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 13:17:47 CEST 2011 on sn-devel-104

13 years agos3: Fix Coverity ID 2296: UNUSED_VALUE
Volker Lendecke [Sun, 27 Mar 2011 09:22:57 +0000 (11:22 +0200)]
s3: Fix Coverity ID 2296: UNUSED_VALUE

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 12:12:35 CEST 2011 on sn-devel-104

13 years agos3: Fix Coverity ID 2327: FORWARD_NULL
Volker Lendecke [Sun, 27 Mar 2011 09:11:07 +0000 (11:11 +0200)]
s3: Fix Coverity ID 2327: FORWARD_NULL

Make check_ft_info robust against an invalid type.

Günther, please check!

13 years agos3: Fix Coverity ID 2328: FORWARD_NULL
Volker Lendecke [Sun, 27 Mar 2011 09:06:02 +0000 (11:06 +0200)]
s3: Fix Coverity ID 2328: FORWARD_NULL

Make startsmbfilepwent robust against an invalid open type

13 years agos3: Fix Coverity ID 2329: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 09:00:36 +0000 (11:00 +0200)]
s3: Fix Coverity ID 2329: RESOURCE_LEAK

13 years agos3: Fix Coverity ID 2330: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 08:59:57 +0000 (10:59 +0200)]
s3: Fix Coverity ID 2330: RESOURCE_LEAK

13 years agos3: Fix Coverity ID 2331: RESOURCE_LEAK
Volker Lendecke [Sun, 27 Mar 2011 08:58:19 +0000 (10:58 +0200)]
s3: Fix Coverity ID 2331: RESOURCE_LEAK

13 years agos3: Fix an uninitialized variable read
Volker Lendecke [Sat, 26 Mar 2011 12:33:40 +0000 (13:33 +0100)]
s3: Fix an uninitialized variable read

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

13 years agos3: Fix Coverity ID 2326, CHECKED_RETURN
Volker Lendecke [Sat, 26 Mar 2011 12:29:37 +0000 (13:29 +0100)]
s3: Fix Coverity ID 2326, CHECKED_RETURN

13 years agoRevert "Fix bug 8021 - Incorrect string termination in volume/volume_name for TRANS2...
Jeremy Allison [Fri, 25 Mar 2011 22:43:54 +0000 (15:43 -0700)]
Revert "Fix bug 8021 - Incorrect string termination in volume/volume_name 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

13 years agoFix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup is confi...
Jeremy Allison [Fri, 25 Mar 2011 21:37:29 +0000 (14:37 -0700)]
Fix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup 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

13 years agos3-netapi: fix memoryleak while not using talloc_tos() in cli_get_session_key() usage.
Günther Deschner [Fri, 25 Mar 2011 11:13:36 +0000 (12:13 +0100)]
s3-netapi: fix memoryleak while not using talloc_tos() in 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

13 years agos3: Compile IDL files in autogen, some configure tests need this
Volker Lendecke [Fri, 25 Mar 2011 09:12:52 +0000 (10:12 +0100)]
s3: Compile IDL files in autogen, some configure tests need this

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Mar 25 16:35:30 CET 2011 on sn-devel-104

13 years agoRevert "s3-includes: fixed CONST_DISCARD() to actually discard const"
Volker Lendecke [Fri, 25 Mar 2011 10:31:38 +0000 (11:31 +0100)]
Revert "s3-includes: fixed CONST_DISCARD() to actually discard const"

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

13 years agocharcnv: ensure convert_string_error doesn't conflict with s3 function
Andrew Tridgell [Fri, 25 Mar 2011 03:34:43 +0000 (14:34 +1100)]
charcnv: ensure convert_string_error doesn't conflict with s3 function

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

13 years agos3-charcnv: fixed converted_size return in fast paths
Andrew Tridgell [Fri, 25 Mar 2011 01:10:41 +0000 (12:10 +1100)]
s3-charcnv: fixed converted_size return in fast paths

13 years agos3-charcnv: removed unused function
Andrew Tridgell [Fri, 25 Mar 2011 00:41:52 +0000 (11:41 +1100)]
s3-charcnv: removed unused function

lp_failed_convert_char() is not needed any more

13 years agolib/util/charset rename iconv_convenience to iconv_handle
Andrew Bartlett [Thu, 24 Mar 2011 21:37:00 +0000 (08:37 +1100)]
lib/util/charset rename iconv_convenience to iconv_handle

This better reflects what this structure is

Andrew Bartlett

13 years agos3-charcnv: convert_string_internal() should not display errors
Andrew Tridgell [Thu, 24 Mar 2011 23:52:32 +0000 (10:52 +1100)]
s3-charcnv: convert_string_internal() should not display errors

debug error display happens in the convert_string() outer function

13 years agos3-string: removed the conv_silent global
Andrew Tridgell [Thu, 24 Mar 2011 04:43:02 +0000 (15:43 +1100)]
s3-string: removed the conv_silent global

use convert_string_error() instead

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos3-string: removed last use of conv_silent
Andrew Tridgell [Thu, 24 Mar 2011 04:44:03 +0000 (15:44 +1100)]
s3-string: removed last use of conv_silent

This replaces the push_ascii_nstring() implementation with a call to
convert_string_error()

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos3-charcnv Add convert_string_error()
Andrew Bartlett [Thu, 24 Mar 2011 22:15:06 +0000 (09:15 +1100)]
s3-charcnv Add convert_string_error()

This function returns errors rather than printing them.

Andrew Bartlett

13 years agocharcnv: added convert_string_error()
Andrew Tridgell [Thu, 24 Mar 2011 04:59:26 +0000 (15:59 +1100)]
charcnv: added convert_string_error()

this will allow us to remove conv_silent

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos3-includes: fixed CONST_DISCARD() to actually discard const
Andrew Tridgell [Thu, 24 Mar 2011 04:40:45 +0000 (15:40 +1100)]
s3-includes: fixed CONST_DISCARD() to actually discard const

a cast is not enough

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agounistr: moved some UCS2 macros into util_unistr
Andrew Tridgell [Thu, 24 Mar 2011 04:39:27 +0000 (15:39 +1100)]
unistr: moved some UCS2 macros into util_unistr

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>

13 years agolibcli: created smb_constants.h
Andrew Tridgell [Thu, 24 Mar 2011 04:35:28 +0000 (15:35 +1100)]
libcli: created smb_constants.h

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)

13 years agotdb: rename convert_string() to tdb_convert_string()
Andrew Tridgell [Thu, 24 Mar 2011 04:33:39 +0000 (15:33 +1100)]
tdb: rename convert_string() to tdb_convert_string()

this prevents a conflict with the convert_string() in samba

13 years agos3-libsmb: work our correct length for EA strings
Andrew Tridgell [Thu, 24 Mar 2011 03:45:27 +0000 (14:45 +1100)]
s3-libsmb: work our correct length for EA strings

13 years agos3-string: sec_len==-1 support is no longer needed
Andrew Tridgell [Thu, 24 Mar 2011 02:00:05 +0000 (13:00 +1100)]
s3-string: sec_len==-1 support is no longer needed

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

13 years agos3-dfs: removed the last user of sec_len==-1 in pull_*_base_talloc()
Andrew Tridgell [Thu, 24 Mar 2011 02:00:37 +0000 (13:00 +1100)]
s3-dfs: removed the last user of sec_len==-1 in pull_*_base_talloc()

hopefully there aren't any others lurking in the code

13 years agos3-lib: make pull_ucs2_base_talloc static
Andrew Tridgell [Thu, 24 Mar 2011 01:09:03 +0000 (12:09 +1100)]
s3-lib: make pull_ucs2_base_talloc static

it is local to charcnv.c

13 years agolib: remove unused pieces of string_wrappers.h
Andrew Tridgell [Thu, 24 Mar 2011 00:42:40 +0000 (11:42 +1100)]
lib: remove unused pieces of string_wrappers.h

13 years agolib: move the string wrappers from source3/include to common lib/util
Andrew Tridgell [Thu, 24 Mar 2011 00:40:43 +0000 (11:40 +1100)]
lib: move the string wrappers from source3/include to common lib/util

this will allow the common charcnv library to use the string wrappers

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>