[SCM] Samba Shared Repository - branch master updated - 30bdb68c9ebe281d8c512f8b737141d76a7c7d68

Jelmer Vernooij jelmer at samba.org
Tue Dec 23 22:32:33 GMT 2008


The branch, master has been updated
       via  30bdb68c9ebe281d8c512f8b737141d76a7c7d68 (commit)
       via  e94e7f82798db59849cb1dc6a99b414adc11594e (commit)
       via  dcc4081f75f0d0c51878385477f7738f49a761dd (commit)
       via  8df6af213a0ca9a70e30c630970b5e2d2debe821 (commit)
       via  bd64688c6aca6512efd0bd7d95000579beea3119 (commit)
       via  32e03f9fa55a4c897ade55b9b2e3740facfdcfd1 (commit)
      from  5c75b65ca15f7c71babd835a45b252fed6534acf (commit)

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


- Log -----------------------------------------------------------------
commit 30bdb68c9ebe281d8c512f8b737141d76a7c7d68
Merge: e94e7f82798db59849cb1dc6a99b414adc11594e 5c75b65ca15f7c71babd835a45b252fed6534acf
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 23:26:32 2008 +0100

    Merge branch 'master' of ssh://git.samba.org/data/git/samba

commit e94e7f82798db59849cb1dc6a99b414adc11594e
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 23:24:12 2008 +0100

    s4: Fix make clean to remove gen_ndr objects.

commit dcc4081f75f0d0c51878385477f7738f49a761dd
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 23:22:57 2008 +0100

    Fix more compiler warnings.

commit 8df6af213a0ca9a70e30c630970b5e2d2debe821
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 23:02:54 2008 +0100

    Use 32-bit smbpid in interface as required by SMB2, in line with
    1bdd08227e7d046305705050f21f0f1b6dd6994a.

commit bd64688c6aca6512efd0bd7d95000579beea3119
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 22:57:11 2008 +0100

    Fix more compiler warnings in various places.

commit 32e03f9fa55a4c897ade55b9b2e3740facfdcfd1
Author: Jelmer Vernooij <jelmer at samba.org>
Date:   Tue Dec 23 22:57:00 2008 +0100

    Install tevent_internal.h, as Samba 4 needs it.

-----------------------------------------------------------------------

Summary of changes:
 lib/tevent/tevent.mk                          |    2 +
 lib/tevent/tevent_internal.h                  |    2 +-
 libcli/nbt/pynbt.c                            |    4 +-
 source4/Makefile                              |    2 +-
 source4/cluster/ctdb/brlock_ctdb.c            |    6 ++--
 source4/dsdb/samdb/ldb_modules/kludge_acl.c   |    8 ++--
 source4/dsdb/samdb/ldb_modules/partition.c    |    8 ++--
 source4/dsdb/schema/schema_description.c      |   12 ++++----
 source4/dsdb/schema/schema_query.c            |   38 ++++++++++++------------
 source4/lib/events/tevent_s4.c                |    4 +-
 source4/lib/ldb/modules/operational.c         |    8 +----
 source4/lib/tls/tls.c                         |   20 ++++++------
 source4/libcli/security/security_descriptor.c |    1 -
 source4/ntvfs/common/brlock.c                 |    4 +-
 source4/ntvfs/common/brlock.h                 |    6 ++--
 source4/ntvfs/ipc/ipc_rap.c                   |    2 -
 source4/ntvfs/posix/pvfs_resolve.c            |    6 ++--
 source4/torture/libnet/libnet_rpc.c           |   26 ++++++++--------
 source4/torture/raw/lookuprate.c              |    2 +-
 source4/torture/raw/tconrate.c                |    1 -
 source4/torture/rpc/netlogon.c                |    1 -
 source4/torture/rpc/samr.c                    |    2 +-
 source4/torture/rpc/schannel.c                |   10 +++---
 source4/utils/ad2oLschema.c                   |    4 +-
 source4/web_server/wsgi.c                     |    4 --
 source4/winbind/wb_init_domain.c              |    1 -
 26 files changed, 86 insertions(+), 98 deletions(-)


