autobuild: intermittent test failure detected

2014-06-13 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/2014-06-14-0214/flakey.log

The samba build logs are available here:

   http://git.samba.org/autobuild.flakey/2014-06-14-0214/samba.stderr
   http://git.samba.org/autobuild.flakey/2014-06-14-0214/samba.stdout
  
The top commit at the time of the failure was:

commit c70932827812cf3b3fc8323e11abf4c98f15ddec
Author: Michael Adam 
Date:   Fri Jun 13 12:05:05 2014 +0200

vfs:gpfs: fix a debug message

Signed-off-by: Michael Adam 
Reviewed-by: Volker Lendecke 

Autobuild-User(master): Volker Lendecke 
Autobuild-Date(master): Fri Jun 13 19:54:23 CEST 2014 on sn-devel-104


[SCM] Samba Shared Repository - branch master updated

2014-06-13 Thread Volker Lendecke
The branch, master has been updated
   via  c709328 vfs:gpfs: fix a debug message
   via  9f6c4c4 vfs: update a bit-rotten comment
   via  ddb7d7f vfs: remove unused function vfs_pread_data()
  from  5c87425 man vfs_streams_xattr: new options "prefix" and 
"store_stream_type"

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


- Log -
commit c70932827812cf3b3fc8323e11abf4c98f15ddec
Author: Michael Adam 
Date:   Fri Jun 13 12:05:05 2014 +0200

vfs:gpfs: fix a debug message

Signed-off-by: Michael Adam 
Reviewed-by: Volker Lendecke 

Autobuild-User(master): Volker Lendecke 
Autobuild-Date(master): Fri Jun 13 19:54:23 CEST 2014 on sn-devel-104

commit 9f6c4c4da20df633b28c1f3047e62c0daa6f6931
Author: Michael Adam 
Date:   Fri Jun 13 11:46:25 2014 +0200

vfs: update a bit-rotten comment

Signed-off-by: Michael Adam 
Reviewed-by: Volker Lendecke 

commit ddb7d7fc940835e04a5d909790f99cee5c74084d
Author: Michael Adam 
Date:   Fri Jun 13 11:39:41 2014 +0200

vfs: remove unused function vfs_pread_data()

Signed-off-by: Michael Adam 
Reviewed-by: Volker Lendecke 

---

Summary of changes:
 source3/include/vfs_macros.h |   11 ++-
 source3/modules/gpfs.c   |2 +-
 source3/smbd/proto.h |2 --
 source3/smbd/vfs.c   |   22 --
 4 files changed, 7 insertions(+), 30 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/include/vfs_macros.h b/source3/include/vfs_macros.h
index 704054d..0f44c62 100644
--- a/source3/include/vfs_macros.h
+++ b/source3/include/vfs_macros.h
@@ -21,11 +21,12 @@
 #ifndef _VFS_MACROS_H
 #define _VFS_MACROS_H
 
-/***
- Don't access conn->vfs.ops.* directly!!!
- Use this macros!
- (Fixes should go also into the vfs_opaque_* and vfs_next_* macros!)
-/
+/*
+ * These macros SMB_VFS_ (and SMB_VFS_NEXT_) are our
+ * interface for the VFS.
+ *
+ * Don't access conn->vfs_handles[->next]->fns->* directly!
+ */
 
 /* Disk operations */
 #define SMB_VFS_CONNECT(conn, service, user) \
diff --git a/source3/modules/gpfs.c b/source3/modules/gpfs.c
index 6250286..53418f0 100644
--- a/source3/modules/gpfs.c
+++ b/source3/modules/gpfs.c
@@ -177,7 +177,7 @@ int smbd_fget_gpfs_winattrs(int fd, struct gpfs_winattr 
*attrs)
 errno = ENOSYS;
 return -1;
 }
-DEBUG(10, ("gpfs_get_winattrs_path:open call %d\n", fd));
+DEBUG(10, ("gpfs_get_winattrs:open call %d\n", fd));
 return gpfs_get_winattrs_fn(fd, attrs);
 }
 
