[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Volker Lendecke
The branch, master has been updated
   via  ad6ee94... s3: Make debug hires timestamp default to true
  from  a355365... s4:dsdb Fix up after the MAP_ constants became LDB_MAP_

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


- Log -
commit ad6ee94950c0977463194b64faee9b012049bfd0
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 10:05:38 2009 +0100

s3: Make debug hires timestamp default to true

It does not cost much and can help a lot when debugging

---

Summary of changes:
 .../smbdotconf/logging/debughirestimestamp.xml |2 +-
 source3/param/loadparm.c   |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/docs-xml/smbdotconf/logging/debughirestimestamp.xml 
b/docs-xml/smbdotconf/logging/debughirestimestamp.xml
index 6f32bd5..6cc7b71 100644
--- a/docs-xml/smbdotconf/logging/debughirestimestamp.xml
+++ b/docs-xml/smbdotconf/logging/debughirestimestamp.xml
@@ -14,5 +14,5 @@
 /para
 
 /description
-value type=defaultno/value
+value type=defaultyes/value
 /samba:parameter
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 049b5e9..93be017 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -4984,7 +4984,7 @@ static void init_globals(bool first_time_only)
Globals.bTimestampLogs = True;
string_set(Globals.szLogLevel, 0);
Globals.bDebugPrefixTimestamp = False;
-   Globals.bDebugHiresTimestamp = False;
+   Globals.bDebugHiresTimestamp = true;
Globals.bDebugPid = False;
Globals.bDebugUid = False;
Globals.bDebugClass = False;


-- 
Samba Shared Repository


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

2009-11-02 Thread Volker Lendecke
The branch, v3-5-test has been updated
   via  cb1b5f9... s3: Make debug hires timestamp default to true
  from  54af185... s3:ldap: don't search when no values where found

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


- Log -
commit cb1b5f93332839eb8fc048895c9c9539f775d9a8
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 10:05:38 2009 +0100

s3: Make debug hires timestamp default to true

It does not cost much and can help a lot when debugging

---

Summary of changes:
 .../smbdotconf/logging/debughirestimestamp.xml |2 +-
 source3/param/loadparm.c   |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/docs-xml/smbdotconf/logging/debughirestimestamp.xml 
b/docs-xml/smbdotconf/logging/debughirestimestamp.xml
index 6f32bd5..6cc7b71 100644
--- a/docs-xml/smbdotconf/logging/debughirestimestamp.xml
+++ b/docs-xml/smbdotconf/logging/debughirestimestamp.xml
@@ -14,5 +14,5 @@
 /para
 
 /description
-value type=defaultno/value
+value type=defaultyes/value
 /samba:parameter
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 049b5e9..93be017 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -4984,7 +4984,7 @@ static void init_globals(bool first_time_only)
Globals.bTimestampLogs = True;
string_set(Globals.szLogLevel, 0);
Globals.bDebugPrefixTimestamp = False;
-   Globals.bDebugHiresTimestamp = False;
+   Globals.bDebugHiresTimestamp = true;
Globals.bDebugPid = False;
Globals.bDebugUid = False;
Globals.bDebugClass = False;


-- 
Samba Shared Repository


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

2009-11-02 Thread Karolin Seeger
The branch, v3-5-test has been updated
   via  efbca60... WHATSNEW: Update release notes.
  from  cb1b5f9... s3: Make debug hires timestamp default to true

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


- Log -
commit efbca60e6d3258da997d11fdb126251c89678a01
Author: Karolin Seeger ksee...@samba.org
Date:   Mon Nov 2 12:45:53 2009 +0100

WHATSNEW: Update release notes.

Karolin

---

Summary of changes:
 WHATSNEW.txt |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index 6f4b159..683b8ef 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -22,6 +22,7 @@ smb.conf changes
--  --- ---
 
create krb5 conf   New yes
+   debug hires timestamp  Changed Default yes
ldap follow referral   New auto
store create time  New no
 


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Günther Deschner
The branch, master has been updated
   via  f35a1b9... s3-gencache: restore gencache_get behavior with NULL 
args (with torture test).
  from  ad6ee94... s3: Make debug hires timestamp default to true

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


- Log -
commit f35a1b95aa6f50cb1bc919f4ab502ef2c38f6bf5
Author: Günther Deschner g...@samba.org
Date:   Mon Nov 2 13:01:58 2009 +0100

s3-gencache: restore gencache_get behavior with NULL args (with torture 
test).

Without this, we panic in wins_srv_is_dead() and fail to start nmbd with
wins support.

Volker, please check.

Guenther

---

Summary of changes:
 source3/lib/gencache.c|   12 
 source3/torture/torture.c |5 +
 2 files changed, 13 insertions(+), 4 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/gencache.c b/source3/lib/gencache.c
index 6de653f..4889d7c 100644
--- a/source3/lib/gencache.c
+++ b/source3/lib/gencache.c
@@ -529,11 +529,15 @@ bool gencache_get(const char *keystr, char **value, 
time_t *ptimeout)
SAFE_FREE(blob.data);
return false;
}
-   *value = SMB_STRDUP((char *)blob.data);
-   data_blob_free(blob);
-   if (*value == NULL) {
-   return false;
+   if (value) {
+   *value = SMB_STRDUP((char *)blob.data);
+   data_blob_free(blob);
+   if (*value == NULL) {
+   return false;
+   }
+   return true;
}
+   data_blob_free(blob);
return true;
 }
 
diff --git a/source3/torture/torture.c b/source3/torture/torture.c
index ca499a9..bda82f2 100644
--- a/source3/torture/torture.c
+++ b/source3/torture/torture.c
@@ -6431,6 +6431,11 @@ static bool run_local_gencache(int dummy)
return False;
}
 
+   if (!gencache_get(foo, NULL, NULL)) {
+   d_printf(%s: gencache_get() failed\n, __location__);
+   return False;
+   }
+
if (!gencache_get(foo, val, tm)) {
d_printf(%s: gencache_get() failed\n, __location__);
return False;


-- 
Samba Shared Repository


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

2009-11-02 Thread Günther Deschner
The branch, v3-5-test has been updated
   via  f39c257... s3-gencache: restore gencache_get behavior with NULL 
args (with torture test).
  from  efbca60... WHATSNEW: Update release notes.

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


- Log -
commit f39c2577fc649713cfd1e0fb9c164ee4423c96f7
Author: Günther Deschner g...@samba.org
Date:   Mon Nov 2 13:01:58 2009 +0100

s3-gencache: restore gencache_get behavior with NULL args (with torture 
test).

Without this, we panic in wins_srv_is_dead() and fail to start nmbd with
wins support.

Volker, please check.

Guenther
(cherry picked from commit f35a1b95aa6f50cb1bc919f4ab502ef2c38f6bf5)

---

Summary of changes:
 source3/lib/gencache.c|   12 
 source3/torture/torture.c |5 +
 2 files changed, 13 insertions(+), 4 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/gencache.c b/source3/lib/gencache.c
index 6de653f..4889d7c 100644
--- a/source3/lib/gencache.c
+++ b/source3/lib/gencache.c
@@ -529,11 +529,15 @@ bool gencache_get(const char *keystr, char **value, 
time_t *ptimeout)
SAFE_FREE(blob.data);
return false;
}
-   *value = SMB_STRDUP((char *)blob.data);
-   data_blob_free(blob);
-   if (*value == NULL) {
-   return false;
+   if (value) {
+   *value = SMB_STRDUP((char *)blob.data);
+   data_blob_free(blob);
+   if (*value == NULL) {
+   return false;
+   }
+   return true;
}
+   data_blob_free(blob);
return true;
 }
 
