[SCM] Samba Shared Repository - branch v4-0-test updated - release-4-0-0alpha3-1616-ge17804b

[Andrew Bartlett]

The branch, v4-0-test has been updated
       via  e17804b8857fdb3c182c5e886323b9d6c194c2ff (commit)
      from  b6f91ce75ae401bed515012fa3019a6241e7ff6d (commit)

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


- Log -----------------------------------------------------------------
commit e17804b8857fdb3c182c5e886323b9d6c194c2ff
Author: Andrew Bartlett <abartlet at samba.org>
Date:   Tue May 20 11:54:50 2008 +1000

    Revert addition of 'mailslot' parameter.
    
    It turns out that the mailslot name (and a useful private prointer) is
    provided in the struct dgram_mailslot_handler.
    
    Andrew Bartlett

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

Summary of changes:
 source/libcli/dgram/dgramsocket.c  |    2 +-
 source/libcli/dgram/libdgram.h     |    1 -
 source/nbt_server/dgram/browse.c   |    1 -
 source/nbt_server/dgram/netlogon.c |    9 +++------
 source/torture/nbt/dgram.c         |    3 +--
 5 files changed, 5 insertions(+), 11 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/libcli/dgram/dgramsocket.c b/source/libcli/dgram/dgramsocket.c
index 2cdda65..06b7bd5 100644
--- a/source/libcli/dgram/dgramsocket.c
+++ b/source/libcli/dgram/dgramsocket.c
@@ -88,7 +88,7 @@ static void dgm_socket_recv(struct nbt_dgram_socket *dgmsock)
 		struct dgram_mailslot_handler *dgmslot;
 		dgmslot = dgram_mailslot_find(dgmsock, mailslot_name);
 		if (dgmslot) {
-			dgmslot->handler(dgmslot, packet, mailslot_name, src);
+			dgmslot->handler(dgmslot, packet, src);
 		} else {
 			DEBUG(2,("No mailslot handler for '%s'\n", mailslot_name));
 		}
diff --git a/source/libcli/dgram/libdgram.h b/source/libcli/dgram/libdgram.h
index 51408d0..e1209e7 100644
--- a/source/libcli/dgram/libdgram.h
+++ b/source/libcli/dgram/libdgram.h
@@ -70,7 +70,6 @@ struct nbt_dgram_socket {
 
 typedef void (*dgram_mailslot_handler_t)(struct dgram_mailslot_handler *, 
 					 struct nbt_dgram_packet *, 
-					 const char *mailslot_name,
 					 struct socket_address *src);
 
 struct dgram_mailslot_handler {
diff --git a/source/nbt_server/dgram/browse.c b/source/nbt_server/dgram/browse.c
index 36f0160..2e12fa1 100644
--- a/source/nbt_server/dgram/browse.c
+++ b/source/nbt_server/dgram/browse.c
@@ -49,7 +49,6 @@ static const char *nbt_browse_opcode_string(enum nbt_browse_opcode r)
 */
 void nbtd_mailslot_browse_handler(struct dgram_mailslot_handler *dgmslot, 
 				  struct nbt_dgram_packet *packet, 
-				  const char *mailslot_name,
 				  struct socket_address *src)
 {
 	struct nbt_browse_packet *browse = talloc(dgmslot, struct nbt_browse_packet);
diff --git a/source/nbt_server/dgram/netlogon.c b/source/nbt_server/dgram/netlogon.c
index c660895..b8fb1b8 100644
--- a/source/nbt_server/dgram/netlogon.c
+++ b/source/nbt_server/dgram/netlogon.c
@@ -38,7 +38,6 @@
 static void nbtd_netlogon_getdc(struct dgram_mailslot_handler *dgmslot, 
 				struct nbtd_interface *iface,
 				struct nbt_dgram_packet *packet, 
-				const char *mailslot_name,
 				const struct socket_address *src,
 				struct nbt_netlogon_packet *netlogon)
 {
@@ -106,7 +105,6 @@ static void nbtd_netlogon_getdc(struct dgram_mailslot_handler *dgmslot,
 static void nbtd_netlogon_samlogon(struct dgram_mailslot_handler *dgmslot,
 				   struct nbtd_interface *iface,
 				   struct nbt_dgram_packet *packet, 
-				   const char *mailslot_name,
 				   const struct socket_address *src,
 				   struct nbt_netlogon_packet *netlogon)
 {
@@ -135,7 +133,7 @@ static void nbtd_netlogon_samlogon(struct dgram_mailslot_handler *dgmslot,
 	}
 
 	if (netlogon->req.logon.sid_size) {
-		if (strcasecmp(mailslot_name, NBT_MAILSLOT_NTLOGON) == 0) {
+		if (strcasecmp(dgmslot->mailslot_name, NBT_MAILSLOT_NTLOGON) == 0) {
 			DEBUG(2,("NBT netlogon query failed because SID specified in request to NTLOGON\n"));
 			/* SID not permitted on NTLOGON (for some reason...) */ 
 			return;
@@ -171,7 +169,6 @@ static void nbtd_netlogon_samlogon(struct dgram_mailslot_handler *dgmslot,
 */
 void nbtd_mailslot_netlogon_handler(struct dgram_mailslot_handler *dgmslot, 
 				    struct nbt_dgram_packet *packet, 
-				    const char *mailslot_name,
 				    struct socket_address *src)
 {
 	NTSTATUS status = NT_STATUS_NO_MEMORY;
@@ -200,11 +197,11 @@ void nbtd_mailslot_netlogon_handler(struct dgram_mailslot_handler *dgmslot,
 
 	switch (netlogon->command) {
 	case LOGON_PRIMARY_QUERY:
-		nbtd_netlogon_getdc(dgmslot, iface, packet, mailslot_name, 
+		nbtd_netlogon_getdc(dgmslot, iface, packet, 
 				    src, netlogon);
 		break;
 	case LOGON_SAM_LOGON_REQUEST:
-		nbtd_netlogon_samlogon(dgmslot, iface, packet, mailslot_name, 
+		nbtd_netlogon_samlogon(dgmslot, iface, packet, 
 				       src, netlogon);
 		break;
 	default:
diff --git a/source/torture/nbt/dgram.c b/source/torture/nbt/dgram.c
index ce57589..2bc3d3d 100644
--- a/source/torture/nbt/dgram.c
+++ b/source/torture/nbt/dgram.c
@@ -39,7 +39,6 @@
 */
 static void netlogon_handler(struct dgram_mailslot_handler *dgmslot, 
 			     struct nbt_dgram_packet *packet, 
-			     const char *mailslot_name,
 			     struct socket_address *src)
 {
 	NTSTATUS status;
@@ -54,7 +53,7 @@ static void netlogon_handler(struct dgram_mailslot_handler *dgmslot,
 		       src->addr, src->port);
 		return;
 	}
-
+	
 	(*replies)++;
 }
 


-- 
Samba Shared Repository