nivanova/samba-autobuild/.git
5 years agosmbd: validate oplock types even for internal and stat opens
Volker Lendecke [Tue, 15 Oct 2013 12:41:11 +0000 (12:41 +0000)]
smbd: validate oplock types even for internal and stat opens

There's no reason why we should not do this. This has turned into a pure
internal consistency check that should apply fine every time.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agotorture: Add reproducer for bug 10216
Volker Lendecke [Fri, 18 Oct 2013 13:11:38 +0000 (13:11 +0000)]
torture: Add reproducer for bug 10216

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agosmbd: Fix bug 10216
Volker Lendecke [Fri, 18 Oct 2013 15:12:35 +0000 (15:12 +0000)]
smbd: Fix bug 10216

While refactoring find_oplock_types to validate_oplock_types I forgot
that stat opens will end up in locking.tdb. So even with a batch oplock
around we can have more than one entry. This means the consistency check
in validate_oplock_types was wrong and too strict.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agoldb: Fix CID 240798 Uninitialized pointer read
Volker Lendecke [Sun, 20 Oct 2013 15:29:29 +0000 (17:29 +0200)]
ldb: Fix CID 240798 Uninitialized pointer read

Not called right now, because nobody tries multiple sort attributes. But if
someone did, build_response would have looked at the uninitialized controls.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Tue Oct 22 03:28:20 CEST 2013 on sn-devel-104

5 years agowinbind3: Fix CID 241468 Resource leak
Volker Lendecke [Sun, 20 Oct 2013 15:25:27 +0000 (17:25 +0200)]
winbind3: Fix CID 241468 Resource leak

We were leaking centry in this error case

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agontvfs: Fix CID 1107225 Resource leak
Volker Lendecke [Sun, 20 Oct 2013 15:16:11 +0000 (17:16 +0200)]
ntvfs: Fix CID 1107225 Resource leak

After the talloc_strdup, we don't need cwd anymore.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agontvfs: Fix Coverity ID 1107224 Resource leak
Volker Lendecke [Sun, 20 Oct 2013 15:11:16 +0000 (17:11 +0200)]
ntvfs: Fix Coverity ID 1107224 Resource leak

In this error path we were leaking "fd".

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agoidmap_autorid: fix traversal of ranges to correctly exclude mapping records.
Michael Adam [Fri, 18 Oct 2013 14:26:41 +0000 (16:26 +0200)]
idmap_autorid: fix traversal of ranges to correctly exclude mapping records.

When listing all ranges with "net idmap get ranges", the database records
of the form "S-1-5-10" --> "GID 100016" were treated here as rangee mapings,
resulting in an faulty printout like this: "RANGE 541346119: S-1-5-10".

This patch fixes this by adding a filter that checks the size of the
value datablob.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Mon Oct 21 19:48:40 CEST 2013 on sn-devel-104

5 years agonsswitch: Fix short writes in winbind_write_sock
Volker Lendecke [Tue, 15 Oct 2013 08:23:10 +0000 (08:23 +0000)]
nsswitch: Fix short writes in winbind_write_sock

We set the socket to nonblocking and don't handle EAGAIN right. We do
a poll anyway, so wait for writability, which should fix this.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10195
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
5 years agos3-rpc_server: Refactor lsasd_create_sockets().
Andreas Schneider [Mon, 21 Oct 2013 06:26:47 +0000 (08:26 +0200)]
s3-rpc_server: Refactor lsasd_create_sockets().

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Mon Oct 21 12:49:44 CEST 2013 on sn-devel-104

5 years agolsasd: Fix CID 1107233 Double close
Volker Lendecke [Sun, 20 Oct 2013 15:37:08 +0000 (17:37 +0200)]
lsasd: Fix CID 1107233 Double close

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
5 years agowinbind3: Fix CID 1107229 Uninitialized pointer read
Volker Lendecke [Wed, 16 Oct 2013 19:02:42 +0000 (21:02 +0200)]
winbind3: Fix CID 1107229 Uninitialized pointer read

The first "goto done" would TALLOC_FREE the uninitialized "value"

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sat Oct 19 03:43:04 CEST 2013 on sn-devel-104

5 years agoscript/mks3param_ctx_table.pl: fix tabs/whitespaces in generated output.
Stefan Metzmacher [Thu, 17 Oct 2013 07:28:57 +0000 (09:28 +0200)]
script/mks3param_ctx_table.pl: fix tabs/whitespaces in generated output.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Fri Oct 18 16:20:01 CEST 2013 on sn-devel-104