Changeset truncated at 500 lines:

diff --git a/lib/tevent/tevent.mk b/lib/tevent/tevent.mk
index 2e4d5a9..6538362 100644
--- a/lib/tevent/tevent.mk
+++ b/lib/tevent/tevent.mk
@@ -21,6 +21,8 @@ installdirs::
 
 installheaders:: installdirs
 	cp $(srcdir)/tevent.h $(DESTDIR)$(includedir)
+	cp $(srcdir)/tevent_internal.h $(DESTDIR)$(includedir)
+	cp $(srcdir)/tevent_util.h $(DESTDIR)$(includedir)
 
 installlibs:: installdirs
 	cp tevent.pc $(DESTDIR)$(libdir)/pkgconfig
diff --git a/lib/tevent/tevent_internal.h b/lib/tevent/tevent_internal.h
index 0d0755f..9606fad 100644
--- a/lib/tevent/tevent_internal.h
+++ b/lib/tevent/tevent_internal.h
@@ -103,7 +103,7 @@ struct ev_debug_ops {
 
 int ev_set_debug(struct event_context *ev,
 		 void (*debug)(void *context, enum ev_debug_level level,
-				const char *fmt, va_list ap),
+				const char *fmt, va_list ap) PRINTF_ATTRIBUTE(3,0),
 		 void *context);
 int ev_set_debug_stderr(struct event_context *ev);
 void ev_debug(struct event_context *ev, enum ev_debug_level level, const char *fmt, ...);
diff --git a/libcli/nbt/pynbt.c b/libcli/nbt/pynbt.c
index 6750ad7..1d8924e 100644
--- a/libcli/nbt/pynbt.c
+++ b/libcli/nbt/pynbt.c
@@ -90,7 +90,7 @@ static bool PyObject_AsDestinationTuple(PyObject *obj, const char **dest_addr, u
 	return false;
 }
 
