Build status as of Tue Sep 1 06:00:01 2009

2009-08-31 Thread build
URL: http://build.samba.org/

--- /home/build/master/cache/broken_results.txt.old 2009-08-31 
00:00:03.0 -0600
+++ /home/build/master/cache/broken_results.txt 2009-09-01 00:00:03.0 
-0600
@@ -1,22 +1,22 @@
-Build status as of Mon Aug 31 06:00:02 2009
+Build status as of Tue Sep  1 06:00:01 2009
 
 Build counts:
 Tree Total  Broken Panic 
 build_farm   0  0  0 
 ccache   2  1  0 
 distcc   0  0  0 
-ldb  25 25 0 
-libreplace   1  1  0 
+ldb  26 26 0 
+libreplace   2  1  0 
 lorikeet 0  0  0 
-pidl 2  2  0 
-ppp  1  0  0 
-rsync24 11 0 
+pidl 1  1  0 
+ppp  0  0  0 
+rsync26 11 0 
 samba-docs   0  0  0 
 samba-web0  0  0 
-samba_3_current 22 21 0 
-samba_3_master 23 23 1 
-samba_3_next 22 22 1 
-samba_4_0_test 25 25 4 
-talloc   24 24 0 
-tdb  23 23 0 
+samba_3_current 23 22 0 
+samba_3_master 24 23 2 
+samba_3_next 24 23 0 
+samba_4_0_test 26 26 3 
+talloc   26 26 0 
+tdb  24 24 0 
 


[SCM] Samba Shared Repository - branch master updated - release-4-0-0alpha8-1318-g792b544

2009-08-31 Thread Günther Deschner
The branch, master has been updated
   via  792b544e3704c792d969ebe55b950919283df5b1 (commit)
  from  94f7c98430372cc3213f56cd1e085e60837c7196 (commit)

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


- Log -
commit 792b544e3704c792d969ebe55b950919283df5b1
Author: Günther Deschner 
Date:   Mon Aug 31 17:12:07 2009 +0200

netlogon: fill in more details for the netr_LogonControlX family of calls.

Guenther

---

Summary of changes:
 librpc/gen_ndr/ndr_netlogon.c |   49 ++--
 librpc/gen_ndr/netlogon.h |   11 ++--
 librpc/idl/netlogon.idl   |   13 --
 3 files changed, 50 insertions(+), 23 deletions(-)


Changeset truncated at 500 lines:

diff --git a/librpc/gen_ndr/ndr_netlogon.c b/librpc/gen_ndr/ndr_netlogon.c
index 81fc9ac..46247b4 100644
--- a/librpc/gen_ndr/ndr_netlogon.c
+++ b/librpc/gen_ndr/ndr_netlogon.c
@@ -5611,9 +5611,14 @@ _PUBLIC_ void ndr_print_netr_InfoFlags(struct ndr_print 
*ndr, const char *name,
 {
ndr_print_uint32(ndr, name, r);
ndr->depth++;
-   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_CTRL_REPL_NEEDED", NETLOGON_CTRL_REPL_NEEDED, r);
-   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_CTRL_REPL_IN_PROGRESS", NETLOGON_CTRL_REPL_IN_PROGRESS, r);
-   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_CTRL_REPL_FULL_SYNC", NETLOGON_CTRL_REPL_FULL_SYNC, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_REPLICATION_NEEDED", NETLOGON_REPLICATION_NEEDED, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_REPLICATION_IN_PROGRESS", NETLOGON_REPLICATION_IN_PROGRESS, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_FULL_SYNC_REPLICATION", NETLOGON_FULL_SYNC_REPLICATION, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "NETLOGON_REDO_NEEDED", 
NETLOGON_REDO_NEEDED, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "NETLOGON_HAS_IP", 
NETLOGON_HAS_IP, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "NETLOGON_HAS_TIMESERV", 
NETLOGON_HAS_TIMESERV, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_DNS_UPDATE_FAILURE", NETLOGON_DNS_UPDATE_FAILURE, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"NETLOGON_VERIFY_STATUS_RETURNED", NETLOGON_VERIFY_STATUS_RETURNED, r);
ndr->depth--;
 }
 
@@ -5882,8 +5887,9 @@ static enum ndr_err_code 
ndr_push_netr_CONTROL_QUERY_INFORMATION(struct ndr_push
NDR_CHECK(ndr_push_unique_ptr(ndr, r->info4));
break; }
 
-   default:
-   return ndr_push_error(ndr, NDR_ERR_BAD_SWITCH, 
"Bad switch value %u at %s", level, __location__);
+   default: {
+   break; }
+
}
}
if (ndr_flags & NDR_BUFFERS) {
@@ -5914,7 +5920,8 @@ static enum ndr_err_code 
ndr_push_netr_CONTROL_QUERY_INFORMATION(struct ndr_push
break;
 
default:
-   return ndr_push_error(ndr, NDR_ERR_BAD_SWITCH, 
"Bad switch value %u at %s", level, __location__);
+   break;
+
}
}
return NDR_ERR_SUCCESS;
@@ -5975,8 +5982,9 @@ static enum ndr_err_code 
ndr_pull_netr_CONTROL_QUERY_INFORMATION(struct ndr_pull
}
break; }
 
