[SCM] Samba Shared Repository - branch v3-6-test updated

2012-10-31 Thread Karolin Seeger
The branch, v3-6-test has been updated
   via  155d132 BUG 9326: Fix net ads join message for the dns domain.
  from  11b798c pam_winbind: fix segfault in pam_sm_authenticate()

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-6-test


- Log -
commit 155d132434e96d69bac31dd7413ecb4d4b09b017
Author: Andreas Schneider a...@samba.org
Date:   Tue Oct 23 17:12:59 2012 +0200

BUG 9326: Fix net ads join message for the dns domain.

We don't get a realm back from the server which is useable as a realm on
Unix. On Unix they are case sensitive and on Windows they aren't. This
confuses uses and if we write realm they try to use it as it came back
in lowercase.

Signed-off-by: Andreas Schneider a...@samba.org

---

Summary of changes:
 source3/utils/net_ads.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/utils/net_ads.c b/source3/utils/net_ads.c
index f751d68..6a7bc53 100644
--- a/source3/utils/net_ads.c
+++ b/source3/utils/net_ads.c
@@ -1436,7 +1436,7 @@ int net_ads_join(struct net_context *c, int argc, const 
char **argv)
d_printf(_(Using short domain name -- %s\n), 
r-out.netbios_domain_name);
 