5 years agoReserve an OID space for external projects
Simo Sorce [Thu, 17 Oct 2013 19:59:55 +0000 (15:59 -0400)]
Reserve an OID space for external projects

Some external, but somewhat related projects, benefit from being
able to use the Samba OID space instead of having to go through IANA.

Reserve 1.3.6.1.4.1.7165.655.x for external projects
And assign 1.3.6.1.4.1.7165.655.1.x to the GSS-NTLMSSP project.

Signed-off-by: Simo Sorce <idra@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Simo Sorce <idra@samba.org>
Autobuild-Date(master): Fri Oct 18 05:47:29 CEST 2013 on sn-devel-104

5 years agos3:nmbd: do a very early cluster probe when starting nmbd.
Michael Adam [Thu, 17 Oct 2013 13:19:41 +0000 (15:19 +0200)]
s3:nmbd: do a very early cluster probe when starting nmbd.

Just as for smbd and winbindd

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Fri Oct 18 01:17:14 CEST 2013 on sn-devel-104

5 years agos3:winbindd: use cluster_probe_ok()
Michael Adam [Thu, 17 Oct 2013 13:16:19 +0000 (15:16 +0200)]
s3:winbindd: use cluster_probe_ok()

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
5 years agos3:smbd: use new function cluster_probe_ok()
Michael Adam [Thu, 17 Oct 2013 13:15:51 +0000 (15:15 +0200)]
s3:smbd: use new function cluster_probe_ok()

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
5 years agos3: add cluster_probe_ok() in a new module util_cluster.
Michael Adam [Thu, 17 Oct 2013 13:10:11 +0000 (15:10 +0200)]
s3: add cluster_probe_ok() in a new module util_cluster.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
5 years agosmbd/winbindd: Do an early check if ctdbd is functional
Volker Lendecke [Thu, 31 Jan 2013 10:15:09 +0000 (11:15 +0100)]
smbd/winbindd: Do an early check if ctdbd is functional

This will avoid panic calls when smbd and winbind is started in cluster
mode before ctdb is functional. It still logs something sane at debug
level 0, but it does not panic and core anymore.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agoctdbd_conn: Remove one call to cluster_fatal
Volker Lendecke [Thu, 31 Jan 2013 10:02:52 +0000 (11:02 +0100)]
ctdbd_conn: Remove one call to cluster_fatal

This is during startup of a ctdb connection, thus it is not as important
as in other cases to immediately exit to free up resources

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agoctdbd_conn: Remove one call to cluster_fatal
Volker Lendecke [Thu, 31 Jan 2013 10:02:52 +0000 (11:02 +0100)]
ctdbd_conn: Remove one call to cluster_fatal

This is during startup of a ctdb connection, thus it is not as important
as in other cases to immediately exit to free up resources

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agoctdbd_conn: Lift the cluster_fatal call in get_cluster_vnn
Volker Lendecke [Thu, 31 Jan 2013 09:54:48 +0000 (10:54 +0100)]
ctdbd_conn: Lift the cluster_fatal call in get_cluster_vnn

We have to report a proper error when ctdbd is not around

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3:libsmb: call smbXcli_tcon_set_fs_attributes() directly
Stefan Metzmacher [Fri, 27 Sep 2013 04:06:32 +0000 (06:06 +0200)]
s3:libsmb: call smbXcli_tcon_set_fs_attributes() directly

We should try to set all attributes we got and not just
FILE_CASE_SENSITIVE_SEARCH.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Thu Oct 17 19:02:03 CEST 2013 on sn-devel-104

5 years agos3:libsmb: remove unused cli_state->case_sensitive
Stefan Metzmacher [Fri, 27 Sep 2013 03:45:21 +0000 (05:45 +0200)]
s3:libsmb: remove unused cli_state->case_sensitive

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3:libsmb: remove unused cli_state->dfsroot
Stefan Metzmacher [Wed, 18 Sep 2013 07:40:08 +0000 (09:40 +0200)]
s3:libsmb: remove unused cli_state->dfsroot

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3:libsmb: add SMB2/3 support to cli_dfs_get_referral()
Stefan Metzmacher [Sun, 13 Oct 2013 14:54:18 +0000 (16:54 +0200)]
s3:libsmb: add SMB2/3 support to cli_dfs_get_referral()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3/libsmb: make use of smbXcli_tcon_is_dfs_share()
Stefan Metzmacher [Fri, 27 Sep 2013 03:44:33 +0000 (05:44 +0200)]
s3/libsmb: make use of smbXcli_tcon_is_dfs_share()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3/libsmb: Use smbXcli_conn_dfs_supported instead of test on CAP_DFS
Luk Claes [Fri, 17 May 2013 18:46:24 +0000 (20:46 +0200)]
s3/libsmb: Use smbXcli_conn_dfs_supported instead of test on CAP_DFS

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Luk Claes <luk@debian.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agos3:libsmb: don't pass down FLAG_CASELESS_PATHNAMES and FLAGS2_DFS_PATHNAMES anymore
Stefan Metzmacher [Fri, 27 Sep 2013 03:34:12 +0000 (05:34 +0200)]
s3:libsmb: don't pass down FLAG_CASELESS_PATHNAMES and FLAGS2_DFS_PATHNAMES anymore

