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 <jel...@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 <jel...@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 <jel...@samba.org> Date: Tue Dec 23 23:22:57 2008 +0100 Fix more compiler warnings. commit 8df6af213a0ca9a70e30c630970b5e2d2debe821 Author: Jelmer Vernooij <jel...@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 <jel...@samba.org> Date: Tue Dec 23 22:57:11 2008 +0100 Fix more compiler warnings in various places. commit 32e03f9fa55a4c897ade55b9b2e3740facfdcfd1 Author: Jelmer Vernooij <jel...@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