diff --git a/source3/torture/torture.c b/source3/torture/torture.c
index ca499a9..bda82f2 100644
--- a/source3/torture/torture.c
+++ b/source3/torture/torture.c
@@ -6431,6 +6431,11 @@ static bool run_local_gencache(int dummy)
return False;
}
 
+   if (!gencache_get(foo, NULL, NULL)) {
+   d_printf(%s: gencache_get() failed\n, __location__);
+   return False;
+   }
+
if (!gencache_get(foo, val, tm)) {
d_printf(%s: gencache_get() failed\n, __location__);
return False;


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Volker Lendecke
The branch, master has been updated
   via  d9bdd17... s3: Fix a 100% CPU loop when ctdbd dies during a traverse
  from  f35a1b9... s3-gencache: restore gencache_get behavior with NULL 
args (with torture test).

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


- Log -
commit d9bdd17522c6d2916aedb29721c79a9c421ec1eb
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 16:59:15 2009 +0100

s3: Fix a 100% CPU loop when ctdbd dies during a traverse

---

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


Changeset truncated at 500 lines:

diff --git a/source3/lib/ctdbd_conn.c b/source3/lib/ctdbd_conn.c
index 0af4204..ffd79c9 100644
--- a/source3/lib/ctdbd_conn.c
+++ b/source3/lib/ctdbd_conn.c
@@ -1167,6 +1167,7 @@ NTSTATUS ctdbd_traverse(uint32 db_id,
 
if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
status = NT_STATUS_OK;
+   break;
}
 
if (!NT_STATUS_IS_OK(status)) {


-- 
Samba Shared Repository


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

2009-11-02 Thread Volker Lendecke
The branch, v3-5-test has been updated
   via  892f929... s3: Fix a 100% CPU loop when ctdbd dies during a traverse
  from  f39c257... s3-gencache: restore gencache_get behavior with NULL 
args (with torture test).

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


- Log -
commit 892f92972112e55fdea250c198623d2fd4f70d7c
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 16:59:15 2009 +0100

s3: Fix a 100% CPU loop when ctdbd dies during a traverse

---

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


Changeset truncated at 500 lines:

diff --git a/source3/lib/ctdbd_conn.c b/source3/lib/ctdbd_conn.c
index 0af4204..ffd79c9 100644
--- a/source3/lib/ctdbd_conn.c
+++ b/source3/lib/ctdbd_conn.c
@@ -1167,6 +1167,7 @@ NTSTATUS ctdbd_traverse(uint32 db_id,
 
if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
status = NT_STATUS_OK;
+   break;
}
 
if (!NT_STATUS_IS_OK(status)) {


-- 
Samba Shared Repository


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

2009-11-02 Thread Jeremy Allison
The branch, v3-5-test has been updated
   via  6a11a1d... ndr64: added support for trailing gap alignment
  from  892f929... s3: Fix a 100% CPU loop when ctdbd dies during a traverse

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


- Log -
commit 6a11a1d720b12889b5cbe6ed1f7751ce0d22d4b9
Author: Andrew Tridgell tri...@samba.org
Date:   Thu Oct 1 16:08:02 2009 +1000

ndr64: added support for trailing gap alignment

NDR64 has a 'trailing gap' alignment, which aligns the end of a
structure on the overall structure alignment.

This explains the discrepancy we had with the RPC-SAMR test and NDR64

---

Summary of changes:
 librpc/ndr/libndr.h  |2 ++
 librpc/ndr/ndr_basic.c   |   18 ++
 pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm |4 
 3 files changed, 24 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/librpc/ndr/libndr.h b/librpc/ndr/libndr.h
index f6f5170..3236932 100644
--- a/librpc/ndr/libndr.h
+++ b/librpc/ndr/libndr.h
@@ -500,6 +500,8 @@ enum ndr_err_code ndr_push_align(struct ndr_push *ndr, 
size_t size);
 enum ndr_err_code ndr_pull_align(struct ndr_pull *ndr, size_t size);
 enum ndr_err_code ndr_push_union_align(struct ndr_push *ndr, size_t size);
 enum ndr_err_code ndr_pull_union_align(struct ndr_pull *ndr, size_t size);
+enum ndr_err_code ndr_push_trailer_align(struct ndr_push *ndr, size_t size);
+enum ndr_err_code ndr_pull_trailer_align(struct ndr_pull *ndr, size_t size);
 enum ndr_err_code ndr_push_bytes(struct ndr_push *ndr, const uint8_t *data, 
uint32_t n);
 enum ndr_err_code ndr_push_zero(struct ndr_push *ndr, uint32_t n);
 enum ndr_err_code ndr_push_array_uint8(struct ndr_push *ndr, int ndr_flags, 
const uint8_t *data, uint32_t n);
diff --git a/librpc/ndr/ndr_basic.c b/librpc/ndr/ndr_basic.c
index f3b7e75..807db59 100644
--- a/librpc/ndr/ndr_basic.c
+++ b/librpc/ndr/ndr_basic.c
@@ -573,6 +573,24 @@ _PUBLIC_ enum ndr_err_code ndr_pull_union_align(struct 
ndr_pull *ndr, size_t siz
return NDR_ERR_SUCCESS;
 }
 
+_PUBLIC_ enum ndr_err_code ndr_push_trailer_align(struct ndr_push *ndr, size_t 
size)
+{
+   /* MS-RPCE section 2.2.5.3.4.1 */
+   if (ndr-flags  LIBNDR_FLAG_NDR64) {
+   return ndr_push_align(ndr, size);
+   }
+   return NDR_ERR_SUCCESS;
+}
+
+_PUBLIC_ enum ndr_err_code ndr_pull_trailer_align(struct ndr_pull *ndr, size_t 
size)
+{
+   /* MS-RPCE section 2.2.5.3.4.1 */
+   if (ndr-flags  LIBNDR_FLAG_NDR64) {
+   return ndr_pull_align(ndr, size);
+   }
+   return NDR_ERR_SUCCESS;
+}
+
 /*
   push some bytes
 */
diff --git a/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm 
b/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm
index 9a02d52..11a43b4 100644
--- a/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm
+++ b/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm
@@ -1235,6 +1235,8 @@ sub ParseStructPushPrimitives($)
}
 
$self-ParseElementPush($_, $ndr, $env, 1, 0) foreach 
(@{$struct-{ELEMENTS}});
+
+   $self-pidl(NDR_CHECK(ndr_push_trailer_align($ndr, 
$struct-{ALIGN})););
 }
 
 sub ParseStructPushDeferred()
@@ -1533,6 +1535,8 @@ sub ParseStructPullPrimitives($)
$self-ParseElementPull($_, $ndr, $env, 1, 0) foreach 
(@{$struct-{ELEMENTS}});
 
$self-add_deferred();
+
+   $self-pidl(NDR_CHECK(ndr_pull_trailer_align($ndr, 
$struct-{ALIGN})););
 }
 
 sub ParseStructPullDeferred($)


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Jeremy Allison
The branch, master has been updated
   via  494d827... Fix bug 6867 - trans2findnext returns reply_nterror(req, 
ntstatus) In a directory with a lot of files. Jeremy.
  from  d9bdd17... s3: Fix a 100% CPU loop when ctdbd dies during a traverse

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