They're now handled at the smbXcli_conn and smbXcli_tcon layer.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3:libsmb: call smbXcli_tcon_{get,set}_fs_attributes() from cli_set_case_sensitive()
Stefan Metzmacher [Fri, 27 Sep 2013 03:11:45 +0000 (05:11 +0200)]
s3:libsmb: call smbXcli_tcon_{get,set}_fs_attributes() from cli_set_case_sensitive()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: add SMB2_HDR_FLAG_DFS for SMB2 Create operations on dfs shares
Stefan Metzmacher [Wed, 18 Sep 2013 01:50:42 +0000 (03:50 +0200)]
libcli/smb: add SMB2_HDR_FLAG_DFS for SMB2 Create operations on dfs shares

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: add FLAG_CASELESS_PATHNAMES based on FILE_CASE_SENSITIVE_SEARCH to smb1...
Stefan Metzmacher [Fri, 27 Sep 2013 03:31:07 +0000 (05:31 +0200)]
libcli/smb: add FLAG_CASELESS_PATHNAMES based on FILE_CASE_SENSITIVE_SEARCH to smb1 requests

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: move Filesystem Attributes defines to smb_constants.h
Stefan Metzmacher [Fri, 27 Sep 2013 03:30:03 +0000 (05:30 +0200)]
libcli/smb: move Filesystem Attributes defines to smb_constants.h

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: add FLAGS2_DFS_PATHNAMES for SMB1 operations against dfs shares
Stefan Metzmacher [Wed, 18 Sep 2013 01:31:33 +0000 (03:31 +0200)]
libcli/smb: add FLAGS2_DFS_PATHNAMES for SMB1 operations against dfs shares

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: add smbXcli_tcon_{set,get}_fs_attributes()
Stefan Metzmacher [Fri, 27 Sep 2013 03:02:09 +0000 (05:02 +0200)]
libcli/smb: add smbXcli_tcon_{set,get}_fs_attributes()

These are the attributes returned from the FileFsAttributeInformation
request.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: add smbXcli_tcon_is_dfs_share()
Stefan Metzmacher [Fri, 13 Sep 2013 09:07:32 +0000 (11:07 +0200)]
libcli/smb: add smbXcli_tcon_is_dfs_share()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agolibcli/smb: Introduce smbXcli_conn_dfs_supported
Luk Claes [Fri, 17 May 2013 18:41:45 +0000 (20:41 +0200)]
libcli/smb: Introduce smbXcli_conn_dfs_supported

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Luk Claes <luk@debian.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3:libsmb: make use of smb1cli_tcon_set_values()
Stefan Metzmacher [Fri, 13 Sep 2013 07:10:42 +0000 (09:10 +0200)]
s3:libsmb: make use of smb1cli_tcon_set_values()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10200

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos3/libsmb: Use smbXcli_conn_use_unicode instead of smb1 specific test
Luk Claes [Fri, 17 May 2013 18:13:50 +0000 (20:13 +0200)]
s3/libsmb: Use smbXcli_conn_use_unicode instead of smb1 specific test

Signed-off-by: Luk Claes <luk@debian.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agoauth:credentials: avoid talloc_reference in cli_credentials_set_netlogon_creds()
Stefan Metzmacher [Fri, 20 Sep 2013 02:33:07 +0000 (04:33 +0200)]
auth:credentials: avoid talloc_reference in cli_credentials_set_netlogon_creds()

Typically cli_credentials_set_netlogon_creds() should be used directly
before the DCERPC bind. And cli_credentials_get_netlogon_creds()
should be only used by the gensec layer, which only needs a copy.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Wed Oct 16 23:56:01 CEST 2013 on sn-devel-104