-   default:
-   return ndr_pull_error(ndr, NDR_ERR_BAD_SWITCH, 
"Bad switch value %u at %s", level, __location__);
+   default: {
+   break; }
+
}
}
if (ndr_flags & NDR_BUFFERS) {
@@ -6018,7 +6026,8 @@ static enum ndr_err_code 
ndr_pull_netr_CONTROL_QUERY_INFORMATION(struct ndr_pull
break;
 
default:
-   return ndr_pull_error(ndr, NDR_ERR_BAD_SWITCH, 
"Bad switch value %u at %s", level, __location__);
+   break;
+
}
}
return NDR_ERR_SUCCESS;
@@ -6067,7 +6076,8 @@ _PUBLIC_ void 
ndr_print_netr_CONTROL_QUERY_INFORMATION(struct ndr_print *ndr, co
break;
 
default:
-   ndr_print_bad_level(ndr, name, level);
+   break;
+
}
 }
 
@@ -6144,8 +6154,9 @@ static enum ndr_err_code 
ndr_push_netr_CONTROL_DATA_INFORMATION(struct ndr_push
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
r->debug_level));
break; }
 
-   default:
-   return ndr_push_error(ndr, NDR_ERR_BAD_SWITCH, 
"Bad switch value %u at %s", level, __location__);
+   default: {
+ 

[SCM] Samba Shared Repository - branch master updated - release-4-0-0alpha8-1317-g94f7c98

2009-08-31 Thread Günther Deschner
The branch, master has been updated
   via  94f7c98430372cc3213f56cd1e085e60837c7196 (commit)
  from  a3e9b62f440340f075859e080086daec88e226f5 (commit)

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


- Log -
commit 94f7c98430372cc3213f56cd1e085e60837c7196
Author: Günther Deschner 
Date:   Mon Aug 31 17:12:34 2009 +0200

s3-netlogon: add DS_WEB_SERVICE_REQUIRED bit for dsgetdcname calls.

Guenther

---

Summary of changes:
 librpc/gen_ndr/ndr_netlogon.c |1 +
 librpc/gen_ndr/netlogon.h |1 +
 librpc/idl/netlogon.idl   |1 +
 3 files changed, 3 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/librpc/gen_ndr/ndr_netlogon.c b/librpc/gen_ndr/ndr_netlogon.c
index 95964ca..81fc9ac 100644
--- a/librpc/gen_ndr/ndr_netlogon.c
+++ b/librpc/gen_ndr/ndr_netlogon.c
@@ -6825,6 +6825,7 @@ _PUBLIC_ void ndr_print_netr_DsRGetDCName_flags(struct 
ndr_print *ndr, const cha
ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "DS_IS_DNS_NAME", 
DS_IS_DNS_NAME, r);
ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "DS_TRY_NEXTCLOSEST_SITE", 
DS_TRY_NEXTCLOSEST_SITE, r);
ndr_print_bitmap_flag(ndr, sizeof(uint32_t), 
"DS_DIRECTORY_SERVICE_6_REQUIRED", DS_DIRECTORY_SERVICE_6_REQUIRED, r);
+   ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "DS_WEB_SERVICE_REQUIRED", 
DS_WEB_SERVICE_REQUIRED, r);
ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "DS_RETURN_DNS_NAME", 
DS_RETURN_DNS_NAME, r);
ndr_print_bitmap_flag(ndr, sizeof(uint32_t), "DS_RETURN_FLAT_NAME", 
DS_RETURN_FLAT_NAME, r);
ndr->depth--;
diff --git a/librpc/gen_ndr/netlogon.h b/librpc/gen_ndr/netlogon.h
index 9cf3828..64a9b1c 100644
--- a/librpc/gen_ndr/netlogon.h
+++ b/librpc/gen_ndr/netlogon.h
@@ -830,6 +830,7 @@ struct netr_Blob {
 #define DS_IS_DNS_NAME ( 0x0002 )
 #define DS_TRY_NEXTCLOSEST_SITE ( 0x0004 )
 #define DS_DIRECTORY_SERVICE_6_REQUIRED ( 0x0008 )
+#define DS_WEB_SERVICE_REQUIRED ( 0x0010 )
 #define DS_RETURN_DNS_NAME ( 0x4000 )
 #define DS_RETURN_FLAT_NAME ( 0x8000 )
 
diff --git a/librpc/idl/netlogon.idl b/librpc/idl/netlogon.idl
index 30a1078..e0eb9ec 100644
--- a/librpc/idl/netlogon.idl
+++ b/librpc/idl/netlogon.idl
@@ -1112,6 +1112,7 @@ interface netlogon
DS_IS_DNS_NAME  = 0x0002,
DS_TRY_NEXTCLOSEST_SITE = 0x0004,
DS_DIRECTORY_SERVICE_6_REQUIRED = 0x0008,
+   DS_WEB_SERVICE_REQUIRED = 0x0010,
DS_RETURN_DNS_NAME  = 0x4000,
DS_RETURN_FLAT_NAME = 0x8000
} netr_DsRGetDCName_flags;


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-5345-g983c6f2

