Browse Source

[SCTP]: Switch sctp_del_bind_addr() to net-endian.

Callers adjusted.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Al Viro 18 years ago
parent
commit
c9a08505ec
3 changed files with 7 additions and 17 deletions
  1. 1 4
      net/sctp/bind_addr.c
  2. 1 1
      net/sctp/sm_make_chunk.c
  3. 5 12
      net/sctp/socket.c

+ 1 - 4
net/sctp/bind_addr.c

@@ -181,13 +181,10 @@ int sctp_del_bind_addr(struct sctp_bind_addr *bp, union sctp_addr *del_addr)
 {
 {
 	struct list_head *pos, *temp;
 	struct list_head *pos, *temp;
 	struct sctp_sockaddr_entry *addr;
 	struct sctp_sockaddr_entry *addr;
-	union sctp_addr tmp;
-
-	flip_to_n(&tmp, del_addr);
 
 
 	list_for_each_safe(pos, temp, &bp->address_list) {
 	list_for_each_safe(pos, temp, &bp->address_list) {
 		addr = list_entry(pos, struct sctp_sockaddr_entry, list);
 		addr = list_entry(pos, struct sctp_sockaddr_entry, list);
-		if (sctp_cmp_addr_exact(&addr->a, &tmp)) {
+		if (sctp_cmp_addr_exact(&addr->a, del_addr)) {
 			/* Found the exact match. */
 			/* Found the exact match. */
 			list_del(pos);
 			list_del(pos);
 			kfree(addr);
 			kfree(addr);

+ 1 - 1
net/sctp/sm_make_chunk.c

@@ -2609,7 +2609,7 @@ static int sctp_asconf_param_success(struct sctp_association *asoc,
 	case SCTP_PARAM_DEL_IP:
 	case SCTP_PARAM_DEL_IP:
 		sctp_local_bh_disable();
 		sctp_local_bh_disable();
 		sctp_write_lock(&asoc->base.addr_lock);
 		sctp_write_lock(&asoc->base.addr_lock);
-		retval = sctp_del_bind_addr(bp, &addr);
+		retval = sctp_del_bind_addr(bp, &tmp);
 		sctp_write_unlock(&asoc->base.addr_lock);
 		sctp_write_unlock(&asoc->base.addr_lock);
 		sctp_local_bh_enable();
 		sctp_local_bh_enable();
 		list_for_each(pos, &asoc->peer.transport_addr_list) {
 		list_for_each(pos, &asoc->peer.transport_addr_list) {

+ 5 - 12
net/sctp/socket.c

@@ -607,9 +607,8 @@ int sctp_bindx_rem(struct sock *sk, struct sockaddr *addrs, int addrcnt)
 	int cnt;
 	int cnt;
 	struct sctp_bind_addr *bp = &ep->base.bind_addr;
 	struct sctp_bind_addr *bp = &ep->base.bind_addr;
 	int retval = 0;
 	int retval = 0;
-	union sctp_addr saveaddr;
 	void *addr_buf;
 	void *addr_buf;
-	struct sockaddr *sa_addr;
+	union sctp_addr *sa_addr;
 	struct sctp_af *af;
 	struct sctp_af *af;
 
 
 	SCTP_DEBUG_PRINTK("sctp_bindx_rem (sk: %p, addrs: %p, addrcnt: %d)\n",
 	SCTP_DEBUG_PRINTK("sctp_bindx_rem (sk: %p, addrs: %p, addrcnt: %d)\n",
@@ -627,19 +626,13 @@ int sctp_bindx_rem(struct sock *sk, struct sockaddr *addrs, int addrcnt)
 			goto err_bindx_rem;
 			goto err_bindx_rem;
 		}
 		}
 
 
-		/* The list may contain either IPv4 or IPv6 address;
-		 * determine the address length to copy the address to
-		 * saveaddr. 
-		 */
-		sa_addr = (struct sockaddr *)addr_buf;
-		af = sctp_get_af_specific(sa_addr->sa_family);
+		sa_addr = (union sctp_addr *)addr_buf;
+		af = sctp_get_af_specific(sa_addr->sa.sa_family);
 		if (!af) {
 		if (!af) {
 			retval = -EINVAL;
 			retval = -EINVAL;
 			goto err_bindx_rem;
 			goto err_bindx_rem;
 		}
 		}
-		memcpy(&saveaddr, sa_addr, af->sockaddr_len); 
-		saveaddr.v4.sin_port = ntohs(saveaddr.v4.sin_port);
-		if (saveaddr.v4.sin_port != bp->port) {
+		if (sa_addr->v4.sin_port != htons(bp->port)) {
 			retval = -EINVAL;
 			retval = -EINVAL;
 			goto err_bindx_rem;
 			goto err_bindx_rem;
 		}
 		}
@@ -654,7 +647,7 @@ int sctp_bindx_rem(struct sock *sk, struct sockaddr *addrs, int addrcnt)
 		sctp_local_bh_disable();
 		sctp_local_bh_disable();
 		sctp_write_lock(&ep->base.addr_lock);
 		sctp_write_lock(&ep->base.addr_lock);
 
 
-		retval = sctp_del_bind_addr(bp, &saveaddr);
+		retval = sctp_del_bind_addr(bp, sa_addr);
 
 
 		sctp_write_unlock(&ep->base.addr_lock);
 		sctp_write_unlock(&ep->base.addr_lock);
 		sctp_local_bh_enable();
 		sctp_local_bh_enable();