5 years agos4:librpc: let dcerpc_schannel_key_recv() return netlogon_creds_CredentialState
Stefan Metzmacher [Fri, 20 Sep 2013 02:16:00 +0000 (04:16 +0200)]
s4:librpc: let dcerpc_schannel_key_recv() return netlogon_creds_CredentialState

cli_credentials_set_netlogon_creds() should only be used directly before
a DCERPC bind in order to pass the session information to the
gensec layer.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc: make dcerpc_schannel_key_send/recv static
Stefan Metzmacher [Fri, 20 Sep 2013 02:14:00 +0000 (04:14 +0200)]
s4:librpc: make dcerpc_schannel_key_send/recv static

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:libsmb: inline trust_pw_change_and_store_it() into trust_pw_find_change_and_store_it()
Stefan Metzmacher [Mon, 2 Sep 2013 18:19:28 +0000 (20:19 +0200)]
s3:libsmb: inline trust_pw_change_and_store_it() into trust_pw_find_change_and_store_it()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:libsmb: remove unused change_trust_account_password()
Stefan Metzmacher [Mon, 2 Sep 2013 18:18:39 +0000 (20:18 +0200)]
s3:libsmb: remove unused change_trust_account_password()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:auth_domain: remove dead code in check_trustdomain_security()
Stefan Metzmacher [Tue, 27 Aug 2013 10:09:51 +0000 (12:09 +0200)]
s3:auth_domain: remove dead code in check_trustdomain_security()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:include: fix messaging_send_buf() protype in messages.h
Stefan Metzmacher [Mon, 2 Sep 2013 17:29:05 +0000 (19:29 +0200)]
s3:include: fix messaging_send_buf() protype in messages.h

The function already used 'uint8_t' instead of 'uint8'.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:include: add forward declaration for struct messaging_context; in g_lock.h
Stefan Metzmacher [Mon, 2 Sep 2013 07:24:42 +0000 (09:24 +0200)]
s3:include: add forward declaration for struct messaging_context; in g_lock.h

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:winbindd: don't hide the error in cm_connect_lsa()
Stefan Metzmacher [Tue, 17 Sep 2013 02:25:39 +0000 (04:25 +0200)]
s3:winbindd: don't hide the error in cm_connect_lsa()

We should not overwrite the error with NT_STATUS_PIPE_NOT_AVAILABLE.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc/rpc: remove unused dcerpc_smb_tree()
Stefan Metzmacher [Tue, 17 Sep 2013 05:19:42 +0000 (07:19 +0200)]
s4:librpc/rpc: remove unused dcerpc_smb_tree()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc/rpc: make use of dcerpc_secondary_smb_send/recv() in dcerpc_secondary_context()
Stefan Metzmacher [Tue, 17 Sep 2013 05:18:19 +0000 (07:18 +0200)]
s4:librpc/rpc: make use of dcerpc_secondary_smb_send/recv() in dcerpc_secondary_context()

This avoids the use of dcerpc_smb_tree(), which is a layer violation.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc/rpc: add dcerpc_secondary_smb_send/recv()
Stefan Metzmacher [Tue, 17 Sep 2013 05:16:29 +0000 (07:16 +0200)]
s4:librpc/rpc: add dcerpc_secondary_smb_send/recv()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc/rpc: remove unused dcerpc_smb_fnum()
Stefan Metzmacher [Tue, 17 Sep 2013 05:04:55 +0000 (07:04 +0200)]
s4:librpc/rpc: remove unused dcerpc_smb_fnum()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agocli_np_tstream: remove unused tstream_cli_np_get_cli_state()
Gregor Beck [Fri, 13 Sep 2013 09:18:42 +0000 (11:18 +0200)]
cli_np_tstream: remove unused tstream_cli_np_get_cli_state()

Signed-off-by: Gregor Beck <gbeck@sernet.de>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:rpcclient: make use of rpcclient_cli_state
Stefan Metzmacher [Fri, 13 Sep 2013 09:06:30 +0000 (11:06 +0200)]
s3:rpcclient: make use of rpcclient_cli_state

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:rpcclient: introduce global rpcclient_cli_state
Stefan Metzmacher [Fri, 13 Sep 2013 09:06:00 +0000 (11:06 +0200)]
s3:rpcclient: introduce global rpcclient_cli_state

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:rpc_client: remove unused rpc_pipe_np_smb_conn()
Gregor Beck [Fri, 13 Sep 2013 09:15:18 +0000 (11:15 +0200)]
s3:rpc_client: remove unused rpc_pipe_np_smb_conn()

