[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Wed May 6 20:54:05 MDT 2015


The branch, master has been updated
       via  57941aa s3: utils: Convert all uses of uint32/16/8 to _t.
       via  bacd990 lib: Fix strv_next for the anchor NULL entry
      from  c49ad3f s3:selftest: run smb2.notify with --signing=required

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


- Log -----------------------------------------------------------------
commit 57941aa044b8e0ddac5cb1004172ba06bb4298a3
Author: Richard Sharpe <rsharpe at samba.org>
Date:   Wed May 6 17:00:06 2015 -0700

    s3: utils: Convert all uses of uint32/16/8 to _t.
    
    Signed-off-by: Richard Sharpe <rsharpe at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>
    
    Autobuild-User(master): Jeremy Allison <jra at samba.org>
    Autobuild-Date(master): Thu May  7 04:53:39 CEST 2015 on sn-devel-104

commit bacd990184e05738f80865dd363add57c59384c0
Author: Volker Lendecke <vl at samba.org>
Date:   Tue Apr 28 10:28:52 2015 +0200

    lib: Fix strv_next for the anchor NULL entry
    
    I swear I have tested this somewhere....
    
    Signed-off-by: Volker Lendecke <vl at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>

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

Summary of changes:
 lib/util/strv.c                       |  7 ++++
 source3/utils/eventlogadm.c           |  2 +-
 source3/utils/log2pcaphex.c           | 56 +++++++++++++-------------
 source3/utils/net.c                   | 10 ++---
 source3/utils/net.h                   |  2 +-
 source3/utils/net_ads.c               |  2 +-
 source3/utils/net_ads_gpo.c           | 14 +++----
 source3/utils/net_groupmap.c          |  4 +-
 source3/utils/net_proto.h             |  2 +-
 source3/utils/net_rap.c               | 40 +++++++++----------
 source3/utils/net_registry.c          |  4 +-
 source3/utils/net_registry_util.c     |  2 +-
 source3/utils/net_rpc.c               | 62 ++++++++++++++---------------
 source3/utils/net_rpc_printer.c       |  2 +-
 source3/utils/net_rpc_registry.c      | 74 +++++++++++++++++------------------
 source3/utils/net_rpc_rights.c        | 24 ++++++------
 source3/utils/net_rpc_service.c       | 22 +++++------
 source3/utils/net_sam.c               | 16 ++++----
 source3/utils/ntlm_auth.c             | 18 ++++-----
 source3/utils/ntlm_auth_diagnostics.c | 18 ++++-----
 source3/utils/ntlm_auth_proto.h       |  8 ++--
 source3/utils/pdbedit.c               |  4 +-
 source3/utils/regedit.h               |  4 +-
 source3/utils/regedit_wrap.c          |  4 +-
 source3/utils/sharesec.c              |  6 +--
 source3/utils/smbcacls.c              | 14 +++----
 source3/utils/smbtree.c               |  6 +--
 27 files changed, 217 insertions(+), 210 deletions(-)


Changeset truncated at 500 lines:

diff --git a/lib/util/strv.c b/lib/util/strv.c
index 2acd145..f5879c7 100644
--- a/lib/util/strv.c
+++ b/lib/util/strv.c
@@ -84,6 +84,13 @@ char *strv_next(char *strv, const char *entry)
 	size_t len, entry_len;
 	char *result;
 
+	if (entry == NULL) {
+		if (strv_valid_entry(strv, strv, &len, &entry_len)) {
+			return strv;
+		}
+		return NULL;
+	}
+
 	if (!strv_valid_entry(strv, entry, &len, &entry_len)) {
 		return NULL;
 	}
diff --git a/source3/utils/eventlogadm.c b/source3/utils/eventlogadm.c
index 4c368b4..5ef091a 100644
--- a/source3/utils/eventlogadm.c
+++ b/source3/utils/eventlogadm.c
@@ -91,7 +91,7 @@ static bool eventlog_add_source( const char *eventlog, const char *sourcename,
 	const char *hive_name, *relpath;
 	enum winreg_CreateAction action;
 	struct registry_value *value;
-	static const uint32 ACCESS = REG_KEY_READ | REG_KEY_WRITE;
+	static const uint32_t ACCESS = REG_KEY_READ | REG_KEY_WRITE;
 	bool ret = false;
 
 	if (!elogs) {
diff --git a/source3/utils/log2pcaphex.c b/source3/utils/log2pcaphex.c
index 866abfd..d565a24 100644
--- a/source3/utils/log2pcaphex.c
+++ b/source3/utils/log2pcaphex.c
@@ -74,47 +74,47 @@ int hexformat = 0;
 
 /* tcpdump file format */
 struct tcpdump_file_header {
-	uint32 magic;
-	uint16 major;
-	uint16 minor;
+	uint32_t magic;
+	uint16_t major;
+	uint16_t minor;
 	int32 zone;
-	uint32 sigfigs;
-	uint32 snaplen;
-	uint32 linktype;
+	uint32_t sigfigs;
+	uint32_t snaplen;
+	uint32_t linktype;
 };
 
 struct tcpdump_packet {
 	struct timeval ts;
-	uint32 caplen;
-	uint32 len;
+	uint32_t caplen;
+	uint32_t len;
 };
 
 typedef struct {
-    uint8  ver_hdrlen;
-    uint8  dscp;
-    uint16 packet_length;
-    uint16 identification;
-    uint8  flags;
-    uint8  fragment;
-    uint8  ttl;
-    uint8  protocol;
-    uint16 hdr_checksum;
-    uint32 src_addr;
-    uint32 dest_addr;
+    uint8_t  ver_hdrlen;
+    uint8_t  dscp;
+    uint16_t packet_length;
+    uint16_t identification;
+    uint8_t  flags;
+    uint8_t  fragment;
+    uint8_t  ttl;
+    uint8_t  protocol;
+    uint16_t hdr_checksum;
+    uint32_t src_addr;
+    uint32_t dest_addr;
 } hdr_ip_t;
 
 static hdr_ip_t HDR_IP = {0x45, 0, 0, 0x3412, 0, 0, 0xff, 6, 0, 0x01010101, 0x02020202};
 
 typedef struct {
-    uint16 source_port;
-    uint16 dest_port;
-    uint32 seq_num;
-    uint32 ack_num;
-    uint8  hdr_length;
-    uint8  flags;
-    uint16 window;
-    uint16 checksum;
-    uint16 urg;
+    uint16_t source_port;
+    uint16_t dest_port;
+    uint32_t seq_num;
+    uint32_t ack_num;
+    uint8_t  hdr_length;
+    uint8_t  flags;
+    uint16_t window;
+    uint16_t checksum;
+    uint16_t urg;
 } hdr_tcp_t;
 
 static hdr_tcp_t HDR_TCP = {139, 139, 0, 0, 0x50, 0, 0, 0, 0};
diff --git a/source3/utils/net.c b/source3/utils/net.c
index f53e627..ceaf5da 100644
--- a/source3/utils/net.c
+++ b/source3/utils/net.c
@@ -370,10 +370,10 @@ static int net_getdomainsid(struct net_context *c, int argc, const char **argv)
 }
 
 static bool search_maxrid(struct pdb_search *search, const char *type,
-			  uint32 *max_rid)
+			  uint32_t *max_rid)
 {
 	struct samr_displayentry *entries;
-	uint32 i, num_entries;
+	uint32_t i, num_entries;
 
 	if (search == NULL) {
 		d_fprintf(stderr, _("get_maxrid: Could not search %s\n"), type);
@@ -387,9 +387,9 @@ static bool search_maxrid(struct pdb_search *search, const char *type,
 	return true;
 }
 
-static uint32 get_maxrid(void)
+static uint32_t get_maxrid(void)
 {
-	uint32 max_rid = 0;
+	uint32_t max_rid = 0;
 
 	if (!search_maxrid(pdb_search_users(talloc_tos(), 0), "users", &max_rid))
 		return 0;
@@ -407,7 +407,7 @@ static uint32 get_maxrid(void)
 
 static int net_maxrid(struct net_context *c, int argc, const char **argv)
 {
-	uint32 rid;
+	uint32_t rid;
 
 	if (argc != 0) {
 	        d_fprintf(stderr, "%s net maxrid\n", _("Usage:"));
diff --git a/source3/utils/net.h b/source3/utils/net.h
index ce19c57..cded8e1 100644
--- a/source3/utils/net.h
+++ b/source3/utils/net.h
@@ -133,7 +133,7 @@ typedef struct copy_clistate {
 	struct cli_state *cli_share_src;
 	struct cli_state *cli_share_dst;
 	char *cwd;
-	uint16 attribute;
+	uint16_t attribute;
 	struct net_context *c;
 }copy_clistate;
 
diff --git a/source3/utils/net_ads.c b/source3/utils/net_ads.c
index b0e7112..a8f3892 100644
--- a/source3/utils/net_ads.c
+++ b/source3/utils/net_ads.c
@@ -230,7 +230,7 @@ static void use_in_memory_ccache(void) {
 }
 
 static ADS_STATUS ads_startup_int(struct net_context *c, bool only_own_domain,
-				  uint32 auth_flags, ADS_STRUCT **ads_ret)
+				  uint32_t auth_flags, ADS_STRUCT **ads_ret)
 {
 	ADS_STRUCT *ads = NULL;
 	ADS_STATUS status;
diff --git a/source3/utils/net_ads_gpo.c b/source3/utils/net_ads_gpo.c
index 8b789e5..f2f65c8 100644
--- a/source3/utils/net_ads_gpo.c
+++ b/source3/utils/net_ads_gpo.c
@@ -34,8 +34,8 @@ static int net_ads_gpo_refresh(struct net_context *c, int argc, const char **arg
 	const char *dn = NULL;
 	struct GROUP_POLICY_OBJECT *gpo_list = NULL;
 	struct GROUP_POLICY_OBJECT *read_list = NULL;
-	uint32 uac = 0;
-	uint32 flags = 0;
+	uint32_t uac = 0;
+	uint32_t flags = 0;
 	struct GROUP_POLICY_OBJECT *gpo;
 	NTSTATUS result;
 	struct security_token *token = NULL;
@@ -305,8 +305,8 @@ static int net_ads_gpo_list(struct net_context *c, int argc, const char **argv)
 	LDAPMessage *res = NULL;
 	TALLOC_CTX *mem_ctx;
 	const char *dn = NULL;
-	uint32 uac = 0;
-	uint32 flags = 0;
+	uint32_t uac = 0;
+	uint32_t flags = 0;
 	struct GROUP_POLICY_OBJECT *gpo_list;
 	struct security_token *token = NULL;
 
@@ -376,8 +376,8 @@ static int net_ads_gpo_apply(struct net_context *c, int argc, const char **argv)
 	ADS_STATUS status;
 	const char *dn = NULL;
 	struct GROUP_POLICY_OBJECT *gpo_list;
-	uint32 uac = 0;
-	uint32 flags = 0;
+	uint32_t uac = 0;
+	uint32_t flags = 0;
 	struct security_token *token = NULL;
 	const char *filter = NULL;
 
@@ -501,7 +501,7 @@ static int net_ads_gpo_link_add(struct net_context *c, int argc, const char **ar
 {
 	ADS_STRUCT *ads;
 	ADS_STATUS status;
-	uint32 gpo_opt = 0;
+	uint32_t gpo_opt = 0;
 	TALLOC_CTX *mem_ctx;
 
 	if (argc < 2 || c->display_usage) {
diff --git a/source3/utils/net_groupmap.c b/source3/utils/net_groupmap.c
index 8c6aa2e..0f86df0 100644
--- a/source3/utils/net_groupmap.c
+++ b/source3/utils/net_groupmap.c
@@ -195,7 +195,7 @@ static int net_groupmap_add(struct net_context *c, int argc, const char **argv)
 	fstring type = "";
 	fstring ntcomment = "";
 	enum lsa_SidType sid_type = SID_NAME_DOM_GRP;
-	uint32 rid = 0;
+	uint32_t rid = 0;
 	gid_t gid;
 	int i;
 	GROUP_MAP *map;
@@ -860,7 +860,7 @@ static bool print_alias_memberships(TALLOC_CTX *mem_ctx,
 				    const struct dom_sid *domain_sid,
 				    const struct dom_sid *member)
 {
-	uint32 *alias_rids;
+	uint32_t *alias_rids;
 	size_t i, num_alias_rids;
 
 	alias_rids = NULL;
diff --git a/source3/utils/net_proto.h b/source3/utils/net_proto.h
index 25e9db2..27cdb8d 100644
--- a/source3/utils/net_proto.h
+++ b/source3/utils/net_proto.h
@@ -332,7 +332,7 @@ int rpc_vampire_keytab(struct net_context *c, int argc, const char **argv);
 
 /* The following definitions come from utils/net_rpc_service.c  */
 
-const char *svc_status_string( uint32 state );
+const char *svc_status_string( uint32_t state );
 int net_rpc_service(struct net_context *c, int argc, const char **argv);
 
 /* The following definitions come from utils/net_rpc_sh_acct.c  */
diff --git a/source3/utils/net_rap.c b/source3/utils/net_rap.c
index d5b78fc..4be7b5f 100644
--- a/source3/utils/net_rap.c
+++ b/source3/utils/net_rap.c
@@ -53,15 +53,15 @@ int net_rap_file_usage(struct net_context *c, int argc, const char **argv)
 /***************************************************************************
   list info on an open file
 ***************************************************************************/
-static void file_fn(const char * pPath, const char * pUser, uint16 perms,
-		    uint16 locks, uint32 id)
+static void file_fn(const char * pPath, const char * pUser, uint16_t perms,
+		    uint16_t locks, uint32_t id)
 {
 	d_printf("%-7.1d %-20.20s 0x%-4.2x %-6.1d %s\n",
 		 id, pUser, perms, locks, pPath);
 }
 
-static void one_file_fn(const char *pPath, const char *pUser, uint16 perms,
-			uint16 locks, uint32 id)
+static void one_file_fn(const char *pPath, const char *pUser, uint16_t perms,
+			uint16_t locks, uint32_t id)
 {
 	d_printf(_("File ID          %d\n"
 		   "User name        %s\n"
@@ -196,14 +196,14 @@ int net_rap_share_usage(struct net_context *c, int argc, const char **argv)
 	return net_share_usage(c, argc, argv);
 }
 
-static void long_share_fn(const char *share_name, uint32 type,
+static void long_share_fn(const char *share_name, uint32_t type,
 			  const char *comment, void *state)
 {
 	d_printf("%-12s %-8.8s %-50s\n",
 		 share_name, net_share_type_str(type), comment);
 }
 
-static void share_fn(const char *share_name, uint32 type,
+static void share_fn(const char *share_name, uint32_t type,
 		     const char *comment, void *state)
 {
 	d_printf("%s\n", share_name);
@@ -348,9 +348,9 @@ int net_rap_session_usage(struct net_context *c, int argc, const char **argv)
 	return -1;
 }
 
-static void list_sessions_func(char *wsname, char *username, uint16 conns,
-			uint16 opens, uint16 users, uint32 sess_time,
-			uint32 idle_time, uint32 user_flags, char *clitype)
+static void list_sessions_func(char *wsname, char *username, uint16_t conns,
+			uint16_t opens, uint16_t users, uint32_t sess_time,
+			uint32_t idle_time, uint32 user_flags, char *clitype)
 {
 	int hrs = idle_time / 3600;
 	int min = (idle_time / 60) % 60;
@@ -361,9 +361,9 @@ static void list_sessions_func(char *wsname, char *username, uint16 conns,
 }
 
 static void display_session_func(const char *wsname, const char *username,
-				 uint16 conns, uint16 opens, uint16 users,
-				 uint32 sess_time, uint32 idle_time,
-				 uint32 user_flags, const char *clitype)
+				 uint16_t conns, uint16_t opens, uint16 users,
+				 uint32_t sess_time, uint32 idle_time,
+				 uint32_t user_flags, const char *clitype)
 {
 	int ihrs = idle_time / 3600;
 	int imin = (idle_time / 60) % 60;
@@ -382,8 +382,8 @@ static void display_session_func(const char *wsname, const char *username,
 		 shrs, smin, ssec, ihrs, imin, isec);
 }
 
-static void display_conns_func(uint16 conn_id, uint16 conn_type, uint16 opens,
-			       uint16 users, uint32 conn_time,
+static void display_conns_func(uint16_t conn_id, uint16_t conn_type, uint16 opens,
+			       uint16_t users, uint32_t conn_time,
 			       const char *username, const char *netname)
 {
 	d_printf("%-14.14s %-8.8s %5d\n",
@@ -496,7 +496,7 @@ int net_rap_session(struct net_context *c, int argc, const char **argv)
 /****************************************************************************
 list a server name
 ****************************************************************************/
-static void display_server_func(const char *name, uint32 m,
+static void display_server_func(const char *name, uint32_t m,
 				const char *comment, void * reserved)
 {
 	d_printf("\t%-16.16s     %s\n", name, comment);
@@ -634,10 +634,10 @@ int net_rap_printq_usage(struct net_context *c, int argc, const char **argv)
 	return -1;
 }
 
-static void enum_queue(const char *queuename, uint16 pri, uint16 start,
-		       uint16 until, const char *sep, const char *pproc,
+static void enum_queue(const char *queuename, uint16_t pri, uint16_t start,
+		       uint16_t until, const char *sep, const char *pproc,
 		       const char *dest, const char *qparms,
-		       const char *qcomment, uint16 status, uint16 jobcount)
+		       const char *qcomment, uint16_t status, uint16_t jobcount)
 {
 	d_printf(_("%-17.17s Queue %5d jobs                      "),
 		 queuename, jobcount);
@@ -660,9 +660,9 @@ static void enum_queue(const char *queuename, uint16 pri, uint16 start,
 	}
 }
 
-static void enum_jobs(uint16 jobid, const char *ownername,
+static void enum_jobs(uint16_t jobid, const char *ownername,
 		      const char *notifyname, const char *datatype,
-		      const char *jparms, uint16 pos, uint16 status,
+		      const char *jparms, uint16_t pos, uint16_t status,
 		      const char *jstatus, unsigned int submitted, unsigned int jobsize,
 		      const char *comment)
 {
diff --git a/source3/utils/net_registry.c b/source3/utils/net_registry.c
index cdfb318..0ebb51b 100644
--- a/source3/utils/net_registry.c
+++ b/source3/utils/net_registry.c
@@ -48,7 +48,7 @@
  * split given path into hive and remaining path and open the hive key
  */
 static WERROR open_hive(TALLOC_CTX *ctx, const char *path,
-			uint32 desired_access,
+			uint32_t desired_access,
 			struct registry_key **hive,
 			char **subkeyname)
 {
@@ -91,7 +91,7 @@ done:
 }
 
 static WERROR open_key(TALLOC_CTX *ctx, const char *path,
-		       uint32 desired_access,
+		       uint32_t desired_access,
 		       struct registry_key **key)
 {
 	WERROR werr;
diff --git a/source3/utils/net_registry_util.c b/source3/utils/net_registry_util.c
index 2020a0a..3552a6a 100644
--- a/source3/utils/net_registry_util.c
+++ b/source3/utils/net_registry_util.c
@@ -79,7 +79,7 @@ void print_registry_value(const struct registry_value *valvalue, bool raw)
 		break;
 	}
 	case REG_MULTI_SZ: {
-		uint32 j;
+		uint32_t j;
 		const char **a;
 
 		if (!pull_reg_multi_sz(talloc_tos(), &valvalue->data, &a)) {
diff --git a/source3/utils/net_rpc.c b/source3/utils/net_rpc.c
index 2df009d..6eb27c9 100644
--- a/source3/utils/net_rpc.c
+++ b/source3/utils/net_rpc.c
@@ -1336,7 +1336,7 @@ static NTSTATUS rpc_sh_handle_user(struct net_context *c,
 	struct policy_handle connect_pol, domain_pol, user_pol;
 	NTSTATUS status, result;
 	struct dom_sid sid;
-	uint32 rid;
+	uint32_t rid;
 	enum lsa_SidType type;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
 
@@ -1590,7 +1590,7 @@ static NTSTATUS rpc_sh_user_flag_edit_internals(struct net_context *c,
 	NTSTATUS status, result;
 	const char *username;
 	const char *oldval = "unknown";
-	uint32 oldflags, newflags;
+	uint32_t oldflags, newflags;
 	bool newval;
 	union samr_UserInfo *info = NULL;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
@@ -2196,7 +2196,7 @@ static NTSTATUS rpc_add_groupmem(struct rpc_pipe_client *pipe_hnd,
 {
 	struct policy_handle connect_pol, domain_pol;
 	NTSTATUS status, result;
-	uint32 group_rid;
+	uint32_t group_rid;
 	struct policy_handle group_pol;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
 
@@ -2307,7 +2307,7 @@ static NTSTATUS rpc_add_aliasmem(struct rpc_pipe_client *pipe_hnd,
 {
 	struct policy_handle connect_pol, domain_pol;
 	NTSTATUS status, result;
-	uint32 alias_rid;
+	uint32_t alias_rid;
 	struct policy_handle alias_pol;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
 
@@ -2460,7 +2460,7 @@ static NTSTATUS rpc_del_groupmem(struct net_context *c,
 {
 	struct policy_handle connect_pol, domain_pol;
 	NTSTATUS status, result;
-	uint32 group_rid;
+	uint32_t group_rid;
 	struct policy_handle group_pol;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
 
@@ -2568,7 +2568,7 @@ static NTSTATUS rpc_del_aliasmem(struct rpc_pipe_client *pipe_hnd,
 {
 	struct policy_handle connect_pol, domain_pol;
 	NTSTATUS status, result;
-	uint32 alias_rid;
+	uint32_t alias_rid;
 	struct policy_handle alias_pol;
 	struct dcerpc_binding_handle *b = pipe_hnd->binding_handle;
 
@@ -2741,7 +2741,7 @@ static NTSTATUS rpc_group_list_internals(struct net_context *c,
 {
 	struct policy_handle connect_pol, domain_pol;
 	NTSTATUS status, result;
-	uint32 start_idx=0, max_entries=250, num_entries, i, loop_count = 0;
+	uint32_t start_idx=0, max_entries=250, num_entries, i, loop_count = 0;
 	struct samr_SamArray *groups = NULL;
 	bool global = false;
 	bool local = false;
@@ -3019,11 +3019,11 @@ static NTSTATUS rpc_list_group_members(struct net_context *c,
 					const char *domain_name,
 					const struct dom_sid *domain_sid,
 					struct policy_handle *domain_pol,
-					uint32 rid)
+					uint32_t rid)
 {
 	NTSTATUS result, status;
 	struct policy_handle group_pol;
-	uint32 num_members, *group_rids;
+	uint32_t num_members, *group_rids;
 	int i;
 	struct samr_RidAttrArray *rids = NULL;
 	struct lsa_Strings names;
@@ -3113,12 +3113,12 @@ static NTSTATUS rpc_list_alias_members(struct net_context *c,
 				       struct cli_state *cli,
 				       TALLOC_CTX *mem_ctx,
 				       struct policy_handle *domain_pol,
-				       uint32 rid)
+				       uint32_t rid)
 {
 	NTSTATUS result, status;
 	struct rpc_pipe_client *lsa_pipe;
 	struct policy_handle alias_pol, lsa_pol;
-	uint32 num_members;
+	uint32_t num_members;
 	struct dom_sid *alias_sids;
 	char **domains;
 	char **names;


-- 
Samba Shared Repository


More information about the samba-cvs mailing list