- Log -
commit 494d8271d43c4bc44a57463e34ae8c35423e4443
Author: Jeremy Allison j...@samba.org
Date:   Mon Nov 2 12:15:25 2009 -0800

Fix bug 6867 - trans2findnext returns reply_nterror(req, ntstatus) In a 
directory with a lot of files.
Jeremy.

---

Summary of changes:
 source3/smbd/trans2.c |   28 
 1 files changed, 16 insertions(+), 12 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index 59b778c..bc7a387 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -2575,23 +2575,26 @@ static void call_trans2findnext(connection_struct *conn,
requires_resume_key = (findnext_flags  
FLAG_TRANS2_FIND_REQUIRE_RESUME);
continue_bit = (findnext_flags  FLAG_TRANS2_FIND_CONTINUE);
 
-   srvstr_get_path_wcard(ctx, params, req-flags2, resume_name,
+   if (!continue_bit) {
+   /* We only need resume_name if continue_bit is zero. */
+   srvstr_get_path_wcard(ctx, params, req-flags2, resume_name,
  params+12,
  total_params - 12, STR_TERMINATE, ntstatus,
  mask_contains_wcard);
-   if (!NT_STATUS_IS_OK(ntstatus)) {
-   /* Win9x or OS/2 can send a resume name of .. or .. This 
will cause the parser to
-  complain (it thinks we're asking for the directory above the 
shared
-  path or an invalid name). Catch this as the resume name is 
only compared, never used in
-  a file access. JRA. */
-   srvstr_pull_talloc(ctx, params, req-flags2,
+   if (!NT_STATUS_IS_OK(ntstatus)) {
+   /* Win9x or OS/2 can send a resume name of .. or .. 
This will cause the parser to
+  complain (it thinks we're asking for the directory 
above the shared
+  path or an invalid name). Catch this as the resume 
name is only compared, never used in
+  a file access. JRA. */
+   srvstr_pull_talloc(ctx, params, req-flags2,
resume_name, params+12,
total_params - 12,
STR_TERMINATE);
 
-   if (!resume_name || !(ISDOT(resume_name) || 
ISDOTDOT(resume_name))) {
-   reply_nterror(req, ntstatus);
-   return;
+   if (!resume_name || !(ISDOT(resume_name) || 
ISDOTDOT(resume_name))) {
+   reply_nterror(req, ntstatus);
+   return;
+   }
}
}
 
@@ -2599,7 +2602,8 @@ static void call_trans2findnext(connection_struct *conn,
 close_after_request=%d, close_if_end = %d requires_resume_key = %d \
 resume_key = %d resume name = %s continue=%d level = %d\n,
dptr_num, max_data_bytes, maxentries, close_after_request, 
close_if_end, 
-   requires_resume_key, resume_key, resume_name, continue_bit, 
info_level));
+   requires_resume_key, resume_key,
+   resume_name ? resume_name : (NULL), continue_bit, 
info_level));
 
if (!maxentries) {
/* W2K3 seems to treat zero as 1. */
@@ -2724,7 +2728,7 @@ total_data=%u (should be %u)\n, (unsigned 
int)total_data, (unsigned int)IVAL(pd
 * depend on the last file name instead.
 */
 
-   if(*resume_name  !continue_bit) {
+   if(!continue_bit  resume_name  *resume_name) {
SMB_STRUCT_STAT st;
 
long current_pos = 0;


-- 
Samba Shared Repository


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

2009-11-02 Thread Jeremy Allison
The branch, v3-5-test has been updated
   via  a385927... Fix bug 6867 - trans2findnext returns reply_nterror(req, 
ntstatus) In a directory with a lot of files. Jeremy.
  from  6a11a1d... ndr64: added support for trailing gap alignment

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


- Log -
commit a38592722b05bf81905198c8f244d7303abee030
Author: Jeremy Allison j...@samba.org
Date:   Mon Nov 2 12:15:25 2009 -0800

Fix bug 6867 - trans2findnext returns reply_nterror(req, ntstatus) In a 
directory with a lot of files.
Jeremy.

---

Summary of changes:
 source3/smbd/trans2.c |   28 
 1 files changed, 16 insertions(+), 12 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index 59b778c..bc7a387 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -2575,23 +2575,26 @@ static void call_trans2findnext(connection_struct *conn,
requires_resume_key = (findnext_flags  
FLAG_TRANS2_FIND_REQUIRE_RESUME);
continue_bit = (findnext_flags  FLAG_TRANS2_FIND_CONTINUE);
 
-   srvstr_get_path_wcard(ctx, params, req-flags2, resume_name,
+   if (!continue_bit) {
+   /* We only need resume_name if continue_bit is zero. */
+   srvstr_get_path_wcard(ctx, params, req-flags2, resume_name,
  params+12,
  total_params - 12, STR_TERMINATE, ntstatus,
  mask_contains_wcard);
-   if (!NT_STATUS_IS_OK(ntstatus)) {
-   /* Win9x or OS/2 can send a resume name of .. or .. This 
will cause the parser to
-  complain (it thinks we're asking for the directory above the 
shared
-  path or an invalid name). Catch this as the resume name is 
only compared, never used in
-  a file access. JRA. */
-   srvstr_pull_talloc(ctx, params, req-flags2,
+   if (!NT_STATUS_IS_OK(ntstatus)) {
+   /* Win9x or OS/2 can send a resume name of .. or .. 
This will cause the parser to
+  complain (it thinks we're asking for the directory 
above the shared
+  path or an invalid name). Catch this as the resume 
name is only compared, never used in
+  a file access. JRA. */
+   srvstr_pull_talloc(ctx, params, req-flags2,
resume_name, params+12,
total_params - 12,
STR_TERMINATE);
 
-   if (!resume_name || !(ISDOT(resume_name) || 
ISDOTDOT(resume_name))) {
-   reply_nterror(req, ntstatus);
-   return;
+   if (!resume_name || !(ISDOT(resume_name) || 
ISDOTDOT(resume_name))) {
+   reply_nterror(req, ntstatus);
+   return;
+   }
}
}
 
@@ -2599,7 +2602,8 @@ static void call_trans2findnext(connection_struct *conn,
 close_after_request=%d, close_if_end = %d requires_resume_key = %d \
 resume_key = %d resume name = %s continue=%d level = %d\n,
dptr_num, max_data_bytes, maxentries, close_after_request, 
close_if_end, 
-   requires_resume_key, resume_key, resume_name, continue_bit, 
info_level));
+   requires_resume_key, resume_key,
+   resume_name ? resume_name : (NULL), continue_bit, 
info_level));
 
if (!maxentries) {
/* W2K3 seems to treat zero as 1. */
@@ -2724,7 +2728,7 @@ total_data=%u (should be %u)\n, (unsigned 
int)total_data, (unsigned int)IVAL(pd
 * depend on the last file name instead.
 */
 
-   if(*resume_name  !continue_bit) {
+   if(!continue_bit  resume_name  *resume_name) {
SMB_STRUCT_STAT st;
 
long current_pos = 0;


-- 
Samba Shared Repository


[SCM] SAMBA-CTDB repository - branch v3-4-ctdb updated - aab0b67097154b444ddb464d65fe128d33f2dab8

2009-11-02 Thread Michael Adam
The branch, v3-4-ctdb has been updated
   via  aab0b67097154b444ddb464d65fe128d33f2dab8 (commit)
  from  be7a16a43a458bb3033bbd2256882ca17a11c6a1 (commit)

http://gitweb.samba.org/?p=obnox/samba-ctdb.git;a=shortlog;h=v3-4-ctdb


- Log -
commit aab0b67097154b444ddb464d65fe128d33f2dab8
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 16:59:15 2009 +0100

s3: Fix a 100% CPU loop when ctdbd dies during a traverse

Signed-off-by: Michael Adam ob...@samba.org

---

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


Changeset truncated at 500 lines:

diff --git a/source3/lib/ctdbd_conn.c b/source3/lib/ctdbd_conn.c
index d34d23e..540e62d 100644
--- a/source3/lib/ctdbd_conn.c
+++ b/source3/lib/ctdbd_conn.c
@@ -1168,6 +1168,7 @@ NTSTATUS ctdbd_traverse(uint32 db_id,
 
if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
status = NT_STATUS_OK;
+   break;
}
 
if (!NT_STATUS_IS_OK(status)) {


-- 
SAMBA-CTDB repository


[SCM] SAMBA-CTDB repository - branch v3-4-ctdb updated - 29173408a3d9050d83aee87b033b0e8937ce9240

2009-11-02 Thread Michael Adam
The branch, v3-4-ctdb has been updated
   via  29173408a3d9050d83aee87b033b0e8937ce9240 (commit)
   via  ddc9184de972168ebb8a7f70efb1e1a19800a745 (commit)
   via  b991de4f20eb46410fc38aae9aad675eaa86ab06 (commit)
   via  7e9c18c7d46b45bb1a2ed4af1f2aacf401c4ff0b (commit)
   via  d5cf73232aff9bd7e5ebb8b70c8f28aaea3780a9 (commit)
   via  bee87fb72bdf4a2374e4ad341bd3376e963be5b8 (commit)
   via  2ba7cb6004251fa0c14a2204bf809803181f76b3 (commit)
   via  0080b44e63e28716b9b44622fa1c2454580a730b (commit)
   via  05031c3629d283632af36fe067727a32b643cc6a (commit)
   via  28a72732a1f613132044f371533c46b3b4e09be9 (commit)
   via  bb3c277954e9ae934d96aa3de6300e62c98a157c (commit)
   via  21b97f4a498ea712140c6724a762f6b3fb3fc7a7 (commit)
   via  9065e4b5f7b4b02179b2263a254a7e7f0c2b508b (commit)
   via  2902d4738a2c8207f7d6cf5b998f01db536d58b7 (commit)
  from  aab0b67097154b444ddb464d65fe128d33f2dab8 (commit)

http://gitweb.samba.org/?p=obnox/samba-ctdb.git;a=shortlog;h=v3-4-ctdb


- Log -
commit 29173408a3d9050d83aee87b033b0e8937ce9240
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 00:51:27 2009 +0100

s3:registry: add an extra check for dsize==0 to regdb_fetch_keys_internal()

Don't only rely on dptr == NULL.
I stumbled over this one when rewriting some of the dbwrap_ctdb code.

Michael

commit ddc9184de972168ebb8a7f70efb1e1a19800a745
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 00:47:37 2009 +0100

s3:registry: add safety check for return value of tdb_unpack to 
regdb_fetch_keys_internal()

Prevents segfaults in some situations.

(For a non existent or empty record, we sometimes rely on the fetch 
operation
 to return dsize==0 and sometimes we rely on dptr==NULL.)

Michael

commit b991de4f20eb46410fc38aae9aad675eaa86ab06
Author: Michael Adam ob...@samba.org
Date:   Sat Oct 31 13:16:34 2009 +0100

s3:dbwrap_ctdb: add debug message to transaction_fetch_start()

for the case that another local process has started a transaction
bewteen releasing the transaction_lock record and starting the
transaction.

Michael

commit 7e9c18c7d46b45bb1a2ed4af1f2aacf401c4ff0b
Author: Michael Adam ob...@samba.org
Date:   Sat Oct 31 13:13:04 2009 +0100

s3:dbwrap_ctdb: split combined check in two and add descriptive debug

in db_ctdb_transaction_fetch_start() for error conditions when re-fetching
the transaction_lock record inside the transaction

Michael

commit d5cf73232aff9bd7e5ebb8b70c8f28aaea3780a9
Author: Michael Adam ob...@samba.org
Date:   Thu Oct 29 00:01:45 2009 +0100

s3:dbwrap_ctdb: fix race condition with concurrent transactions on the same 
node.

In ctdb_transaction_commit(), when the trans2_commit control fails, there
is a race condition in the 1 second sleep between the local 
transaction_cancel
and the call to ctdb_replay_transaction(): The database is not locked, and
neither is the transaction_lock record. So another client can start and 
possibly
complete a new transaction in this gap, but only on the same node: The 
locking
of the transaction_lock record on a different node which involves migration 
of
the record to the other node has been disabled by introduction of the
transaction_active flag on the db which closes precisely this gap from the 
start
of the commit until the call to TRANS2_FINISH or TRANS2_ERROR.
But this mechanism does not cover the case where a process on the same node
tries to start a transaction: There is no obstacle to locking the 
transaction_lock
record because the record does not need to be migrated.

This commit closes this race condition in ctdb_transaction_fetch_start()
by using the new ctdb_ctrl_transaction_active() call to ask the local
ctdb daemon whether it has a transaction running on the database.
If so, the check is repeated until the running transaction is done.

This does introduce an additional call to the local ctdbd when starting
transactions, but it does close the (hopefully) last race condition.

Michael

commit bee87fb72bdf4a2374e4ad341bd3376e963be5b8
Author: Michael Adam ob...@samba.org
Date:   Wed Oct 28 23:56:59 2009 +0100

s3:configure: add a check for the new CTDB_CONTROL_TRANS2_ACTIVE

Michael

commit 2ba7cb6004251fa0c14a2204bf809803181f76b3
Author: Michael Adam ob...@samba.org
Date:   Wed Oct 28 23:56:03 2009 +0100

s3:dbwrap_ctdb: add new db_ctdb_transaction_active() that calls 
CTDB_CONTROL_TRANS2_COMMIT

Michael

commit 0080b44e63e28716b9b44622fa1c2454580a730b
Author: Michael Adam ob...@samba.org
Date:   Wed Oct 28 01:54:04 2009 +0100

s3:dbwrap_ctdb: fix a race in starting concurrent transactions on a single 
node

There are two races in concurrent transactions on a single node.
One in 

[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Michael Adam
The branch, master has been updated
   via  cc5b22a... s3:registry: add an extra check for dsize==0 to 
regdb_fetch_keys_internal()
   via  f6f2151... s3:registry: add safety check for return value of 
tdb_unpack to regdb_fetch_keys_internal()
   via  25bdf27... s3:dbwrap_ctdb: add debug message to 
transaction_fetch_start()
   via  9fef6a6... s3:dbwrap_ctdb: split combined check in two and add 
descriptive debug
   via  f37439e... s3:dbwrap_ctdb: fix race condition with concurrent 
transactions on the same node.
   via  08d2a3f... s3:configure: add a check for the new 
CTDB_CONTROL_TRANS2_ACTIVE
   via  9be4d3d... s3:dbwrap_ctdb: add new db_ctdb_transaction_active() 
that calls CTDB_CONTROL_TRANS2_COMMIT
   via  9bd6b9d... s3:dbwrap_ctdb: fix a race in starting concurrent 
transactions on a single node
   via  8d61b8a... s3:dbwrap_ctdb: use db_ctdb_ltdb_fetch() inside 
db_ctdb_transaction_fetch_start
   via  0ec476f... s3:dbwrap_ctdb: use db_ctdb_ltdb_fetch() inside 
db_ctdb_transaction_fetch()
   via  4973ff6... s3:dbwrap_ctdb: add a function db_ctdb_ltdb_fetch()
   via  6a89834... s3:dbrwap_ctdb: add a function db_ctdb_ltdb_store()
   via  d5aa758... s3:dbwrap_ctdb: reformat a comment slightly to enhance 
clearness.
  from  494d827... Fix bug 6867 - trans2findnext returns reply_nterror(req, 
ntstatus) In a directory with a lot of files. Jeremy.

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


- Log -
commit cc5b22a0165a1c7d6a0839edd67b8107fc883464
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 00:51:27 2009 +0100

s3:registry: add an extra check for dsize==0 to regdb_fetch_keys_internal()

Don't only rely on dptr == NULL.
I stumbled over this one when rewriting some of the dbwrap_ctdb code.

Michael

commit f6f2151a3947cc105481c64a31d2405f239948cc
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 00:47:37 2009 +0100

s3:registry: add safety check for return value of tdb_unpack to 
regdb_fetch_keys_internal()

Prevents segfaults in some situations.

(For a non existent or empty record, we sometimes rely on the fetch 
operation
 to return dsize==0 and sometimes we rely on dptr==NULL.)

Michael

commit 25bdf27eaa634f9e54e6252397942beb46f07db5
Author: Michael Adam ob...@samba.org
Date:   Sat Oct 31 13:16:34 2009 +0100

s3:dbwrap_ctdb: add debug message to transaction_fetch_start()

for the case that another local process has started a transaction
bewteen releasing the transaction_lock record and starting the
transaction.

Michael

commit 9fef6ad59ee8cc2ef41f96efa3dd2efba30c
Author: Michael Adam ob...@samba.org
Date:   Sat Oct 31 13:13:04 2009 +0100

s3:dbwrap_ctdb: split combined check in two and add descriptive debug

in db_ctdb_transaction_fetch_start() for error conditions when re-fetching
the transaction_lock record inside the transaction

Michael

commit f37439efd2fbd9a9e995d838da20d60337ca07f7
Author: Michael Adam ob...@samba.org
Date:   Thu Oct 29 00:01:45 2009 +0100

s3:dbwrap_ctdb: fix race condition with concurrent transactions on the same 
node.

In ctdb_transaction_commit(), when the trans2_commit control fails, there
is a race condition in the 1 second sleep between the local 
transaction_cancel
and the call to ctdb_replay_transaction(): The database is not locked, and
neither is the transaction_lock record. So another client can start and 
possibly
complete a new transaction in this gap, but only on the same node: The 
locking
of the transaction_lock record on a different node which involves migration 
of
the record to the other node has been disabled by introduction of the
transaction_active flag on the db which closes precisely this gap from the 
start
of the commit until the call to TRANS2_FINISH or TRANS2_ERROR.
But this mechanism does not cover the case where a process on the same node
tries to start a transaction: There is no obstacle to locking the 
transaction_lock
record because the record does not need to be migrated.

This commit closes this race condition in ctdb_transaction_fetch_start()
by using the new ctdb_ctrl_transaction_active() call to ask the local
ctdb daemon whether it has a transaction running on the database.
If so, the check is repeated until the running transaction is done.

This does introduce an additional call to the local ctdbd when starting
transactions, but it does close the (hopefully) last race condition.

Michael

commit 08d2a3f4bf1a4e3ce571ff59c9a582243c3db57a
Author: Michael Adam ob...@samba.org
Date:   Wed Oct 28 23:56:59 2009 +0100

s3:configure: add a check for the new CTDB_CONTROL_TRANS2_ACTIVE

Michael

commit 9be4d3dd4f46f0fbd4e20ec33bae7f2e23dba2fd
Author: Michael Adam ob...@samba.org
Date:  

[SCM] Samba Shared Repository - branch master updated

2009-11-02 Thread Jeremy Allison
The branch, master has been updated
   via  f9c9dee... Convert from numbers to correct SMB_FIND_XX constant 
names. Jeremy.
  from  cc5b22a... s3:registry: add an extra check for dsize==0 to 
regdb_fetch_keys_internal()

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


- Log -
commit f9c9dee01373d6fd069e36ba6024580b8962f230
Author: Jeremy Allison j...@samba.org
Date:   Mon Nov 2 16:17:36 2009 -0800

Convert from numbers to correct SMB_FIND_XX constant names.
Jeremy.

---

Summary of changes:
 source3/libsmb/clilist.c |   20 
 1 files changed, 12 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/libsmb/clilist.c b/source3/libsmb/clilist.c
index a84a647..5a97204 100644
--- a/source3/libsmb/clilist.c
+++ b/source3/libsmb/clilist.c
@@ -38,8 +38,9 @@ static size_t calc_next_entry_offset(const char *base, const 
char *pdata_end)
 /
  Interpret a long filename structure - this is mostly guesses at the moment.
  The length of the structure is returned
- The structure of a long filename depends on the info level. 260 is used
- by NT and 2 is used by OS/2
+ The structure of a long filename depends on the info level.
+ SMB_FIND_FILE_BOTH_DIRECTORY_INFO is used
+ by NT and SMB_FIND_EA_SIZE is used by OS/2
 /
 
 static size_t interpret_long_filename(TALLOC_CTX *ctx,
@@ -64,7 +65,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
finfo-cli = cli;
 
switch (level) {
-   case 1: /* OS/2 understands this */
+   case SMB_FIND_INFO_STANDARD: /* OS/2 understands this */
/* these dates are converted to GMT by
make_unix_date */
if (pdata_end - base  27) {
@@ -109,7 +110,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
p += ret;
return PTR_DIFF(p, base);
 
-   case 2: /* this is what OS/2 uses mostly */
+   case SMB_FIND_EA_SIZE: /* this is what OS/2 uses mostly */
/* these dates are converted to GMT by
make_unix_date */
if (pdata_end - base  31) {
@@ -138,7 +139,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
p += ret;
return PTR_DIFF(p, base) + 1;
 
-   case 260: /* NT uses this, but also accepts 2 */
+   case SMB_FIND_FILE_BOTH_DIRECTORY_INFO: /* NT uses this, but 
also accepts 2 */
{
size_t namelen, slen;
 
@@ -248,8 +249,10 @@ int cli_list_new(struct cli_state *cli,const char 
*Mask,uint16 attribute,
TALLOC_CTX *frame = talloc_stackframe();
DATA_BLOB last_name_raw = data_blob(NULL, 0);
 
-   /* NT uses 260, OS/2 uses 2. Both accept 1. */
-   info_level = (cli-capabilitiesCAP_NT_SMBS)?260:1;
+   /* NT uses SMB_FIND_FILE_BOTH_DIRECTORY_INFO,
+  OS/2 uses SMB_FIND_EA_SIZE. Both accept SMB_FIND_INFO_STANDARD. */
+   info_level = (cli-capabilitiesCAP_NT_SMBS)?
+   SMB_FIND_FILE_BOTH_DIRECTORY_INFO : SMB_FIND_INFO_STANDARD;
 
mask = SMB_STRDUP(Mask);
if (!mask) {
@@ -389,7 +392,8 @@ int cli_list_new(struct cli_state *cli,const char 
*Mask,uint16 attribute,
 
/* we might need the lastname for continuations */
for (p2=p,i=0;iff_searchcount  p2  rdata_end;i++) {
-   if ((info_level == 260)  (i == ff_searchcount-1)) {
+   if ((info_level == SMB_FIND_FILE_BOTH_DIRECTORY_INFO) 
+   (i == ff_searchcount-1)) {
/* Last entry - fixup the last offset length. */
SIVAL(p2,0,PTR_DIFF((rdata + data_len),p2));
}


-- 
Samba Shared Repository


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

2009-11-02 Thread Jeremy Allison
The branch, v3-5-test has been updated
   via  21e527c... Convert from numbers to correct SMB_FIND_XX constant 
names. Jeremy.
  from  a385927... Fix bug 6867 - trans2findnext returns reply_nterror(req, 
ntstatus) In a directory with a lot of files. Jeremy.

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


- Log -
commit 21e527c5d3d98c6e730c23281d68b0b18737b704
Author: Jeremy Allison j...@samba.org
Date:   Mon Nov 2 16:17:36 2009 -0800

Convert from numbers to correct SMB_FIND_XX constant names.
Jeremy.

---

Summary of changes:
 source3/libsmb/clilist.c |   20 
 1 files changed, 12 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/libsmb/clilist.c b/source3/libsmb/clilist.c
index a84a647..5a97204 100644
--- a/source3/libsmb/clilist.c
+++ b/source3/libsmb/clilist.c
@@ -38,8 +38,9 @@ static size_t calc_next_entry_offset(const char *base, const 
char *pdata_end)
 /
  Interpret a long filename structure - this is mostly guesses at the moment.
  The length of the structure is returned
- The structure of a long filename depends on the info level. 260 is used
- by NT and 2 is used by OS/2
+ The structure of a long filename depends on the info level.
+ SMB_FIND_FILE_BOTH_DIRECTORY_INFO is used
+ by NT and SMB_FIND_EA_SIZE is used by OS/2
 /
 
 static size_t interpret_long_filename(TALLOC_CTX *ctx,
@@ -64,7 +65,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
finfo-cli = cli;
 
switch (level) {
-   case 1: /* OS/2 understands this */
+   case SMB_FIND_INFO_STANDARD: /* OS/2 understands this */
/* these dates are converted to GMT by
make_unix_date */
if (pdata_end - base  27) {
@@ -109,7 +110,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
p += ret;
return PTR_DIFF(p, base);
 
-   case 2: /* this is what OS/2 uses mostly */
+   case SMB_FIND_EA_SIZE: /* this is what OS/2 uses mostly */
/* these dates are converted to GMT by
make_unix_date */
if (pdata_end - base  31) {
@@ -138,7 +139,7 @@ static size_t interpret_long_filename(TALLOC_CTX *ctx,
p += ret;
return PTR_DIFF(p, base) + 1;
 
-   case 260: /* NT uses this, but also accepts 2 */
+   case SMB_FIND_FILE_BOTH_DIRECTORY_INFO: /* NT uses this, but 
also accepts 2 */
{
size_t namelen, slen;
 
@@ -248,8 +249,10 @@ int cli_list_new(struct cli_state *cli,const char 
*Mask,uint16 attribute,
TALLOC_CTX *frame = talloc_stackframe();
DATA_BLOB last_name_raw = data_blob(NULL, 0);
 
-   /* NT uses 260, OS/2 uses 2. Both accept 1. */
-   info_level = (cli-capabilitiesCAP_NT_SMBS)?260:1;
+   /* NT uses SMB_FIND_FILE_BOTH_DIRECTORY_INFO,
+  OS/2 uses SMB_FIND_EA_SIZE. Both accept SMB_FIND_INFO_STANDARD. */
+   info_level = (cli-capabilitiesCAP_NT_SMBS)?
+   SMB_FIND_FILE_BOTH_DIRECTORY_INFO : SMB_FIND_INFO_STANDARD;
 
mask = SMB_STRDUP(Mask);
if (!mask) {
@@ -389,7 +392,8 @@ int cli_list_new(struct cli_state *cli,const char 
*Mask,uint16 attribute,
 
/* we might need the lastname for continuations */
for (p2=p,i=0;iff_searchcount  p2  rdata_end;i++) {
-   if ((info_level == 260)  (i == ff_searchcount-1)) {
+   if ((info_level == SMB_FIND_FILE_BOTH_DIRECTORY_INFO) 
+   (i == ff_searchcount-1)) {
/* Last entry - fixup the last offset length. */
SIVAL(p2,0,PTR_DIFF((rdata + data_len),p2));
}


-- 
Samba Shared Repository


[SCM] CTDB repository - branch master updated - ctdb-1.0.102-21-g020e2e3

2009-11-02 Thread Ronnie Sahlberg
The branch, master has been updated
   via  020e2e30e56b9675f345ee62d6bf585396208059 (commit)
   via  fb83dba255fc91413a475b273e374e0c4d538137 (commit)
   via  209ff041596e39688186c5863ed3e816b8e4 (commit)
   via  f98fefbc566eefbfcc660646af6e25256ab82b13 (commit)
  from  a5d90d8ed8b44355c4ffb9c32ded772025fcc174 (commit)

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


- Log -
commit 020e2e30e56b9675f345ee62d6bf585396208059
Author: Ronnie Sahlberg ronniesahlb...@gmail.com
Date:   Tue Nov 3 11:46:37 2009 +1100

new version 1.0.103

commit fb83dba255fc91413a475b273e374e0c4d538137
Author: Ronnie Sahlberg ronniesahlb...@gmail.com
Date:   Tue Nov 3 10:48:27 2009 +1100

move the check to skip vacuuming on persistent database to the 
ctdb_vacuuming_init() function

commit 209ff041596e39688186c5863ed3e816b8e4
Author: Michael Adam ob...@samba.org
Date:   Mon Nov 2 01:37:07 2009 +0100

packaging: use githash in rpm release by default.

setting USE_GITHASH=no in the environment makes
makerpms.sh omit the git hash

Michael

commit f98fefbc566eefbfcc660646af6e25256ab82b13
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 00:04:27 2009 +0100

server: disable vacuuming for persistent tdbs.

The vacuum process treats persistent databases the same as
non-persistent and thus ignores the extra state for transactions.
This way, it breaks the api-level transactions.

Michael

---

Summary of changes:
 packaging/RPM/{ctdb.spec = ctdb.spec.in} |   12 ++--
 packaging/RPM/makerpms.sh |   11 +++
 server/ctdb_ltdb_server.c |3 ++-
 server/ctdb_vacuum.c  |5 +
 4 files changed, 28 insertions(+), 3 deletions(-)
 rename packaging/RPM/{ctdb.spec = ctdb.spec.in} (98%)


Changeset truncated at 500 lines:

diff --git a/packaging/RPM/ctdb.spec b/packaging/RPM/ctdb.spec.in
similarity index 98%
rename from packaging/RPM/ctdb.spec
rename to packaging/RPM/ctdb.spec.in
index a64b4b8..2052ae4 100644
--- a/packaging/RPM/ctdb.spec
+++ b/packaging/RPM/ctdb.spec.in
@@ -4,8 +4,8 @@ Summary: Clustered TDB
 Vendor: Samba Team
 Packager: Samba Team sa...@samba.org
 Name: ctdb
-Version: 1.0.102
-Release: 1
+Version: 1.0.103
+Release: 1GITHASH
 Epoch: 0
 License: GNU GPL version 3
 Group: System Environment/Daemons
@@ -118,6 +118,14 @@ rm -rf $RPM_BUILD_ROOT
 %{_libdir}/pkgconfig/ctdb.pc
 
 %changelog
+* Tue Nov 3 2009 : Version 1.0.103
+ - Dont use vacuuming on persistent databases
+ - Michael A : transaction updates to persistent databases
+ - Dont activate service automatically when installing the RPM. Leave this to 
the admin.
+ - Create a child process to send all log messages to, to prevent a hung/slow 
syslogd
+   from blocking the main daemon. In this case, discard log messages instead 
and let the child
+   process block.
+ - Michael A: updates to log messages
 * Thu Oct 29 2009 : Version 1.0.102
  - Wolfgang: fix for the vacuuming code
  - Wolfgang: stronger tests for persistent database filename tests
diff --git a/packaging/RPM/makerpms.sh b/packaging/RPM/makerpms.sh
index 846ab7f..a51099c 100755
--- a/packaging/RPM/makerpms.sh
+++ b/packaging/RPM/makerpms.sh
@@ -41,8 +41,19 @@ SPECDIR=`rpm --eval %_specdir`
 SRCDIR=`rpm --eval %_sourcedir`
 
 SPECFILE=ctdb.spec
+SPECFILE_IN=ctdb.spec.in
 RPMBUILD=rpmbuild
 
+GITHASH=.$(git log --pretty=format:%h -1)
+
+if test x$USE_GITHASH = xno ; then
+   GITHASH=
+fi
+
+sed -e s/GITHASH/${GITHASH}/g \
+${DIRNAME}/${SPECFILE_IN} \
+${DIRNAME}/${SPECFILE}
+
 VERSION=$(grep ^Version ${DIRNAME}/${SPECFILE} | sed -e 's/^Version:\ \+//')
 RELEASE=$(grep ^Release ${DIRNAME}/${SPECFILE} | sed -e 's/^Release:\ \+//')
 
diff --git a/server/ctdb_ltdb_server.c b/server/ctdb_ltdb_server.c
index 09e40c7..1aa0f69 100644
--- a/server/ctdb_ltdb_server.c
+++ b/server/ctdb_ltdb_server.c
@@ -295,7 +295,8 @@ static int ctdb_local_attach(struct ctdb_context *ctdb, 
const char *db_name, boo
 
ret = ctdb_vacuum_init(ctdb_db);
if (ret != 0) {
-   DEBUG(DEBUG_CRIT,(Failed to setup vacuuming for database 
'%s'\n, ctdb_db-db_name));
+   DEBUG(DEBUG_CRIT,(Failed to setup vacuuming for 
+ database '%s'\n, ctdb_db-db_name));
talloc_free(ctdb_db);
return -1;
}
diff --git a/server/ctdb_vacuum.c b/server/ctdb_vacuum.c
index 80e5dbd..52f2060 100644
--- a/server/ctdb_vacuum.c
+++ b/server/ctdb_vacuum.c
@@ -872,6 +872,11 @@ ctdb_vacuum_event(struct event_context *ev, struct 
timed_event *te,
  */
 int ctdb_vacuum_init(struct ctdb_db_context *ctdb_db)
 {
+   if (ctdb_db-persistent != 0) {
+   DEBUG(DEBUG_ERR,(Vacuuming is disabled for persistent database 
%s\n, ctdb_db-db_name));
+

[SCM] CTDB repository - annotated tag ctdb-1.0.103 created - ctdb-1.0.103

2009-11-02 Thread Ronnie Sahlberg
The annotated tag, ctdb-1.0.103 has been created
at  b7306b101ed1c165089930346d52fba352d8150f (tag)
   tagging  020e2e30e56b9675f345ee62d6bf585396208059 (commit)
  replaces  ctdb-1.0.102
 tagged by  Ronnie Sahlberg
on  Tue Nov 3 14:22:34 2009 +1100

- Log -
tag for 103
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQBK76IK2aJ36aon/y8RAhkEAJ9v+526Ib72CQqrOHemgaihQGYLoQCfVNDH
tIsFh98nKfE4E3J5NuW/j2Y=
=ObnZ
-END PGP SIGNATURE-

Michael Adam (17):
  server: trans2_active: don't report a transaction active on the node that 
performs the transaction
  client: improve control timed out debug message
  server: add positive debug statements to trans2_commit and trans2_finished
  server: extend a debug message in ctdb_control_trans2_error()
  server: fix a debug message in trans2_commit - log the correct db_id
  server: output client_id in some debug messages in trans2_commit
  server: line-wrap a debug statement in trans2_commit
  server: uniformly log db and client ids as 8-digit hex numbers in 
trans2_commit
  client: log db_id as 8-digit hex in ctdb_transaction_fetch_start()
  server: fix debug message in trans2_commit (refusing persistent store 
during transaction)
  packaging(RPM): don't touch the run levels in ctdb install/udpate.
  client: add a debug message when a transaction_commit needs to be retried
  client: fix a race in the local race condition fix in transaction_start
  Revert dont exit on a commit failure
  client: randomize the transaction_start retry loop:
  server: disable vacuuming for persistent tdbs.
  packaging: use githash in rpm release by default.

Ronnie Sahlberg (4):
  create a child process to write to syslog.
  start the syslog child a little later, after we have forked and detached 
from the local shell
  move the check to skip vacuuming on persistent database to the 
ctdb_vacuuming_init() function
  new version 1.0.103

---


-- 
CTDB repository


Build status as of Tue Nov 3 07:00:04 2009

2009-11-02 Thread build
URL: http://build.samba.org/

--- /home/build/master/cache/broken_results.txt.old 2009-11-02 
00:00:29.0 -0700
+++ /home/build/master/cache/broken_results.txt 2009-11-03 00:00:36.0 
-0700
@@ -1,22 +1,22 @@
-Build status as of Mon Nov  2 07:00:04 2009
+Build status as of Tue Nov  3 07:00:04 2009
 
 Build counts:
 Tree Total  Broken Panic 
 build_farm   0  0  0 
-ccache   5  1  0 
+ccache   6  1  0 
 distcc   0  0  0 
-ldb  22 22 0 
-libreplace   1  1  0 
+ldb  26 26 0 
+libreplace   2  1  0 
 lorikeet 0  0  0 
-pidl 2  2  0 
-ppp  3  0  0 
+pidl 3  3  0 
+ppp  4  0  0 
 rsync28 11 0 
 samba-docs   0  0  0 
 samba-web0  0  0 
-samba_3_current 2  2  0 
+samba_3_current 2  1  0 
 samba_3_master 25 24 1 
-samba_3_next 25 24 0 
-samba_4_0_test 26 24 0 
-talloc   2  2  0 
-tdb  6  6  0 
+samba_3_next 24 22 0 
+samba_4_0_test 25 23 0 
+talloc   3  3  0 
+tdb  4  4  0 
 


[SCM] SAMBA-CTDB repository - branch v3-2-ctdb updated - 3.2.11-ctdb-65-7-g67e1376

2009-11-02 Thread Michael Adam
The branch, v3-2-ctdb has been updated
   via  67e13766284ac39e9a7e1098a7252c7f5243d447 (commit)
   via  3d5b9abb49b9d525106eb76097075153881dad36 (commit)
  from  587caa3d8ad6a97c480e371a3cc5bc3b2c932ec7 (commit)

http://gitweb.samba.org/?p=obnox/samba-ctdb.git;a=shortlog;h=v3-2-ctdb


- Log -
commit 67e13766284ac39e9a7e1098a7252c7f5243d447
Author: Volker Lendecke v...@samba.org
Date:   Mon Nov 2 16:59:15 2009 +0100

s3: Fix a 100% CPU loop when ctdbd dies during a traverse

commit 3d5b9abb49b9d525106eb76097075153881dad36
Author: Michael Adam ob...@samba.org
Date:   Tue Nov 3 08:08:34 2009 +0100

v3-2-ctdb: Bump the ctdb vendor patch level to 68.

Michael

---

Summary of changes:
 source/VERSION  |2 +-
 source/lib/ctdbd_conn.c |1 +
 2 files changed, 2 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/VERSION b/source/VERSION
index 7374600..e8f05af 100644
--- a/source/VERSION
+++ b/source/VERSION
@@ -96,4 +96,4 @@ SAMBA_VERSION_IS_GIT_SNAPSHOT=
 #  -  CVS 3.0.0rc2-VendorVersion#
 
 SAMBA_VERSION_VENDOR_SUFFIX=ctdb
-SAMBA_VERSION_VENDOR_PATCH=67
+SAMBA_VERSION_VENDOR_PATCH=68
diff --git a/source/lib/ctdbd_conn.c b/source/lib/ctdbd_conn.c
index 33a69f6..0020fcd 100644
--- a/source/lib/ctdbd_conn.c
+++ b/source/lib/ctdbd_conn.c
@@ -1162,6 +1162,7 @@ NTSTATUS ctdbd_traverse(uint32 db_id,
 
if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
status = NT_STATUS_OK;
+   break;
}
 
if (!NT_STATUS_IS_OK(status)) {


-- 
SAMBA-CTDB repository