Signed-off-by: Gregor Beck <gbeck@sernet.de>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3-spoolss: do not use rpc_pipe_np_smb_conn()
Gregor Beck [Fri, 13 Sep 2013 09:13:15 +0000 (11:13 +0200)]
s3-spoolss: do not use rpc_pipe_np_smb_conn()

Signed-off-by: Gregor Beck <gbeck@sernet.de>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:lib/netapi: do not use rpc_pipe_np_smb_conn()
Gregor Beck [Fri, 13 Sep 2013 09:12:25 +0000 (11:12 +0200)]
s3:lib/netapi: do not use rpc_pipe_np_smb_conn()

Signed-off-by: Gregor Beck <gbeck@sernet.de>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3-net: do not use rpc_pipe_np_smb_conn()
Gregor Beck [Fri, 13 Sep 2013 09:11:55 +0000 (11:11 +0200)]
s3-net: do not use rpc_pipe_np_smb_conn()

Signed-off-by: Gregor Beck <gbeck@sernet.de>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:tortore:raw/qfileinfo: don't use dcerpc_pipe_open_smb() to open a named pipe
Stefan Metzmacher [Tue, 17 Sep 2013 05:01:57 +0000 (07:01 +0200)]
s4:tortore:raw/qfileinfo: don't use dcerpc_pipe_open_smb() to open a named pipe

We can directly use smb_raw_open() to open a handle to a named pipe.
This avoids the need for the layer violation functions
dcerpc_smb_tree() and dcerpc_smb_fnum().

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:torture/samba3rpc: let rpc.authcontext be more robust against low level changes...
Stefan Metzmacher [Tue, 17 Sep 2013 11:15:38 +0000 (13:15 +0200)]
s4:torture/samba3rpc: let rpc.authcontext be more robust against low level changes (part2)

We now use smbXcli_conn_is_connected() and
dcerpc_binding_handle_is_connected() to verify only the dcerpc layer
got an error. The expected error is EIO mapped to NT_STATUS_IO_DEVICE_ERROR.

NT_STATUS_INVALID_HANDLE should only be visible at the SMB layer,
but we keep this as allowed return value for now, until
the dcerpc layer is fixed.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:torture/samba3rpc: let rpc.authcontext be more robust against low level code chang...
Stefan Metzmacher [Tue, 17 Sep 2013 11:15:38 +0000 (13:15 +0200)]
s4:torture/samba3rpc: let rpc.authcontext be more robust against low level code changes (part1)

Some code uses the low level smbXcli_session structure instead of
the smbcli_session structure and doesn't 'see' updates to the
smbcli_session structure.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:rpc_client: pass object and table to rpccli_bh_create()
Stefan Metzmacher [Tue, 27 Aug 2013 11:23:04 +0000 (13:23 +0200)]
s3:rpc_client: pass object and table to rpccli_bh_create()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos4:librpc: implement dcerpc_binding_handle_auth_info()
Stefan Metzmacher [Mon, 12 Aug 2013 06:19:57 +0000 (08:19 +0200)]
s4:librpc: implement dcerpc_binding_handle_auth_info()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agos3:rpc_client: implement dcerpc_binding_handle_auth_info()
Stefan Metzmacher [Mon, 12 Aug 2013 06:19:35 +0000 (08:19 +0200)]
s3:rpc_client: implement dcerpc_binding_handle_auth_info()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agolibrpc/rpc: add dcerpc_binding_handle_auth_info()
Stefan Metzmacher [Mon, 12 Aug 2013 06:19:08 +0000 (08:19 +0200)]
librpc/rpc: add dcerpc_binding_handle_auth_info()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agodoc: add "spoolss: architecture" parameter usage
David Disseldorp [Mon, 14 Oct 2013 10:01:58 +0000 (12:01 +0200)]
doc: add "spoolss: architecture" parameter usage

Windows spoolss print clients only allow association of server-side
drivers with printers when the driver architecture matches the
advertised print server architecture. Samba's spoolss print server
architecture can be changed using this parameter.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Wed Oct 16 17:12:34 CEST 2013 on sn-devel-104

5 years agolib/param: Add documentation on how loadparm works
Andrew Bartlett [Wed, 16 Oct 2013 01:45:31 +0000 (14:45 +1300)]
lib/param: Add documentation on how loadparm works

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed Oct 16 11:39:41 CEST 2013 on sn-devel-104