diff --git a/source3/smbd/proto.h b/source3/smbd/proto.h
index 3c6accf..1e17f5b 100644
--- a/source3/smbd/proto.h
+++ b/source3/smbd/proto.h
@@ -1129,8 +1129,6 @@ bool vfs_init_custom(connection_struct *conn, const char 
*vfs_object);
 bool smbd_vfs_init(connection_struct *conn);
 NTSTATUS vfs_file_exist(connection_struct *conn, struct smb_filename 
*smb_fname);
 ssize_t vfs_read_data(files_struct *fsp, char *buf, size_t byte_count);
-ssize_t vfs_pread_data(files_struct *fsp, char *buf,
-size_t byte_count, off_t offset);
 ssize_t vfs_write_data(struct smb_request *req,
files_struct *fsp,
const char *buffer,
diff --git a/source3/smbd/vfs.c b/source3/smbd/vfs.c
index 897bf1f..d5390ed 100644
--- a/source3/smbd/vfs.c
+++ b/source3/smbd/vfs.c
@@ -393,28 +393,6 @@ ssize_t vfs_read_data(files_struct *fsp, char *buf, size_t 
byte_count)
return (ssize_t)total;
 }
 
-ssize_t vfs_pread_data(files_struct *fsp, char *buf,
-size_t byte_count, off_t offset)
-{
-   size_t total=0;
-
-   while (total < byte_count)
-   {
-   ssize_t ret = SMB_VFS_PREAD(fsp, buf + total,
-   byte_count - total, offset + total);
-
-   if (ret == 0) return total;
-   if (ret == -1) {
-   if (errno == EINTR)
-   continue;
-   else
-   return -1;
-   }
-   total += ret;
-   }
-   return (ssize_t)total;
-}
-
 /
  Write data to a fd on the vfs.
 /


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2014-06-13 Thread Michael Adam
The branch, master has been updated
   via  5c87425 man vfs_streams_xattr: new options "prefix" and 
"store_stream_type"
   via  634bcb0 vfs_streams_xattr: add options "prefix" and 
"store_stream_type"
   via  f03e8b1 Convert samba_private_attr_name() to a public function
   via  3defbe2 locking: use correct conversion specifier for printing 
variables
  from  0057388 leases: Fix uint32->hyper

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


- Log -
commit 5c874251ff78708706c7b421bb24c00ccd995a65
Author: Ralph Boehme 
Date:   Mon May 26 16:12:17 2014 +0200

man vfs_streams_xattr: new options "prefix" and "store_stream_type"

Add documentation for the two new options "streams_xattr:prefix" and
"streams_xattr:store_stream_type".

Signed-off-by: Ralph Boehme 
Reviewed-by: Christof Schmitt 
Reviewed-by: Michael Adam 

Autobuild-User(master): Michael Adam 
Autobuild-Date(master): Fri Jun 13 15:51:43 CEST 2014 on sn-devel-104

commit 634bcb09a08b927fd79ae0e16aeee2a123605f94
Author: Ralph Boehme 
Date:   Tue May 20 15:17:01 2014 +0200

vfs_streams_xattr: add options "prefix" and "store_stream_type"

Add module options that can be used to configure the stream prefix the
module uses (option "prefix", a string) and whether the stream type
"$DATA" is appended to the xattr name on disk (option
"store_stream_type", a boolean).

The default "prefix" is "user.DosStream" and the default for
"store_stream_type" is true, this gives unchanged default behaviour
when not specifying this option.

OS X SMB clients will send xattrs as named streams over the wire, by
setting the options to the following values

  streams_xattr:prefix = user.
  streams_xattr:store_stream_type = no

OS X xattrs will be stored on disk on the server with their unmodified
names and as such provide interoperability with other protocols like
AFP.

In order to prevent access to our internal Samba xattrs, check the
xattr name with the function samba_private_attr_name() made public by
the previous commit.

Signed-off-by: Ralph Boehme 
Reviewed-by: Christof Schmitt 
Reviewed-by: Michael Adam 

commit f03e8b12c6959fb8213fbfcd3792b70094bd9a71
Author: Ralph Boehme 
Date:   Wed May 21 11:52:27 2014 +0200