2009-08-31 Thread Karolin Seeger
The branch, v3-3-test has been updated
   via  983c6f22f411aab2488fe41b5b06174c55108868 (commit)
  from  570a8cf5bb6924905b3ad20353d1e7b0ca087748 (commit)

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


- Log -
commit 983c6f22f411aab2488fe41b5b06174c55108868
Author: Volker Lendecke 
Date:   Tue Jul 14 23:12:59 2009 +0200

Fix bug 5886

Ok, that's a very long-standing one. I finally got around to install a 
recent
OpenLDAP and test the different variants of setting a NULL password etc.

Thanks all for your patience!

Volker

---

Summary of changes:
 source/passdb/pdb_ldap.c |   34 ++
 1 files changed, 26 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/passdb/pdb_ldap.c b/source/passdb/pdb_ldap.c
index bc485e3..73f3e87 100644
--- a/source/passdb/pdb_ldap.c
+++ b/source/passdb/pdb_ldap.c
@@ -1700,6 +1700,7 @@ static NTSTATUS ldapsam_modify_entry(struct pdb_methods 
*my_methods,
char *utf8_password;
char *utf8_dn;
size_t converted_size;
+   int ret;
 
if (!ldap_state->is_nds_ldap) {
 
@@ -1731,14 +1732,31 @@ static NTSTATUS ldapsam_modify_entry(struct pdb_methods 
*my_methods,
}
 
if ((ber_printf (ber, "{") < 0) ||
-   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_ID, 
utf8_dn) < 0) ||
-   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_NEW, 
utf8_password) < 0) ||
-   (ber_printf (ber, "n}") < 0)) {
-   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a 
value <0\n"));
-   ber_free(ber,1);
-   SAFE_FREE(utf8_dn);
-   SAFE_FREE(utf8_password);
-   return NT_STATUS_UNSUCCESSFUL;
+   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_ID,
+utf8_dn) < 0)) {
+   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a "
+"value <0\n"));
+   ber_free(ber,1);
+   SAFE_FREE(utf8_dn);
+   SAFE_FREE(utf8_password);
+   return NT_STATUS_UNSUCCESSFUL;
+   }
+
+   if ((utf8_password != NULL) && (*utf8_password != '\0')) {
+   ret = ber_printf(ber, "ts}",
+LDAP_TAG_EXOP_MODIFY_PASSWD_NEW,
+utf8_password);
+   } else {
+   ret = ber_printf(ber, "}");
+   }
+
+   if (ret < 0) {
+   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a "
+"value <0\n"));
+   ber_free(ber,1);
+   SAFE_FREE(utf8_dn);
+   SAFE_FREE(utf8_password);
+   return NT_STATUS_UNSUCCESSFUL;
}
 