5 years agoproto: Remove manually written lp_ prototypes
Andrew Bartlett [Mon, 14 Oct 2013 02:49:25 +0000 (15:49 +1300)]
proto: Remove manually written lp_ prototypes

This also ensures we remove prototypes from parameters we remove or
rename, and easily see how many special cases we have left.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agoparam: Autogenerate s3 lp_ctx glue table
Andrew Bartlett [Mon, 14 Oct 2013 02:46:43 +0000 (15:46 +1300)]
param: Autogenerate s3 lp_ctx glue table

This allows us to use more lpcfg_ functions without adding them
manually.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agos3/param: Autogenerate parameters prototypes again after proto.h was frozen
Andrew Bartlett [Mon, 14 Oct 2013 02:39:10 +0000 (15:39 +1300)]
s3/param: Autogenerate parameters prototypes again after proto.h was frozen

This autogenerates the parameters so that we can keep everything in sync easier,
particularly when adding new parameters.  This will also make it easier to move
to a fully autogenerated system in the future, as it reduces special cases.

Andrew Bartlett

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agoparam: Skip generating hooks for local and string parameters
Andrew Bartlett [Mon, 14 Oct 2013 02:36:18 +0000 (15:36 +1300)]
param: Skip generating hooks for local and string parameters

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agolib/param: Do not attempt to access the s3 function for allocated and subbed string...
Andrew Bartlett [Mon, 14 Oct 2013 02:34:40 +0000 (15:34 +1300)]
lib/param: Do not attempt to access the s3 function for allocated and subbed string parameters

This allows us not to generate array entries for these, which in turn allows
us to avoid initialising them.  The issue is that we do not have the
% macro sub context nor a talloc context handy (yet).

Andrew Bartlett

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agobuild: Move loadparm-related build rules to source3/param/wscript_build
Andrew Bartlett [Mon, 14 Oct 2013 00:47:27 +0000 (13:47 +1300)]
build: Move loadparm-related build rules to source3/param/wscript_build

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agolib/param: lp_magicchar takes a const struct share_params *p so should be FN_LOCAL_PA...
Andrew Bartlett [Mon, 14 Oct 2013 02:33:20 +0000 (15:33 +1300)]
lib/param: lp_magicchar takes a const struct share_params *p so should be FN_LOCAL_PARM_CHAR

This was found when trying to autogenerate prototypes for lp_ functions again.

Andrew Bartlett

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
5 years agoAdd NetWkstaGetInfo.
Hans Leidekker [Mon, 14 Oct 2013 18:43:27 +0000 (20:43 +0200)]
Add NetWkstaGetInfo.

Modified to include common.h entry for netapitest_wksta function by Kai
Blin <kai@samba.org>

Signed-off-by: Hans Leidekker <hans@meelstraat.net>
Reviewed-by: Kai Blin <kai@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Oct 16 07:00:45 CEST 2013 on sn-devel-104

5 years agosmbd: Simplify open_file_ntcreate
Volker Lendecke [Thu, 26 Sep 2013 22:20:36 +0000 (15:20 -0700)]
smbd: Simplify open_file_ntcreate

Only one call to delay_for_oplocks left. Metze showed me the new logic:
BATCH is broken if we have a sharing violation. Exclusive is broken
otherwise. That's it.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Wed Oct 16 02:51:53 CEST 2013 on sn-devel-104

5 years agosmbd: Remove "file_existed" handling from open_mode_check
Volker Lendecke [Thu, 26 Sep 2013 21:32:09 +0000 (14:32 -0700)]
smbd: Remove "file_existed" handling from open_mode_check

No clue what this does. In open_directory, "dir_existed" is not used after
open_mode_check. In open_file_ntcreate it's used, but I can't think of a case
right now where we would find a formerly nonexisting file to exist suddenly.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Decouple grant_fsp_oplock_type from oplock validation
Volker Lendecke [Tue, 3 Sep 2013 09:02:12 +0000 (09:02 +0000)]
smbd: Decouple grant_fsp_oplock_type from oplock validation

This makes grant_fsp_oplock_type independent from the values computed
in validate_oplock_types. It *might* make oplock calculation a bit
slower for heavily shared files, as we are walking the share mode array
twice. But we are doing so much stuff in open that I doubt the difference
is measurable. It clears up the code for me however, and I think that's
worth it.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Reduce the complexity of open_file_ntcreate
Volker Lendecke [Mon, 2 Sep 2013 12:25:07 +0000 (12:25 +0000)]
smbd: Reduce the complexity of open_file_ntcreate