Convert samba_private_attr_name() to a public function

Signed-off-by: Ralph Boehme 
Reviewed-by: Christof Schmitt 
Reviewed-by: Michael Adam 

commit 3defbe273459262f333cca91e90a0941a812df1a
Author: Ralph Boehme 
Date:   Sun Jun 1 15:16:16 2014 +0200

locking: use correct conversion specifier for printing variables

Fix several occurences of using printf conversion to fload when
printing offset and count variables in locking debug messages and
smbstatus.

Conversion to float may lead to wrong results with very large values.

Signed-off-by: Ralph Boehme 
Reviewed-by: Volker Lendecke 
Reviewed-by: Michael Adam 

---

Summary of changes:
 docs-xml/manpages/vfs_streams_xattr.8.xml |   34 -
 source3/locking/locking.c |4 +-
 source3/locking/posix.c   |   13 ++--
 source3/modules/vfs_streams_xattr.c   |  122 ++---
 source3/smbd/proto.h  |1 +
 source3/smbd/trans2.c |2 +-
 source3/utils/status.c|4 +-
 7 files changed, 137 insertions(+), 43 deletions(-)


Changeset truncated at 500 lines:

diff --git a/docs-xml/manpages/vfs_streams_xattr.8.xml 
b/docs-xml/manpages/vfs_streams_xattr.8.xml
index 5aa5aa8..2deb802 100644
--- a/docs-xml/manpages/vfs_streams_xattr.8.xml
+++ b/docs-xml/manpages/vfs_streams_xattr.8.xml
@@ -33,8 +33,11 @@
alternate data streams in the file system. As a normal posix file
system does not support the concept of multiple data streams per file,
the streams_xattr module stores the data in posix extended attributes
-   (xattrs). The name of these attributes is
-   user.DosStream."ADS-NAME".
+   (xattrs). The name of these attributes by default is
+   user.DosStream."ADS-NAME". The prefix "user.DosStream." can be changed
+   with the module option streams_xattr:prefix, but be
+   aware that this will also expose those ADS over the SMB extended
+   attributes interface.
 
The file system that is shared with this module enabled must
support xattrs.
@@ -46,6 +49,33 @@
 
 
 
+   OPTIONS
+
+   
+
+ 
+   streams_xattr:prefix = STRING
+   
+ Name prefix used when storing an ADS in an xattr, defaults 
to
+ user.DosStream.. Changing this will also
+ expose ADS over the SMB extended attributes interface.
+   

[SCM] CTDB repository - branch 2.5 updated - ctdb-2.5.3-50-gc6f91dd

2014-06-13 Thread Amitay Isaacs
The branch, 2.5 has been updated
   via  c6f91dd3df0f8604a2eadef3c44025e07a1c1a03 (commit)
   via  515736c53eb87fd76a16a6fef59ab8255b07da24 (commit)
   via  ef604d51ee17d88727406425074f631c996bca0e (commit)
   via  904bb9b95625541c77ccda1f8e03057b9446b879 (commit)
   via  a3d3b994df3e51316bc355fba6402d62e0cbf519 (commit)
   via  262ec107e269a349baae3ca6b72b4a1a0a18f72d (commit)
   via  6a8c8a0029b1d1b7a8f80343666f30385c24223c (commit)
   via  6aabb15346e5be2470d95e96b6a36aa2317325b0 (commit)
   via  23561cca2b3496d5a47977d5ce2600e8951dec39 (commit)
   via  ba2f7b3ea67468f38f2f80266c20a3c29a1c1f79 (commit)
   via  2b41fd554889876b848c30790c0272e58dbd025c (commit)
   via  1a4433d0d620d8104bc75ba5d10e4394a0d37e5a (commit)
   via  d94e082ebcdfb6cb6e050fa2a5ace6ff13869cf4 (commit)
   via  57eb4e57beef2b5cde351e9634fb4afd70354ac0 (commit)
   via  b4bb7bfd2d4eaee57707a2716006337f2e594bee (commit)
   via  385d1ee07ad1f8476a2810be1f8551e9d28fc336 (commit)
   via  7d30073689fd6be1f284bd6e6423abc4a8fd882d (commit)
   via  25a6bfc9ddfd42b9fc3ee0069b793311b649eaf1 (commit)
   via  5c94a3b492b600265ddd3b85394f19c66a9b566a (commit)
   via  5821e5ebc4e2845947e60ca4c1751cdade389576 (commit)
   via  9d4f41da1282369ffca3ef9e4259d0b0d48eb1ad (commit)
   via  2b2b7941ccbd1a1ae0e8743ed63ea8ecebf031e2 (commit)
   via  e015f6643efc274c282d8d31e77aaa7c9470b3e3 (commit)
   via  8c95317efb9d673aef4c059b3fcbb75197fb046f (commit)
  from  31a4dde35d9037fe1bc48128186619b820358e8b (commit)

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