if ((rc = ber_flatten (ber, &bv))<0) {


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-4-test updated - release-4-0-0alpha7-1229-g86c1dbb

2009-08-31 Thread Karolin Seeger
The branch, v3-4-test has been updated
   via  86c1dbb473323ef9480cb57584be9e02363e80af (commit)
  from  6fb3222504a0d2b42cd9292f9e11b2ff246d5c19 (commit)

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


- Log -
commit 86c1dbb473323ef9480cb57584be9e02363e80af
Author: Olaf Flebbe 
Date:   Mon Aug 17 17:31:01 2009 +0200

make smbcontrol smbd ping work proper checking for arguments handle short 
pid_t correctly

Fixes bug #6655.

(cherry picked from commit 5359e397ff190c35414f6961be61a5110e237dd5)

---

Summary of changes:
 source3/lib/util.c |   22 --
 source3/utils/smbcontrol.c |   13 +
 2 files changed, 21 insertions(+), 14 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/lib/util.c b/source3/lib/util.c
index 61a73e6..3d7336f 100644
--- a/source3/lib/util.c
+++ b/source3/lib/util.c
@@ -2743,14 +2743,15 @@ bool procid_is_me(const struct server_id *pid)
 
 struct server_id interpret_pid(const char *pid_string)
 {
-#ifdef CLUSTER_SUPPORT
-   unsigned int vnn, pid;
struct server_id result;
-   if (sscanf(pid_string, "%u:%u", &vnn, &pid) == 2) {
+   int pid;
+#ifdef CLUSTER_SUPPORT
+   unsigned int vnn;
+   if (sscanf(pid_string, "%u:%d", &vnn, &pid) == 2) {
result.vnn = vnn;
result.pid = pid;
}
-   else if (sscanf(pid_string, "%u", &pid) == 1) {
+   else if (sscanf(pid_string, "%d", &pid) == 1) {
result.vnn = get_my_vnn();
result.pid = pid;
}
@@ -2758,10 +2759,19 @@ struct server_id interpret_pid(const char *pid_string)
result.vnn = NONCLUSTER_VNN;
result.pid = -1;
}
-   return result;
 #else
-   return pid_to_procid(atoi(pid_string));
+   if (sscanf(pid_string, "%d", &pid) != 1) {
+   result.pid = -1;
+   } else {
+   result.pid = pid;
+   }
 #endif
+   /* Assigning to result.pid may have overflowed
+  Map negative pid to -1: i.e. error */
+   if (result.pid < 0) {
+   result.pid = -1;
+   }
+   return result;
 }
 
 char *procid_str(TALLOC_CTX *mem_ctx, const struct server_id *pid)
diff --git a/source3/utils/smbcontrol.c b/source3/utils/smbcontrol.c
index fc7d0aa..7fab6eb 100644
--- a/source3/utils/smbcontrol.c
+++ b/source3/utils/smbcontrol.c
@@ -1247,15 +1247,12 @@ static struct server_id parse_dest(const char *dest)
dest = "winbindd";
}
 
-   if (!(strequal(dest, "winbindd") || strequal(dest, "nmbd"))) {
-   /* Check for numeric pid number */
+   /* Check for numeric pid number */
+   result = interpret_pid(dest);
 
-   result = interpret_pid(dest);
-
-   /* Zero isn't valid if not smbd. */
-   if (result.pid && procid_valid(&result)) {
-   return result;
-   }
+   /* Zero isn't valid if not "all". */
+   if (result.pid && procid_valid(&result)) {
+   return result;
}
 
/* Look up other destinations in pidfile directory */


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-4-test updated - release-4-0-0alpha7-1228-g6fb3222

2009-08-31 Thread Karolin Seeger
The branch, v3-4-test has been updated
   via  6fb3222504a0d2b42cd9292f9e11b2ff246d5c19 (commit)
  from  2467671ba55c5ac834a8aa7ae8ad736958baab1d (commit)

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


- Log -
commit 6fb3222504a0d2b42cd9292f9e11b2ff246d5c19
Author: Volker Lendecke 
Date:   Tue Jul 14 23:12:59 2009 +0200

Fix bug 5886

Ok, that's a very long-standing one. I finally got around to install a 
recent
OpenLDAP and test the different variants of setting a NULL password etc.

Thanks all for your patience!

Volker

---

Summary of changes:
 source3/passdb/pdb_ldap.c |   34 ++
 1 files changed, 26 insertions(+), 8 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/passdb/pdb_ldap.c b/source3/passdb/pdb_ldap.c
index 0886a10..cf4889b 100644
--- a/source3/passdb/pdb_ldap.c
+++ b/source3/passdb/pdb_ldap.c
@@ -1700,6 +1700,7 @@ static NTSTATUS ldapsam_modify_entry(struct pdb_methods 
*my_methods,
char *utf8_password;
char *utf8_dn;
size_t converted_size;
+   int ret;
 
if (!ldap_state->is_nds_ldap) {
 
@@ -1731,14 +1732,31 @@ static NTSTATUS ldapsam_modify_entry(struct pdb_methods 
*my_methods,
}
 
if ((ber_printf (ber, "{") < 0) ||
-   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_ID, 
utf8_dn) < 0) ||
-   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_NEW, 
utf8_password) < 0) ||
-   (ber_printf (ber, "n}") < 0)) {
-   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a 
value <0\n"));
-   ber_free(ber,1);
-   SAFE_FREE(utf8_dn);
-   SAFE_FREE(utf8_password);
-   return NT_STATUS_UNSUCCESSFUL;
+   (ber_printf (ber, "ts", LDAP_TAG_EXOP_MODIFY_PASSWD_ID,
+utf8_dn) < 0)) {
+   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a "
+"value <0\n"));
+   ber_free(ber,1);
+   SAFE_FREE(utf8_dn);
+   SAFE_FREE(utf8_password);
+   return NT_STATUS_UNSUCCESSFUL;
+   }
+
+   if ((utf8_password != NULL) && (*utf8_password != '\0')) {
+   ret = ber_printf(ber, "ts}",
+LDAP_TAG_EXOP_MODIFY_PASSWD_NEW,
+utf8_password);
+   } else {
+   ret = ber_printf(ber, "}");
+   }
+
+   if (ret < 0) {
+   DEBUG(0,("ldapsam_modify_entry: ber_printf returns a "
+"value <0\n"));
+   ber_free(ber,1);
+   SAFE_FREE(utf8_dn);
+   SAFE_FREE(utf8_password);
+   return NT_STATUS_UNSUCCESSFUL;
}
 
if ((rc = ber_flatten (ber, &bv))<0) {


-- 
Samba Shared Repository