This removes two variables in open_file_ntcreate based on the observation
that for exclusive and batch oplocks there can only be one entry. So
in these cases we don't need to keep pointers from find_oplock_types to
delay_for_oplocks. We can just reference the only share mode entry around.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Factor out remove_stale_share_mode_entries
Volker Lendecke [Mon, 2 Sep 2013 11:37:57 +0000 (11:37 +0000)]
smbd: Factor out remove_stale_share_mode_entries

Will be used in the next commit

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Make find_oplock_types return bool
Volker Lendecke [Tue, 15 Oct 2013 10:01:54 +0000 (10:01 +0000)]
smbd: Make find_oplock_types return bool

smb_panic() does not take a printf style argument. This improves debug
output by easily printing the index that we fell over. Also, doing
smb_panic deep down is bad style IMHO.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Make loop index type match loop limit
Volker Lendecke [Tue, 15 Oct 2013 09:56:21 +0000 (09:56 +0000)]
smbd: Make loop index type match loop limit

share_mode_data.num_share_modes is a uint32.

48 bytes less in .o text size for -O3 :-)

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Unify delay_for_*_oplocks
Volker Lendecke [Tue, 20 Aug 2013 11:58:15 +0000 (11:58 +0000)]
smbd: Unify delay_for_*_oplocks

This is the same code in both routines

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Simplify find_oplock_types a bit
Volker Lendecke [Mon, 2 Sep 2013 12:33:40 +0000 (12:33 +0000)]
smbd: Simplify find_oplock_types a bit

Define a variable to dereference lck->data just once. Believe it or not,
this saves a few bytes .o with -O3 :-)

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
5 years agos4:torture: add smb2.session.reauth6 : test failing reauth
Michael Adam [Thu, 19 Sep 2013 20:00:19 +0000 (22:00 +0200)]
s4:torture: add smb2.session.reauth6 : test failing reauth

This attempts reauth with invalid creds, hence
triggering the error path in the reauth code.
This invalidates the session and subsequente requests
on that connection fail.

https://bugzilla.samba.org/show_bug.cgi?id=10208

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Tue Oct 15 22:50:27 CEST 2013 on sn-devel-104

5 years agolibcli/smb: add smb2cli_tcon_is_encryption_on()
Michael Adam [Fri, 20 Sep 2013 05:46:54 +0000 (07:46 +0200)]
libcli/smb: add smb2cli_tcon_is_encryption_on()

https://bugzilla.samba.org/show_bug.cgi?id=10208

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agosmbd:smb2: fix crash when smb2 session reauth fails
Michael Adam [Thu, 19 Sep 2013 21:41:51 +0000 (23:41 +0200)]
smbd:smb2: fix crash when smb2 session reauth fails

https://bugzilla.samba.org/show_bug.cgi?id=10208

Authentication error in smb2 session reauth invalidates
the session. In this case the session must in contrast
to successful session setup requests be torn down and live
no longer than the request.

The talloc move of the session from the global session
table to the request ensures that the session setup
reply can still be correctly signed, but subsequent
requests on the connection don't find a session any more.

Pair-Programmed-With: Jeremy Allison <jra@samba.org>
Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>

Signed-off-by: Michael Adam <obnox@samba.org>
5 years agosmbd: Inline break_level2_to_none_async
Volker Lendecke [Fri, 4 Oct 2013 10:11:38 +0000 (10:11 +0000)]
smbd: Inline break_level2_to_none_async

With the special case for bug 5980 in do_break_to_none we only have
one caller: process_oplock_async_level2_break_message. The further
goal is to merge process_oplock_async_level2_break_message with
process_oplock_break_message.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Tue Oct 15 03:42:53 CEST 2013 on sn-devel-104

5 years agosmbd: Remove a special case for level2 break
Volker Lendecke [Fri, 4 Oct 2013 09:24:29 +0000 (09:24 +0000)]
smbd: Remove a special case for level2 break

With the level2 indicator in brlock.tdb this race condition does not
exist anymore

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agosmbd: Remove some FAKE_LEVEL_II comments
Volker Lendecke [Fri, 13 Sep 2013 13:18:15 +0000 (15:18 +0200)]
smbd: Remove some FAKE_LEVEL_II comments

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agosmbd: Add debugs to brlock.c
Volker Lendecke [Fri, 13 Sep 2013 12:13:51 +0000 (14:13 +0200)]
smbd: Add debugs to brlock.c

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agotorture: Extend raw.oplock.batch10
Volker Lendecke [Mon, 9 Sep 2013 18:53:15 +0000 (18:53 +0000)]
torture: Extend raw.oplock.batch10