- Log -
commit c6f91dd3df0f8604a2eadef3c44025e07a1c1a03
Author: Amitay Isaacs 
Date:   Fri Jun 6 16:49:13 2014 +1000

daemon: Reset scheduler policy for helper processes

Do not run helper processes with real-time priority.

This regression was caused when locking and eventscript code switched
to use vfork() and helper instead of ctdb_fork().

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

Autobuild-User(master): Martin Schwenke 
Autobuild-Date(master): Thu Jun 12 08:10:36 CEST 2014 on sn-devel-104

(Imported from commit 9e6e48750d1fad8ec7cd076f7ca7d2b028007258)

commit 515736c53eb87fd76a16a6fef59ab8255b07da24
Author: Amitay Isaacs 
Date:   Tue Jun 10 12:17:32 2014 +1000

daemon: Do not complain if node is inactive and db is not attached

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit e9eed41d7ffded2dfd1406f60ad08fe070fd52c1)

commit ef604d51ee17d88727406425074f631c996bca0e
Author: Amitay Isaacs 
Date:   Tue Jun 10 11:56:39 2014 +1000

build: Remove duplicate library dependency

LIB_FLAGS includes TDB_LIBS.

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 79a6bd269291a4b4dabc57254b79bbfa82ac70e4)

commit 904bb9b95625541c77ccda1f8e03057b9446b879
Author: Amitay Isaacs 
Date:   Fri Jun 6 16:46:15 2014 +1000

daemon: Remove duplicate code from helper binaries

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 71ed758c72db36234209428fa54be82fe8f0899a)

commit a3d3b994df3e51316bc355fba6402d62e0cbf519
Author: Amitay Isaacs 
Date:   Fri Jun 6 15:43:22 2014 +1000

logging: Move controls handling functions from common to server

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit e7c72588d187f9505444f48a1c5b19e47b15ccd9)

commit 262ec107e269a349baae3ca6b72b4a1a0a18f72d
Author: Amitay Isaacs 
Date:   Tue Jun 10 14:52:19 2014 +1000

logging: Split ringbuffer handling code from ctdb_collect_log

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 27d1137e26703d7fa5fba638e56961384a140eb2)

commit 6a8c8a0029b1d1b7a8f80343666f30385c24223c
Author: Amitay Isaacs 
Date:   Fri Jun 6 15:25:08 2014 +1000

common: Move mkdir_p_or_die to system utilities

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 49e34651a95a336c9e410e4b52a8cfd491ca0a67)

commit 6aabb15346e5be2470d95e96b6a36aa2317325b0
Author: Amitay Isaacs 
Date:   Fri Jun 6 15:24:20 2014 +1000

daemon: Rename ctdb_mkdir_p_or_die to mkdir_p_or_die

This function does not require ctdb context.

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 4736486188afb8ff6243bcc8a27daa9dbcc64b7a)

commit 23561cca2b3496d5a47977d5ce2600e8951dec39
Author: Amitay Isaacs 
Date:   Tue Jun 10 17:04:34 2014 +1000

common: Move lockdown_memory to system utilities

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

(Imported from commit 507207e40b193ba753604666c56dfa29c29b7145)

commit ba2f7b3ea67468f38f2f80266c20