if (r-out.dns_domain_name) {
-   d_printf(_(Joined '%s' to realm '%s'\n), r-in.machine_name,
+   d_printf(_(Joined '%s' to dns domain '%s'\n), 
r-in.machine_name,
r-out.dns_domain_name);
} else {
d_printf(_(Joined '%s' to domain '%s'\n), r-in.machine_name,


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-6-test updated

2012-10-31 Thread Karolin Seeger
The branch, v3-6-test has been updated
   via  e95da42 s4:torture/smb2: improve the smb2.create.blob test
  from  155d132 BUG 9326: Fix net ads join message for the dns domain.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-6-test


- Log -
commit e95da423965cf332f6d0a849725cd0808c42c1e2
Author: Stefan Metzmacher me...@samba.org
Date:   Wed Sep 19 21:18:46 2012 +0200

s4:torture/smb2: improve the smb2.create.blob test

metze
(cherry picked from commit e6c600aa2c751e694917322378417816c3e58eb6)

See https://bugzilla.samba.org/show_bug.cgi?id=9209 for details.

---

Summary of changes:
 source4/torture/smb2/create.c |   93 -
 1 files changed, 91 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/torture/smb2/create.c b/source4/torture/smb2/create.c
index f11c837..3850a3c 100644
--- a/source4/torture/smb2/create.c
+++ b/source4/torture/smb2/create.c
@@ -444,14 +444,103 @@ static bool test_create_blob(struct torture_context 
*tctx, struct smb2_tree *tre
status = smb2_util_close(tree, io.out.file.handle);
CHECK_STATUS(status, NT_STATUS_OK);
 
-   torture_comment(tctx, Testing bad tag length\n);
+   torture_comment(tctx, Testing bad tag length 0\n);
+   ZERO_STRUCT(io.in.blobs);
status = smb2_create_blob_add(tctx, io.in.blobs,
- xxx, data_blob(NULL, 0));
+ x, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_INVALID_PARAMETER);
+
+   torture_comment(tctx, Testing bad tag length 1\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ x, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_INVALID_PARAMETER);
+
+   torture_comment(tctx, Testing bad tag length 2\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ xx, data_blob(NULL, 0));
CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_INVALID_PARAMETER);
 
+   torture_comment(tctx, Testing bad tag length 3\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ xxx, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
status = smb2_create(tree, tctx, io);
CHECK_STATUS(status, NT_STATUS_INVALID_PARAMETER);
 
+   torture_comment(tctx, Testing tag length 4\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ , data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+
+   torture_comment(tctx, Testing tag length 5\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ x, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+
+   torture_comment(tctx, Testing tag length 6\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ xx, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+
+   torture_comment(tctx, Testing tag length 7\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ xxx, data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+
+   torture_comment(tctx, Testing tag length 8\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ , data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+
+   torture_comment(tctx, Testing tag length 16\n);
+   ZERO_STRUCT(io.in.blobs);
+   status = smb2_create_blob_add(tctx, io.in.blobs,
+ , data_blob(NULL, 0));
+   CHECK_STATUS(status, NT_STATUS_OK);
+   status = smb2_create(tree, tctx, io);
+   CHECK_STATUS(status, NT_STATUS_OK);
+

[SCM] Samba Shared Repository - branch v4-0-test updated

2012-10-31 Thread Karolin Seeger
The branch, v4-0-test has been updated
   via  f40bfca packaging: Add NetworkManager dispatcher script for winbind.
   via  1ce0680 BUG 9326: Fix net ads join message for the dns domain.
   via  b5a9e17 lib/replace: Return size of xattr if size argument is 0
   via  a210201 s3: vfs_streams_depot: add delete_lost option (bug #9300)
   via  0c454c5 s3: make recursive_rmdir function non-static
  from  7e100a2 WHATSNEW: Correct list of changed parameters.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v4-0-test


- Log -
commit f40bfcaf4ce8e6a9001d843bbc48505b769d2777
Author: Andreas Schneider a...@cryptomilk.org
Date:   Fri Oct 26 12:46:21 2012 +0200

packaging: Add NetworkManager dispatcher script for winbind.

Signed-off-by: Andreas Schneider a...@cryptomilk.org
Signed-off-by: Andreas Schneider a...@samba.org

https://bugzilla.samba.org/show_bug.cgi?id=9336
Add NetworkManager dispatcher script for winbind (systemd) (edit)

Autobuild-User(v4-0-test): Karolin Seeger ksee...@samba.org
Autobuild-Date(v4-0-test): Wed Oct 31 11:36:42 CET 2012 on sn-devel-104

commit 1ce06808962e62cc2f9c8f2c58eb488f3ff0df59
Author: Andreas Schneider a...@samba.org
Date:   Tue Oct 23 17:12:59 2012 +0200

BUG 9326: Fix net ads join message for the dns domain.

We don't get a realm back from the server which is useable as a realm on
Unix. On Unix they are case sensitive and on Windows they aren't. This
confuses uses and if we write realm they try to use it as it came back
in lowercase.

Signed-off-by: Andreas Schneider a...@samba.org

commit b5a9e1799d4e270a7061834d6a48d9effb29dedb
Author: Andrew Bartlett abart...@samba.org
Date:   Sat Oct 27 19:15:58 2012 +1100

lib/replace: Return size of xattr if size argument is 0

This makes rep_{f,}getxattr a more complete replacement for the linux 
function.

Andrew Bartlett

https://bugzilla.samba.org/show_bug.cgi?id=9338
libreplace does not replace getxattr correctly for 0 length semantics

commit a210201abc243faeb78316559487c25a6e78d8ab
Author: Björn Baumbach b...@sernet.de
Date:   Thu Sep 27 12:40:47 2012 +0200

s3: vfs_streams_depot: add delete_lost option (bug #9300)

With this option lost stream directories will be removed
instead of renamed.

Autobuild-User(master): Volker Lendecke v...@samba.org
Autobuild-Date(master): Mon Oct  1 18:47:30 CEST 2012 on sn-devel-104
(cherry picked from commit 8da8a2289ea51d4fcdf6b5352a46c14d36d8f072)

commit 0c454c5810ff26967da40d01979bb55982a8a50f
Author: Björn Baumbach b...@sernet.de
Date:   Mon Oct 1 09:55:28 2012 +0200

s3: make recursive_rmdir function non-static

Part of fix for bug #9300.

(cherry picked from commit 7a76762c688f4fc7519dbd204b036963c460e093)

---

Summary of changes:
 lib/replace/xattr.c |   15 +-
 packaging/NetworkManager/30-winbind-systemd |   20 
 packaging/NetworkManager/README |6 +++
 source3/modules/vfs_streams_depot.c |   65 +-
 source3/smbd/close.c|8 ++--
 source3/smbd/proto.h|3 +
 source3/utils/net_ads.c |2 +-
 7 files changed, 88 insertions(+), 31 deletions(-)
 create mode 100755 packaging/NetworkManager/30-winbind-systemd
 create mode 100644 packaging/NetworkManager/README


Changeset truncated at 500 lines:

diff --git a/lib/replace/xattr.c b/lib/replace/xattr.c
index 8e1c989..a26ff67 100644
--- a/lib/replace/xattr.c
+++ b/lib/replace/xattr.c
@@ -71,7 +71,9 @@ ssize_t rep_getxattr (const char *path, const char *name, 
void *value, size_t si
 * that the buffer is large enough to fit the returned value.
 */
if((retval=extattr_get_file(path, attrnamespace, attrname, NULL, 0)) = 
0) {
-   if(retval  size) {
+   if (size == 0) {
+   return retval;
+   } else if (retval  size) {
errno = ERANGE;
return -1;
}
@@ -88,6 +90,9 @@ ssize_t rep_getxattr (const char *path, const char *name, 
void *value, size_t si
if (strncmp(name, system, 6) == 0) flags |= ATTR_ROOT;
 
retval = attr_get(path, attrname, (char *)value, valuelength, flags);
+   if (size == 0  retval == -1  errno == E2BIG) {
+   return valuelength;
+   }
 
return retval ? retval : valuelength;
 #elif defined(HAVE_ATTROPEN)
@@ -126,7 +131,9 @@ ssize_t rep_fgetxattr (int filedes, const char *name, void 
*value, size_t size)
const char *attrname = ((s=strchr(name, '.')) == NULL) ? name : s + 1;
 
if((retval=extattr_get_fd(filedes, attrnamespace, attrname, NULL, 0)) 
= 0) {
-   if(retval  

[SCM] Samba Shared Repository - branch v3-5-test updated

2012-10-31 Thread Karolin Seeger
The branch, v3-5-test has been updated
   via  92bd768 Revert Fix bug #7781 (Samba transforms ShareName to 
lowercase when adding new share via MMC)
   via  79564b8 Revert Revert s3-smbd: Don't segfault if user specified 
ports out for range.
  from  fce3a18 Revert s3-smbd: Don't segfault if user specified ports out 
for range.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-5-test


- Log -
commit 92bd768ed56585c2a45d0ca41eec9e6a1e3701ae
Author: Karolin Seeger ksee...@samba.org
Date:   Wed Oct 31 11:40:26 2012 +0100

Revert Fix bug #7781 (Samba transforms ShareName to lowercase when 
adding new share via MMC)

This reverts commit 157b88da4db727eafa682c7fc7eab11d5955f57b.

This one seems to break make test on my system.

Karolin

commit 79564b889dba69fa39d9839e5d45457657ab0950
Author: Karolin Seeger ksee...@samba.org
Date:   Wed Oct 31 11:39:34 2012 +0100

Revert Revert s3-smbd: Don't segfault if user specified ports out for 
range.

This reverts commit fce3a18d3d5ed46f8e0d1653f862e46b5dff0e03.

This patch does not seem to cause the issue. Sorry for the noise.

Karolin

---

Summary of changes:
 source3/include/proto.h |2 +-
 source3/lib/dummysmbd.c |2 +-
 source3/lib/util_str.c  |4 --
 source3/modules/vfs_xattr_tdb.c |3 +-
 source3/param/loadparm.c|9 +++-
 source3/printing/nt_printing.c  |   12 -
 source3/registry/reg_backend_printing.c |7 +--
 source3/rpc_server/srv_srvsvc_nt.c  |   75 ++-
 source3/smbd/lanman.c   |   14 ++
 source3/smbd/msdfs.c|3 +-
 source3/smbd/server.c   |   23 +-
 source3/smbd/service.c  |   16 ++-
 source3/smbd/smb2_tcon.c|2 +-
 13 files changed, 82 insertions(+), 90 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/include/proto.h b/source3/include/proto.h
index cee5d6a..785cc30 100644
--- a/source3/include/proto.h
+++ b/source3/include/proto.h
@@ -7002,7 +7002,7 @@ bool set_conn_connectpath(connection_struct *conn, const 
char *connectpath);
 bool set_current_service(connection_struct *conn, uint16 flags, bool do_chdir);
 void load_registry_shares(void);
 int add_home_service(const char *service, const char *username, const char 
*homedir);
-int find_service(const char *service_in, fstring service);
+int find_service(fstring service);
 connection_struct *make_connection_snum(struct smbd_server_connection *sconn,
int snum, user_struct *vuser,
DATA_BLOB password,
diff --git a/source3/lib/dummysmbd.c b/source3/lib/dummysmbd.c
index 28c6f0e..a41e6dc 100644
--- a/source3/lib/dummysmbd.c
+++ b/source3/lib/dummysmbd.c
@@ -28,7 +28,7 @@ int get_client_fd(void)
return -1;
 }
 
-int find_service(const char *service_in, fstring service)
+int find_service(fstring service)
 {
return -1;
 }
diff --git a/source3/lib/util_str.c b/source3/lib/util_str.c
index 17a4a8f..d869637 100644
--- a/source3/lib/util_str.c
+++ b/source3/lib/util_str.c
@@ -2301,10 +2301,6 @@ bool validate_net_name( const char *name,
 {
int i;
 
-   if (!name) {
-   return false;
-   }
-
for ( i=0; imax_len  name[i]; i++ ) {
/* fail if strchr_m() finds one of the invalid characters */
if ( name[i]  strchr_m( invalid_chars, name[i] ) ) {
diff --git a/source3/modules/vfs_xattr_tdb.c b/source3/modules/vfs_xattr_tdb.c
index fa8db04..f7fbfce 100644
--- a/source3/modules/vfs_xattr_tdb.c
+++ b/source3/modules/vfs_xattr_tdb.c
@@ -733,7 +733,8 @@ static int xattr_tdb_connect(vfs_handle_struct *handle, 
const char *service,
return res;
}
 
-   snum = find_service(service, sname);
+   fstrcpy(sname, service);
+   snum = find_service(sname);
if (snum == -1) {
/*
 * Should not happen, but we should not fail just *here*.
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index eaff9e6..8c1cf09 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -9471,10 +9471,15 @@ struct share_params *get_share_params(TALLOC_CTX 
*mem_ctx,
  const char *sharename)
 {
struct share_params *result;
-   fstring sname;
+   char *sname;
int snum;
 
-   snum = find_service(sharename, sname);
+   if (!(sname = SMB_STRDUP(sharename))) {
+   return NULL;
+   }
+
+   snum = find_service(sname);
+   SAFE_FREE(sname);
 
if (snum  0) {
return NULL;
diff --git a/source3/printing/nt_printing.c 

autobuild: intermittent test failure detected

2012-10-31 Thread autobuild
The autobuild test system has detected an intermittent failing test in 
the current master tree.

The autobuild log of the failure is available here:

   http://git.samba.org/autobuild.flakey/2012-10-31-1606/flakey.log

The samba3 build logs are available here:

   http://git.samba.org/autobuild.flakey/2012-10-31-1606/samba3.stderr
   http://git.samba.org/autobuild.flakey/2012-10-31-1606/samba3.stdout

The source4 build logs are available here:

   http://git.samba.org/autobuild.flakey/2012-10-31-1606/samba.stderr
   http://git.samba.org/autobuild.flakey/2012-10-31-1606/samba.stdout
  
The top commit at the time of the failure was:

commit 59e9661de2dd19808295002f1329b27d5dca09e6
Author: Jeremy Allison j...@samba.org
Date:   Mon Oct 29 14:49:36 2012 -0700

Add regression test for bug #9329 - Directory listing with SeBackup can 
crash smbd.

Ensure we exercise the SeBackup code path on directory listings.

Signed-off-by: Jeremy Allison j...@samba.org
Reviewed-by: Michael Adam ob...@samba.org

Autobuild-User(master): Michael Adam ob...@samba.org
Autobuild-Date(master): Wed Oct 31 03:21:38 CET 2012 on sn-devel-104


[SCM] Samba Shared Repository - branch master updated

2012-10-31 Thread Björn Jacke
The branch, master has been updated
   via  ffb608b util: remove accidently committed hunk
  from  59e9661 Add regression test for bug #9329 - Directory listing with 
SeBackup can crash smbd.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit ffb608b54d452e2c96222840f1bd48759f32d0b5
Author: Björn Jacke b...@sernet.de
Date:   Wed Oct 31 15:31:08 2012 +0100

util: remove accidently committed hunk

Autobuild-User(master): Björn Jacke b...@sernet.de
Autobuild-Date(master): Wed Oct 31 19:25:30 CET 2012 on sn-devel-104

---

Summary of changes:
 lib/util/util_net.c |3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)


Changeset truncated at 500 lines:

diff --git a/lib/util/util_net.c b/lib/util/util_net.c
index 536c0a4..83afda4 100644
--- a/lib/util/util_net.c
+++ b/lib/util/util_net.c
@@ -817,9 +817,6 @@ static const smb_socket_option socket_options[] = {
 #ifdef TCP_QUICKACK
   {TCP_QUICKACK, IPPROTO_TCP, TCP_QUICKACK, 0, OPT_BOOL},
 #endif
-#ifdef TCP_FASTOPEN
-  {TCP_FASTOPEN, IPPROTO_TCP, TCP_FASTOPEN, 0, OPT_BOOL},
-#endif
 #ifdef TCP_NODELAYACK
   {TCP_NODELAYACK, IPPROTO_TCP, TCP_NODELAYACK, 0, OPT_BOOL},
 #endif


-- 
Samba Shared Repository