With FAKE_LEVEL_II_OPLOCKS around we did not grant LEVEL2 after
a NO_OPLOCK file got written to. Windows does grant LEVEL2 in this
case. With the have_level2_oplocks in brlocks.tdb we can now grant LEVEL2
in this case as well.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agosmbd: Remove FAKE_LEVEL_II_OPLOCK
Volker Lendecke [Wed, 11 Sep 2013 16:07:33 +0000 (16:07 +0000)]
smbd: Remove FAKE_LEVEL_II_OPLOCK

FAKE_LEVEL_II_OPLOCK was an indicator to break level2 oplock holders
on write.  This information is now being held in brlock.tdb, which makes
the FAKE_LEVEL_II_OPLOCK type unnecessary.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agosmbd: Put "have_read_oplocks" into brlock.tdb
Volker Lendecke [Wed, 11 Sep 2013 12:48:14 +0000 (12:48 +0000)]
smbd: Put "have_read_oplocks" into brlock.tdb

This implements an idea by metze: Right now Samba does not grant level2
oplocks where it should: After an initial no-oplock open that has been
written to, we don't have the FAKE_LEVEL2_OPLOCK entry in locking.tdb
around anymore, this downgraded to NO_OPLOCK. Windows in this case will
grant level2 if being asked, we don't.  Part of the reason for this
is that we don't have a proper mechanism to communicate the fact that
level2 needs to be broken to other smbds. Metze's insight was that we
have to look into brlock.tdb for every write anyway, so this might be
the right place to store this information.

My first reaction was that this is really hackish, but on further thought
this is not. oplocks depend on brlocks anyway, and we have the proper
mechanisms in place for brlocks.

The format for this change is to add one byte to the end of the brlock.tdb
record with value 1 if we have level2 oplocks around. Thus this patch
effectively reverts 8f41142 which I discovered while writing this
change. We now legally have unaligned records.

We can certainly talk about the format, but I'm not yet convinced we
need an idl for this yet. This is a potentially very hot code path,
and ndr marshalling has a cost.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
5 years agodocs: Explain why this option should not be used
Andrew Bartlett [Fri, 11 Oct 2013 00:39:09 +0000 (13:39 +1300)]
docs: Explain why this option should not be used

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Oct 15 01:51:39 CEST 2013 on sn-devel-104

5 years agos3-winbindd: Remove undocumented winbindd:socket dir parameter
Andrew Bartlett [Fri, 11 Oct 2013 00:34:13 +0000 (13:34 +1300)]
s3-winbindd: Remove undocumented winbindd:socket dir parameter

This uses the documeted "winbindd socket directory" parameter instead.

This came about due to the merge of the two smb.conf tables in s3 and
s4 for the Samba 4.0 release.  The s4 code used a real parameter,
which caused this to be documented, whereas no automatic procedure
existed to notice the parametric option and the need to document that.
The fact that this was not used consistently in both codebases is one
of the many areas of technical debt we still need to pay off here.

Andrew Bartlett

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
5 years agoauth4: Remove an unused variable
Volker Lendecke [Sun, 13 Oct 2013 11:58:44 +0000 (13:58 +0200)]
auth4: Remove an unused variable

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
5 years agotalloc: Add a warning to talloc_reference() documentation.
Andreas Schneider [Mon, 14 Oct 2013 11:17:12 +0000 (13:17 +0200)]
talloc: Add a warning to talloc_reference() documentation.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Kai Blin <kai@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Mon Oct 14 23:05:54 CEST 2013 on sn-devel-104

5 years agoparam: disable print notify backchannel by default
David Disseldorp [Mon, 14 Oct 2013 11:53:22 +0000 (13:53 +0200)]
param: disable print notify backchannel by default

In handling RemoteFindFirstPrinterChangeNotifyEx requests, the spoolss
server can establish a "backchannel" connection to the print client, as
a mechanism for sending print notifications. This behaviour is governed
by the "print notify backchannel" smb.conf parameter.

This change sets "print notify backchannel" to "no" by default, which
sees Samba respond to RemoteFindFirstPrinterChangeNotifyEx requests with
WERR_SERVER_UNAVAILABLE. In recieving such a response, print clients can
fall back to polling for print queue changes.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Mon Oct 14 18:49:41 CEST 2013 on sn-devel-104