The branch, master has been updated via 50be0b2 s3: Remove superfluous ; via 3bdd2ef nsswitch: remove unused code from wbinfo.c from a62d67e s3: Fix a typo
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 50be0b2c1e7e0573245e21df886ab491a0738226 Author: Günther Deschner <g...@samba.org> Date: Wed Feb 2 12:50:42 2011 +0100 s3: Remove superfluous ; Guenther Autobuild-User: Günther Deschner <g...@samba.org> Autobuild-Date: Wed Feb 2 15:44:21 CET 2011 on sn-devel-104 commit 3bdd2efff8d11448275965b464ac194a2ee0d3ae Author: Günther Deschner <g...@samba.org> Date: Wed Feb 2 12:43:59 2011 +0100 nsswitch: remove unused code from wbinfo.c Guenther ----------------------------------------------------------------------- Summary of changes: nsswitch/wbinfo.c | 27 --------------------------- source3/lib/ctdbd_conn.c | 2 +- source3/lib/secdesc.c | 8 ++++---- source3/modules/vfs_acl_common.c | 4 ++-- source3/modules/vfs_xattr_tdb.c | 4 ++-- source3/nmbd/nmbd_winsserver.c | 2 +- source3/registry/regfio.c | 2 +- source3/rpc_client/cli_pipe_schannel.c | 2 +- source3/rpc_server/srv_samr_nt.c | 2 +- source3/rpc_server/srv_spoolss_nt.c | 2 +- source3/rpc_server/srv_spoolss_util.c | 2 +- source3/rpc_server/srv_srvsvc_nt.c | 2 +- source3/rpcclient/cmd_spoolss.c | 2 +- source3/winbindd/winbindd_lookupsid.c | 2 +- 14 files changed, 18 insertions(+), 45 deletions(-) Changeset truncated at 500 lines: diff --git a/nsswitch/wbinfo.c b/nsswitch/wbinfo.c index a4b2a3c..1056fb3 100644 --- a/nsswitch/wbinfo.c +++ b/nsswitch/wbinfo.c @@ -139,31 +139,6 @@ static bool parse_wbinfo_domain_user(const char *domuser, fstring domain, return true; } -/* Parse string of "uid,sid" or "gid,sid" into separate int and string values. - * Return true if input was valid, false otherwise. */ -static bool parse_mapping_arg(char *arg, int *id, char **sid) -{ - char *tmp, *endptr; - - if (!arg || !*arg) - return false; - - tmp = strtok(arg, ","); - *sid = strtok(NULL, ","); - - if (!tmp || !*tmp || !*sid || !**sid) - return false; - - /* Because atoi() can return 0 on invalid input, which would be a valid - * UID/GID we must use strtoul() and do error checking */ - *id = strtoul(tmp, &endptr, 10); - - if (endptr[0] != '\0') - return false; - - return true; -} - /* pull pwent info for a given user */ static bool wbinfo_get_userinfo(char *user) @@ -1862,10 +1837,8 @@ int main(int argc, char **argv, char **envp) TALLOC_CTX *frame = talloc_stackframe(); poptContext pc; static char *string_arg; - char *string_subarg = NULL; static char *opt_domain_name; static int int_arg; - int int_subarg = -1; int result = 1; bool verbose = false; bool use_ntlmv2 = false; diff --git a/source3/lib/ctdbd_conn.c b/source3/lib/ctdbd_conn.c index d201693..bad5285 100644 --- a/source3/lib/ctdbd_conn.c +++ b/source3/lib/ctdbd_conn.c @@ -155,7 +155,7 @@ static bool ctdbd_working(struct ctdbd_connection *conn, uint32_t vnn) ret = true; fail: TALLOC_FREE(outdata.dptr); - return ret;; + return ret; } uint32 ctdbd_vnn(const struct ctdbd_connection *conn) diff --git a/source3/lib/secdesc.c b/source3/lib/secdesc.c index 9ceba10..001eccb 100644 --- a/source3/lib/secdesc.c +++ b/source3/lib/secdesc.c @@ -266,7 +266,7 @@ NTSTATUS marshall_sec_desc(TALLOC_CTX *mem_ctx, if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(0, ("ndr_push_security_descriptor failed: %s\n", ndr_errstr(ndr_err))); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *data = blob.data; @@ -292,7 +292,7 @@ NTSTATUS marshall_sec_desc_buf(TALLOC_CTX *mem_ctx, if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(0, ("ndr_push_sec_desc_buf failed: %s\n", ndr_errstr(ndr_err))); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *data = blob.data; @@ -328,7 +328,7 @@ NTSTATUS unmarshall_sec_desc(TALLOC_CTX *mem_ctx, uint8 *data, size_t len, DEBUG(0, ("ndr_pull_security_descriptor failed: %s\n", ndr_errstr(ndr_err))); TALLOC_FREE(result); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *psecdesc = result; @@ -364,7 +364,7 @@ NTSTATUS unmarshall_sec_desc_buf(TALLOC_CTX *mem_ctx, uint8_t *data, size_t len, DEBUG(0, ("ndr_pull_sec_desc_buf failed: %s\n", ndr_errstr(ndr_err))); TALLOC_FREE(result); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *psecdesc_buf = result; diff --git a/source3/modules/vfs_acl_common.c b/source3/modules/vfs_acl_common.c index 3296ddc..c89ca7a 100644 --- a/source3/modules/vfs_acl_common.c +++ b/source3/modules/vfs_acl_common.c @@ -86,7 +86,7 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob, if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(5, ("parse_acl_blob: ndr_pull_xattr_NTACL failed: %s\n", ndr_errstr(ndr_err))); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } switch (xacl.version) { @@ -153,7 +153,7 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd, if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(5, ("create_acl_blob: ndr_push_xattr_NTACL failed: %s\n", ndr_errstr(ndr_err))); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } return NT_STATUS_OK; diff --git a/source3/modules/vfs_xattr_tdb.c b/source3/modules/vfs_xattr_tdb.c index 435b39a..594f2e6 100644 --- a/source3/modules/vfs_xattr_tdb.c +++ b/source3/modules/vfs_xattr_tdb.c @@ -56,7 +56,7 @@ static NTSTATUS xattr_tdb_pull_attrs(TALLOC_CTX *mem_ctx, DEBUG(0, ("ndr_pull_tdb_xattrs failed: %s\n", ndr_errstr(ndr_err))); TALLOC_FREE(result); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *presult = result; @@ -80,7 +80,7 @@ static NTSTATUS xattr_tdb_push_attrs(TALLOC_CTX *mem_ctx, if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { DEBUG(0, ("ndr_push_tdb_xattrs failed: %s\n", ndr_errstr(ndr_err))); - return ndr_map_error2ntstatus(ndr_err);; + return ndr_map_error2ntstatus(ndr_err); } *data = make_tdb_data(blob.data, blob.length); diff --git a/source3/nmbd/nmbd_winsserver.c b/source3/nmbd/nmbd_winsserver.c index d4580e4..2f48bc7 100644 --- a/source3/nmbd/nmbd_winsserver.c +++ b/source3/nmbd/nmbd_winsserver.c @@ -2138,7 +2138,7 @@ void wins_process_name_release_request(struct subnet_record *subrec, uint16 nb_flags = get_nb_flags(nmb->additional->rdata); struct name_record *namerec = NULL; struct in_addr from_ip; - bool releasing_group_name = (nb_flags & NB_GROUP) ? True : False;; + bool releasing_group_name = (nb_flags & NB_GROUP) ? True : False; putip((char *)&from_ip,&nmb->additional->rdata[2]); diff --git a/source3/registry/regfio.c b/source3/registry/regfio.c index 7ab262d..e300b14 100644 --- a/source3/registry/regfio.c +++ b/source3/registry/regfio.c @@ -1478,7 +1478,7 @@ static REGF_HBIN* regf_hbin_allocate( REGF_FILE *file, uint32 block_size ) hbin->file_off = sbuf.st_ex_size; hbin->free_off = HBIN_HEADER_REC_SIZE; - hbin->free_size = block_size - hbin->free_off + sizeof(uint32);; + hbin->free_size = block_size - hbin->free_off + sizeof(uint32); hbin->block_size = block_size; hbin->first_hbin_off = hbin->file_off - REGF_BLOCKSIZE; diff --git a/source3/rpc_client/cli_pipe_schannel.c b/source3/rpc_client/cli_pipe_schannel.c index 7bc8d80..2c605c6 100644 --- a/source3/rpc_client/cli_pipe_schannel.c +++ b/source3/rpc_client/cli_pipe_schannel.c @@ -76,7 +76,7 @@ static NTSTATUS get_schannel_session_key_common(struct rpc_pipe_client *netlogon return NT_STATUS_INVALID_NETWORK_RESPONSE; } - return NT_STATUS_OK;; + return NT_STATUS_OK; } /**************************************************************************** diff --git a/source3/rpc_server/srv_samr_nt.c b/source3/rpc_server/srv_samr_nt.c index 116d529..5087ec3 100644 --- a/source3/rpc_server/srv_samr_nt.c +++ b/source3/rpc_server/srv_samr_nt.c @@ -1491,7 +1491,7 @@ NTSTATUS _samr_QueryDisplayInfo(struct pipes_struct *p, temp_size=max_entries*struct_size; if (temp_size > r->in.buf_size) { - max_entries = MIN((r->in.buf_size / struct_size),max_entries);; + max_entries = MIN((r->in.buf_size / struct_size),max_entries); DEBUG(5, ("_samr_QueryDisplayInfo: buffer size limits to " "only %d entries\n", max_entries)); } diff --git a/source3/rpc_server/srv_spoolss_nt.c b/source3/rpc_server/srv_spoolss_nt.c index defffb6..a06e643 100644 --- a/source3/rpc_server/srv_spoolss_nt.c +++ b/source3/rpc_server/srv_spoolss_nt.c @@ -4798,7 +4798,7 @@ static WERROR string_array_from_driver_info(TALLOC_CTX *mem_ctx, const char **array = NULL; if (string_array == NULL) { - return WERR_INVALID_PARAMETER;; + return WERR_INVALID_PARAMETER; } for (i=0; string_array[i] && string_array[i][0] != '\0'; i++) { diff --git a/source3/rpc_server/srv_spoolss_util.c b/source3/rpc_server/srv_spoolss_util.c index 79549ac..70601c8 100644 --- a/source3/rpc_server/srv_spoolss_util.c +++ b/source3/rpc_server/srv_spoolss_util.c @@ -461,7 +461,7 @@ static WERROR winreg_printer_enumvalues(TALLOC_CTX *mem_ctx, uint8_t *data; uint32_t data_size; uint32_t length; - char n = '\0';; + char n = '\0'; name_buf.name = &n; name_buf.size = max_valnamelen + 2; diff --git a/source3/rpc_server/srv_srvsvc_nt.c b/source3/rpc_server/srv_srvsvc_nt.c index 82b8939..fe325d2 100644 --- a/source3/rpc_server/srv_srvsvc_nt.c +++ b/source3/rpc_server/srv_srvsvc_nt.c @@ -387,7 +387,7 @@ static void init_srv_share_info_502(struct pipes_struct *p, struct sec_desc_buf *sd_buf = NULL; size_t sd_size = 0; TALLOC_CTX *ctx = p->mem_ctx; - char *remark = talloc_strdup(ctx, lp_comment(snum));; + char *remark = talloc_strdup(ctx, lp_comment(snum)); if (remark) { remark = talloc_sub_advanced( diff --git a/source3/rpcclient/cmd_spoolss.c b/source3/rpcclient/cmd_spoolss.c index 248996e..25a1ff6 100644 --- a/source3/rpcclient/cmd_spoolss.c +++ b/source3/rpcclient/cmd_spoolss.c @@ -1890,7 +1890,7 @@ static WERROR cmd_spoolss_setdriver(struct rpc_pipe_client *cli, } if (!W_ERROR_IS_OK(result)) { printf("SetPrinter call failed!\n"); - goto done;; + goto done; } printf("Successfully set %s to driver %s.\n", argv[1], argv[2]); diff --git a/source3/winbindd/winbindd_lookupsid.c b/source3/winbindd/winbindd_lookupsid.c index 596699f..f38dc71 100644 --- a/source3/winbindd/winbindd_lookupsid.c +++ b/source3/winbindd/winbindd_lookupsid.c @@ -54,7 +54,7 @@ struct tevent_req *winbindd_lookupsid_send(TALLOC_CTX *mem_ctx, if (!string_to_sid(&state->sid, request->data.sid)) { DEBUG(5, ("%s not a SID\n", request->data.sid)); tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER); - return tevent_req_post(req, ev);; + return tevent_req_post(req, ev); } subreq = wb_lookupsid_send(state, ev, &state->sid); -- Samba Shared Repository