Changeset 1136:140450615773 in freeDiameter for libfdcore/endpoints.c

Timestamp:

May 16, 2013, 3:56:31 PM (11 years ago)

Author:

Sebastien Decugis <sdecugis@freediameter.net>

Branch:

default

Phase:

public

Message:

Revert changeset 1122 (ADDRESS_AVP_INCLUDE_PORT) as this will create too much interop issues. The Host-IP-Address AVP is not normally used to discover peer address, but only for validation of the addresses where a packet is received from -- which is quite useless with a connected transport connection, but anyway...

File:

• libfdcore/endpoints.c (modified) (1 diff)

libfdcore/endpoints.c

@@ -203 +203 @@
 }
 
-/* Remove any endpoint from the exclude list in the list */
-int fd_ep_filter_list( struct fd_list * list, struct fd_list * exclude_list )
-{
-        struct fd_list * li_out, *li_ex, *li;
-        struct fd_endpoint * out, * ex;
-        
-        TRACE_ENTRY("%p %p", list, exclude_list);
-        CHECK_PARAMS(list && exclude_list);
-        
-        /* initialize. Both lists are ordered */
-        li_out = list->next;
-        li_ex = exclude_list->next;
-        
-        /* Now browse both lists in parallel */
-        while ((li_out != list) && (li_ex != exclude_list)) {
-                int cmp;
-                in_port_t * port_out, *port_ex;
-                
-                out = (struct fd_endpoint *)li_out;
-                ex = (struct fd_endpoint *)li_ex;
-
-                /* Compare the next elements families */
-                if (out->sa.sa_family < ex->sa.sa_family) {
-                        li_out = li_out->next;
-                        continue;
-                }
-                if (out->sa.sa_family > ex->sa.sa_family) {
-                        li_ex = li_ex->next;
-                        continue;
-                }
-
-                /* Then compare the address fields */
-                switch (out->sa.sa_family) {
-                        case AF_INET:
-                                cmp = memcmp(&out->sin.sin_addr, &ex->sin.sin_addr, sizeof(struct in_addr));
-                                port_out = &out->sin.sin_port;
-                                port_ex = &ex->sin.sin_port;
-                                break;
-                        case AF_INET6:
-                                cmp = memcmp(&out->sin6.sin6_addr, &ex->sin6.sin6_addr, sizeof(struct in6_addr));
-                                port_out = &out->sin6.sin6_port;
-                                port_ex = &ex->sin6.sin6_port;
-                                break;
-                        default:
-                                /* Filter this out */
-                                cmp = 0;
-                }
-                if (cmp < 0) {
-                        li_out = li_out->next;
-                        continue;
-                }
-                if (cmp > 0) {
-                        li_ex = li_ex->next;
-                        continue;
-                }
-                
-                if (port_out && (*port_out != 0) && (*port_ex != 0)) {
-                        if (*port_out < *port_ex) {
-                                li_out = li_out->next;
-                                continue;
-                        }
-                        if (*port_out > *port_ex) {
-                                li_ex = li_ex->next;
-                                continue;
-                        }
-                }
-        
-                /* We remove this element then loop */
-                li = li_out;
-                li_out = li->next;
-                fd_list_unlink(li);
-                free(li);
-        }
-        
-        return 0;
-
-}
-
-
 /* Reset the given flag(s) from all items in the list */
 int fd_ep_clearflags( struct fd_list * list, uint32_t flags )