-static bool PyObject_AsNBTName(PyObject *obj, struct nbt_name_socket *socket, struct nbt_name *name)
+static bool PyObject_AsNBTName(PyObject *obj, struct nbt_name_socket *name_socket, struct nbt_name *name)
 {
 	if (PyTuple_Check(obj)) {
 		if (PyTuple_Size(obj) == 2) {
@@ -121,7 +121,7 @@ static bool PyObject_AsNBTName(PyObject *obj, struct nbt_name_socket *socket, st
 	return false;
 }
 
-static PyObject *PyObject_FromNBTName(struct nbt_name_socket *socket, struct smb_iconv_convenience *ic,
+static PyObject *PyObject_FromNBTName(struct nbt_name_socket *name_socket, struct smb_iconv_convenience *ic,
 				      struct nbt_name *name)
 {
 	if (name->scope) {
diff --git a/source4/Makefile b/source4/Makefile
index 442a62e..fff2dbf 100644
--- a/source4/Makefile
+++ b/source4/Makefile
@@ -144,7 +144,7 @@ include $(srcdir)/static_deps.mk
 endif
 
 clean::
-	@find ../lib ../libcli -name '*.o' -o -name '*.ho' | xargs rm -f
+	@find ../lib ../libcli ../librpc ../nsswitch -name '*.o' -o -name '*.ho' | xargs rm -f
 
 DEFAULT_HEADERS = $(srcdir)/../lib/util/dlinklist.h \
 		  $(srcdir)/version.h
diff --git a/source4/cluster/ctdb/brlock_ctdb.c b/source4/cluster/ctdb/brlock_ctdb.c
index 01d6090..1cc6beb 100644
--- a/source4/cluster/ctdb/brlock_ctdb.c
+++ b/source4/cluster/ctdb/brlock_ctdb.c
@@ -378,7 +378,7 @@ reply:
 */
 static NTSTATUS brl_ctdb_lock(struct brl_context *brl,
 			      struct brl_handle *brlh,
-			      uint16_t smbpid,
+			      uint32_t smbpid,
 			      uint64_t start, uint64_t size, 
 			      enum brl_type lock_type,
 			      void *notify_ptr)
@@ -609,7 +609,7 @@ found:
 */
 static NTSTATUS brl_ctdb_unlock(struct brl_context *brl,
 				struct brl_handle *brlh, 
-				uint16_t smbpid,
+				uint32_t smbpid,
 				uint64_t start, uint64_t size)
 {
 	struct ctdb_call call;
@@ -782,7 +782,7 @@ static int brl_ctdb_locktest_func(struct ctdb_call_info *call)
 */
 static NTSTATUS brl_ctdb_locktest(struct brl_context *brl,
 				  struct brl_handle *brlh,
-				  uint16_t smbpid, 
+				  uint32_t smbpid, 
 				  uint64_t start, uint64_t size, 
 				  enum brl_type lock_type)
 {
diff --git a/source4/dsdb/samdb/ldb_modules/kludge_acl.c b/source4/dsdb/samdb/ldb_modules/kludge_acl.c
index 275e9b5..bb95c7e 100644
--- a/source4/dsdb/samdb/ldb_modules/kludge_acl.c
+++ b/source4/dsdb/samdb/ldb_modules/kludge_acl.c
@@ -93,7 +93,7 @@ static int kludge_acl_allowedAttributes(struct ldb_context *ldb, struct ldb_mess
 	struct ldb_message_element *allowedAttributes;
 	const struct dsdb_schema *schema = dsdb_get_schema(ldb);
 	TALLOC_CTX *mem_ctx;
-	char **objectclass_list, **attr_list;
+	const char **objectclass_list, **attr_list;
 	int i, ret;
 
  	/* If we don't have a schema yet, we can't do anything... */
@@ -118,7 +118,7 @@ static int kludge_acl_allowedAttributes(struct ldb_context *ldb, struct ldb_mess
 	   we alter the element array in ldb_msg_add_empty() */
 	oc_el = ldb_msg_find_element(msg, "objectClass");
 	
-	objectclass_list = talloc_array(mem_ctx, char *, oc_el->num_values + 1);
+	objectclass_list = talloc_array(mem_ctx, const char *, oc_el->num_values + 1);
 	if (!objectclass_list) {
 		ldb_oom(ldb);
 		talloc_free(mem_ctx);
@@ -126,11 +126,11 @@ static int kludge_acl_allowedAttributes(struct ldb_context *ldb, struct ldb_mess
 	}
 
 	for (i=0; oc_el && i < oc_el->num_values; i++) {
-		objectclass_list[i] = (char *)oc_el->values[i].data;
+		objectclass_list[i] = (const char *)oc_el->values[i].data;
 	}
 	objectclass_list[i] = NULL;
 
-	attr_list = dsdb_full_attribute_list(mem_ctx, schema, (const char **)objectclass_list, DSDB_SCHEMA_ALL);
+	attr_list = dsdb_full_attribute_list(mem_ctx, schema, objectclass_list, DSDB_SCHEMA_ALL);
 	if (!attr_list) {
 		ldb_asprintf_errstring(ldb, "kludge_acl: Failed to get list of attributes create %s attribute", attrName);
 		talloc_free(mem_ctx);
diff --git a/source4/dsdb/samdb/ldb_modules/partition.c b/source4/dsdb/samdb/ldb_modules/partition.c
index 0d2ce64..0000c87 100644
--- a/source4/dsdb/samdb/ldb_modules/partition.c
+++ b/source4/dsdb/samdb/ldb_modules/partition.c
@@ -1065,11 +1065,11 @@ static int partition_extended(struct ldb_module *module, struct ldb_request *req
 
 static int partition_sort_compare(const void *v1, const void *v2)
 {
-	struct dsdb_control_current_partition *p1;
-	struct dsdb_control_current_partition *p2;
+	const struct dsdb_control_current_partition *p1;
+	const struct dsdb_control_current_partition *p2;
 
-	p1 = *((struct dsdb_control_current_partition **)v1);
-	p2 = *((struct dsdb_control_current_partition **)v2);
+	p1 = *((struct dsdb_control_current_partition * const*)v1);
+	p2 = *((struct dsdb_control_current_partition * const*)v2);
 
 	return ldb_dn_compare(p1->dn, p2->dn);
 }
diff --git a/source4/dsdb/schema/schema_description.c b/source4/dsdb/schema/schema_description.c
index 5474eb5..d616f3b 100644
--- a/source4/dsdb/schema/schema_description.c
+++ b/source4/dsdb/schema/schema_description.c
@@ -215,8 +215,8 @@ char *schema_class_description(TALLOC_CTX *mem_ctx,
 			       const char **auxillary_classes,
 			       const char *subClassOf,
 			       int objectClassCategory,
-			       char **must,
-			       char **may,
+			       const char **must,
+			       const char **may,
 			       const char *schemaHexGUID)
 {
 	char *schema_entry = talloc_asprintf(mem_ctx, 
@@ -343,10 +343,10 @@ char *schema_class_to_dITContentRule(TALLOC_CTX *mem_ctx, const struct dsdb_clas
 {
 	int i;
 	char *schema_description;
-	char **aux_class_list = NULL;
-	char **attrs;
-	char **must_attr_list = NULL;
-	char **may_attr_list = NULL;
+	const char **aux_class_list = NULL;
+	const char **attrs;
+	const char **must_attr_list = NULL;
+	const char **may_attr_list = NULL;
 	TALLOC_CTX *tmp_ctx = talloc_new(mem_ctx);
 	const struct dsdb_class *aux_class;
 	if (!tmp_ctx) {
diff --git a/source4/dsdb/schema/schema_query.c b/source4/dsdb/schema/schema_query.c
index ca26ffd..f848fee 100644
--- a/source4/dsdb/schema/schema_query.c
+++ b/source4/dsdb/schema/schema_query.c
@@ -212,18 +212,18 @@ WERROR dsdb_linked_attribute_lDAPDisplayName_list(const struct dsdb_schema *sche
 	return WERR_OK;
 }
 
-char **merge_attr_list(TALLOC_CTX *mem_ctx, 
-		       char **attrs, const char **new_attrs) 
+const char **merge_attr_list(TALLOC_CTX *mem_ctx, 
+		       const char **attrs, const char * const*new_attrs) 
 {
-	char **ret_attrs;
+	const char **ret_attrs;
 	int i;
-	size_t new_len, orig_len = str_list_length((const char **)attrs);
+	size_t new_len, orig_len = str_list_length(attrs);
 	if (!new_attrs) {
 		return attrs;
 	}
 
 	ret_attrs = talloc_realloc(mem_ctx, 
-				   attrs, char *, orig_len + str_list_length(new_attrs) + 1);
+				   attrs, const char *, orig_len + str_list_length(new_attrs) + 1);
 	if (ret_attrs) {
 		for (i=0; i < str_list_length(new_attrs); i++) {
 			ret_attrs[orig_len + i] = new_attrs[i];
@@ -241,9 +241,9 @@ char **merge_attr_list(TALLOC_CTX *mem_ctx,
   considering subclasses, auxillary classes etc)
 */
 
-char **dsdb_attribute_list(TALLOC_CTX *mem_ctx, const struct dsdb_class *class, enum dsdb_attr_list_query query)
+const char **dsdb_attribute_list(TALLOC_CTX *mem_ctx, const struct dsdb_class *class, enum dsdb_attr_list_query query)
 {
-	char **attr_list = NULL;
+	const char **attr_list = NULL;
 	switch (query) {
 	case DSDB_SCHEMA_ALL_MAY:
 		attr_list = merge_attr_list(mem_ctx, attr_list, class->mayContain);
@@ -281,7 +281,7 @@ char **dsdb_attribute_list(TALLOC_CTX *mem_ctx, const struct dsdb_class *class,
 	return attr_list;
 }
 
-static char **dsdb_full_attribute_list_internal(TALLOC_CTX *mem_ctx, 
+static const char **dsdb_full_attribute_list_internal(TALLOC_CTX *mem_ctx, 
 						const struct dsdb_schema *schema, 
 						const char **class_list,
 						enum dsdb_attr_list_query query)
@@ -289,39 +289,39 @@ static char **dsdb_full_attribute_list_internal(TALLOC_CTX *mem_ctx,
 	int i;
 	const struct dsdb_class *class;
 	
-	char **attr_list = NULL;
-	char **this_class_list;
-	char **recursive_list;
+	const char **attr_list = NULL;
+	const char **this_class_list;
+	const char **recursive_list;
 
 	for (i=0; class_list && class_list[i]; i++) {
 		class = dsdb_class_by_lDAPDisplayName(schema, class_list[i]);
 		
 		this_class_list = dsdb_attribute_list(mem_ctx, class, query);
-		attr_list = merge_attr_list(mem_ctx, attr_list, (const char **)this_class_list);
+		attr_list = merge_attr_list(mem_ctx, attr_list, this_class_list);
 
 		recursive_list = dsdb_full_attribute_list_internal(mem_ctx, schema, 
 								   class->systemAuxiliaryClass, 
 								   query);
 		
-		attr_list = merge_attr_list(mem_ctx, attr_list, (const char **)recursive_list);
+		attr_list = merge_attr_list(mem_ctx, attr_list, recursive_list);
 		
 		recursive_list = dsdb_full_attribute_list_internal(mem_ctx, schema, 
 								   class->auxiliaryClass, 
 								   query);
 		
-		attr_list = merge_attr_list(mem_ctx, attr_list, (const char **)recursive_list);
+		attr_list = merge_attr_list(mem_ctx, attr_list, recursive_list);
 		
 	}
 	return attr_list;
 }
 
-char **dsdb_full_attribute_list(TALLOC_CTX *mem_ctx, 
+const char **dsdb_full_attribute_list(TALLOC_CTX *mem_ctx, 
 				const struct dsdb_schema *schema, 
 				const char **class_list,
 				enum dsdb_attr_list_query query)
 {
-	char **attr_list = dsdb_full_attribute_list_internal(mem_ctx, schema, class_list, query);
-	size_t new_len = str_list_length((const char **)attr_list);
+	const char **attr_list = dsdb_full_attribute_list_internal(mem_ctx, schema, class_list, query);
+	size_t new_len = str_list_length(attr_list);
 
 	/* Remove duplicates */
 	if (new_len > 1) {
@@ -331,8 +331,8 @@ char **dsdb_full_attribute_list(TALLOC_CTX *mem_ctx,
 		      (comparison_fn_t)strcasecmp);
 		
 		for (i=1 ; i < new_len; i++) {
-			char **val1 = &attr_list[i-1];
-			char **val2 = &attr_list[i];
+			const char **val1 = &attr_list[i-1];
+			const char **val2 = &attr_list[i];
 			if (ldb_attr_cmp(*val1, *val2) == 0) {
 				memmove(val1, val2, (new_len - i) * sizeof( *attr_list)); 
 				new_len--;
diff --git a/source4/lib/events/tevent_s4.c b/source4/lib/events/tevent_s4.c
index 80267fd..e711e43 100644
--- a/source4/lib/events/tevent_s4.c
+++ b/source4/lib/events/tevent_s4.c
@@ -17,8 +17,8 @@
 */
 
 #include "includes.h"
-#include "tevent.h"
-#include "tevent_internal.h"
+#include <tevent.h>
+#include <tevent_internal.h>
 
 /*
   this is used to catch debug messages from events
diff --git a/source4/lib/ldb/modules/operational.c b/source4/lib/ldb/modules/operational.c
index 345441b..4193f91 100644
--- a/source4/lib/ldb/modules/operational.c
+++ b/source4/lib/ldb/modules/operational.c
@@ -268,16 +268,12 @@ static int operational_search(struct ldb_module *module, struct ldb_request *req
 		}
 	}
 
-	/* use new set of attrs if any */
-	if (search_attrs == NULL) {
-		search_attrs = req->op.search.attrs;
-	}
-
 	ret = ldb_build_search_req_ex(&down_req, module->ldb, ac,
 					req->op.search.base,
 					req->op.search.scope,
 					req->op.search.tree,
-					(const char * const *)search_attrs,
+					/* use new set of attrs if any */
+					search_attrs?req->op.search.attrs:search_attrs,
 					req->controls,
 					ac, operational_callback,
 					req);
diff --git a/source4/lib/tls/tls.c b/source4/lib/tls/tls.c
index 24e4632..d16df25 100644
--- a/source4/lib/tls/tls.c
+++ b/source4/lib/tls/tls.c
@@ -454,7 +454,7 @@ init_failed:
   setup for a new connection
 */
 struct socket_context *tls_init_server(struct tls_params *params, 
-				       struct socket_context *socket,
+				       struct socket_context *socket_ctx,
 				       struct fd_event *fde, 
 				       const char *plain_chars)
 {
@@ -463,9 +463,9 @@ struct socket_context *tls_init_server(struct tls_params *params,
 	struct socket_context *new_sock;
 	NTSTATUS nt_status;
 	
-	nt_status = socket_create_with_ops(socket, &tls_socket_ops, &new_sock, 
+	nt_status = socket_create_with_ops(socket_ctx, &tls_socket_ops, &new_sock, 
 					   SOCKET_TYPE_STREAM, 
-					   socket->flags | SOCKET_FLAG_ENCRYPT);
+					   socket_ctx->flags | SOCKET_FLAG_ENCRYPT);
 	if (!NT_STATUS_IS_OK(nt_status)) {
 		return NULL;
 	}
@@ -475,13 +475,13 @@ struct socket_context *tls_init_server(struct tls_params *params,
 		return NULL;
 	}
 
-	tls->socket          = socket;
+	tls->socket          = socket_ctx;
 	tls->fde             = fde;
 	if (talloc_reference(tls, fde) == NULL) {
 		talloc_free(new_sock);
 		return NULL;
 	}
-	if (talloc_reference(tls, socket) == NULL) {
+	if (talloc_reference(tls, socket_ctx) == NULL) {
 		talloc_free(new_sock);
 		return NULL;
 	}
@@ -534,7 +534,7 @@ failed:
 /*
   setup for a new client connection
 */
-struct socket_context *tls_init_client(struct socket_context *socket,
+struct socket_context *tls_init_client(struct socket_context *socket_ctx,
 				       struct fd_event *fde,
 				       const char *ca_path)
 {
@@ -545,9 +545,9 @@ struct socket_context *tls_init_client(struct socket_context *socket,
 	struct socket_context *new_sock;
 	NTSTATUS nt_status;
 	
-	nt_status = socket_create_with_ops(socket, &tls_socket_ops, &new_sock, 
+	nt_status = socket_create_with_ops(socket_ctx, &tls_socket_ops, &new_sock, 
 					   SOCKET_TYPE_STREAM, 
-					   socket->flags | SOCKET_FLAG_ENCRYPT);
+					   socket_ctx->flags | SOCKET_FLAG_ENCRYPT);
 	if (!NT_STATUS_IS_OK(nt_status)) {
 		return NULL;
 	}
@@ -555,12 +555,12 @@ struct socket_context *tls_init_client(struct socket_context *socket,
 	tls = talloc(new_sock, struct tls_context);
 	if (tls == NULL) return NULL;
 
-	tls->socket          = socket;
+	tls->socket          = socket_ctx;
 	tls->fde             = fde;
 	if (talloc_reference(tls, fde) == NULL) {
 		return NULL;
 	}
-	if (talloc_reference(tls, socket) == NULL) {
+	if (talloc_reference(tls, socket_ctx) == NULL) {
 		return NULL;
 	}
 	new_sock->private_data    = tls;
diff --git a/source4/libcli/security/security_descriptor.c b/source4/libcli/security/security_descriptor.c
index 2bce8e8..8e9c7eb 100644
--- a/source4/libcli/security/security_descriptor.c
+++ b/source4/libcli/security/security_descriptor.c
@@ -53,7 +53,6 @@ static struct security_acl *security_acl_dup(TALLOC_CTX *mem_ctx,
 					     const struct security_acl *oacl)
 {
 	struct security_acl *nacl;
-	int i;
 
 	nacl = talloc (mem_ctx, struct security_acl);
 	if (nacl == NULL) {
diff --git a/source4/ntvfs/common/brlock.c b/source4/ntvfs/common/brlock.c
index 21670bb..a5bc5c1 100644
--- a/source4/ntvfs/common/brlock.c
+++ b/source4/ntvfs/common/brlock.c
@@ -71,7 +71,7 @@ struct brl_handle *brl_create_handle(TALLOC_CTX *mem_ctx, struct ntvfs_handle *n
 */
 NTSTATUS brl_lock(struct brl_context *brl,
 		  struct brl_handle *brlh,
-		  uint16_t smbpid,
+		  uint32_t smbpid,
 		  uint64_t start, uint64_t size, 
 		  enum brl_type lock_type,
 		  void *notify_ptr)
@@ -85,7 +85,7 @@ NTSTATUS brl_lock(struct brl_context *brl,
 */
 NTSTATUS brl_unlock(struct brl_context *brl,
 		    struct brl_handle *brlh, 
-		    uint16_t smbpid,
+		    uint32_t smbpid,
 		    uint64_t start, uint64_t size)
 {
 	return ops->brl_unlock(brl, brlh, smbpid, start, size);
diff --git a/source4/ntvfs/common/brlock.h b/source4/ntvfs/common/brlock.h
index aff30d1..75f142b 100644
--- a/source4/ntvfs/common/brlock.h
+++ b/source4/ntvfs/common/brlock.h
@@ -28,20 +28,20 @@ struct brlock_ops {
 	struct brl_handle *(*brl_create_handle)(TALLOC_CTX *, struct ntvfs_handle *, DATA_BLOB *);
 	NTSTATUS (*brl_lock)(struct brl_context *,
 			     struct brl_handle *,
-			     uint16_t ,
+			     uint32_t ,
 			     uint64_t , uint64_t , 
 			     enum brl_type ,
 			     void *);
 	NTSTATUS (*brl_unlock)(struct brl_context *,
 			       struct brl_handle *, 
-			       uint16_t ,
+			       uint32_t ,
 			       uint64_t , uint64_t );
 	NTSTATUS (*brl_remove_pending)(struct brl_context *,
 				       struct brl_handle *, 
 				       void *);
 	NTSTATUS (*brl_locktest)(struct brl_context *,
 				 struct brl_handle *,
-				 uint16_t , 
+				 uint32_t , 
 				 uint64_t , uint64_t , 
 				 enum brl_type );
 	NTSTATUS (*brl_close)(struct brl_context *,
diff --git a/source4/ntvfs/ipc/ipc_rap.c b/source4/ntvfs/ipc/ipc_rap.c
index 4969f1a..07a8b6d 100644
--- a/source4/ntvfs/ipc/ipc_rap.c
+++ b/source4/ntvfs/ipc/ipc_rap.c
@@ -53,8 +53,6 @@
 } while (0)
 
 
-#define NERR_Success 0
-#define NERR_badpass 86
 #define NERR_notsupported 50
 
 struct rap_string_heap {
diff --git a/source4/ntvfs/posix/pvfs_resolve.c b/source4/ntvfs/posix/pvfs_resolve.c
index f08de72..43547c2 100644
--- a/source4/ntvfs/posix/pvfs_resolve.c
+++ b/source4/ntvfs/posix/pvfs_resolve.c
@@ -190,16 +190,16 @@ static NTSTATUS parse_stream_name(struct smb_iconv_convenience *ic,
 				  struct pvfs_filename *name,
 				  const char *s)
 {
-	char *p;
+	char *p, *stream_name;
 	if (s[1] == '\0') {
 		return NT_STATUS_OBJECT_NAME_INVALID;
 	}
-	name->stream_name = talloc_strdup(name, s+1);
+	name->stream_name = stream_name = talloc_strdup(name, s+1);
 	if (name->stream_name == NULL) {
 		return NT_STATUS_NO_MEMORY;
 	}
 
-	p = name->stream_name;
+	p = stream_name;
 
 	while (*p) {


-- 
Samba Shared Repository


More information about the samba-cvs mailing list