Changes in uspace/lib/net/tl/tl_common.c [c5b59ce:14f1db0] in mainline

File:

• uspace/lib/net/tl/tl_common.c (modified) (7 diffs)

uspace/lib/net/tl/tl_common.c ¶

@@ -38 +38 @@
 #include <async.h>
 #include <ipc/services.h>
-#include <errno.h>
-#include <err.h>
-
+
+#include <net_err.h>
 #include <packet/packet.h>
 #include <packet/packet_client.h>
@@ -52 +51 @@
 #include <ip_remote.h>
 #include <socket_codes.h>
+#include <socket_errno.h>
 #include <ip_interface.h>
 #include <tl_interface.h>
@@ -58 +58 @@
 DEVICE_MAP_IMPLEMENT(packet_dimensions, packet_dimension_t);
 
-int
-tl_get_address_port(const struct sockaddr *addr, int addrlen, uint16_t *port)
-{
-        const struct sockaddr_in *address_in;
-        const struct sockaddr_in6 *address_in6;
-
-        if ((addrlen <= 0) || ((size_t) addrlen < sizeof(struct sockaddr)))
-                return EINVAL;
-
-        switch (addr->sa_family) {
-        case AF_INET:
-                if (addrlen != sizeof(struct sockaddr_in))
-                        return EINVAL;
-
-                address_in = (struct sockaddr_in *) addr;
-                *port = ntohs(address_in->sin_port);
-                break;
-        case AF_INET6:
-                if (addrlen != sizeof(struct sockaddr_in6))
-                                return EINVAL;
-
-                address_in6 = (struct sockaddr_in6 *) addr;
-                *port = ntohs(address_in6->sin6_port);
-                break;
-        default:
-                return EAFNOSUPPORT;
-        }
-
+int tl_get_address_port(const struct sockaddr * addr, int addrlen, uint16_t * port){
+        const struct sockaddr_in * address_in;
+        const struct sockaddr_in6 * address_in6;
+
+        if((addrlen <= 0) || ((size_t) addrlen < sizeof(struct sockaddr))){
+                return EINVAL;
+        }
+        switch(addr->sa_family){
+                case AF_INET:
+                        if(addrlen != sizeof(struct sockaddr_in)){
+                                return EINVAL;
+                        }
+                        address_in = (struct sockaddr_in *) addr;
+                        *port = ntohs(address_in->sin_port);
+                        break;
+                case AF_INET6:
+                        if(addrlen != sizeof(struct sockaddr_in6)){
+                                return EINVAL;
+                        }
+                        address_in6 = (struct sockaddr_in6 *) addr;
+                        *port = ntohs(address_in6->sin6_port);
+                        break;
+                default:
+                        return EAFNOSUPPORT;
+        }
         return EOK;
 }
@@ -115 +112 @@
                 return EBADMEM;
         
-        *packet_dimension = packet_dimensions_find(packet_dimensions,
-            device_id);
+        *packet_dimension = packet_dimensions_find(packet_dimensions, device_id);
         if (!*packet_dimension) {
                 /* Ask for and remember them if not found */
@@ -140 +136 @@
 }
 
-int
-tl_update_ip_packet_dimension(packet_dimensions_ref packet_dimensions,
-    device_id_t device_id, size_t content)
-{
+int tl_update_ip_packet_dimension(packet_dimensions_ref packet_dimensions, device_id_t device_id, size_t content){
         packet_dimension_ref packet_dimension;
 
         packet_dimension = packet_dimensions_find(packet_dimensions, device_id);
-        if (!packet_dimension)
+        if(! packet_dimension){
                 return ENOENT;
+        }
         packet_dimension->content = content;
-
-        if (device_id != DEVICE_INVALID_ID) {
-                packet_dimension = packet_dimensions_find(packet_dimensions,
-                    DEVICE_INVALID_ID);
-
-                if (packet_dimension) {
-                        if (packet_dimension->content >= content)
+        if(device_id != DEVICE_INVALID_ID){
+                packet_dimension = packet_dimensions_find(packet_dimensions, DEVICE_INVALID_ID);
+                if(packet_dimension){
+                        if(packet_dimension->content >= content){
                                 packet_dimension->content = content;
-                        else
-                                packet_dimensions_exclude(packet_dimensions,
-                                    DEVICE_INVALID_ID);
-
+                        }else{
+                                packet_dimensions_exclude(packet_dimensions, DEVICE_INVALID_ID);
+                        }
                 }
         }
-
         return EOK;
 }
 
-int tl_set_address_port(struct sockaddr * addr, int addrlen, uint16_t port)
-{
-        struct sockaddr_in *address_in;
-        struct sockaddr_in6 *address_in6;
+int tl_set_address_port(struct sockaddr * addr, int addrlen, uint16_t port){
+        struct sockaddr_in * address_in;
+        struct sockaddr_in6 * address_in6;
         size_t length;
 
-        if (addrlen < 0)
-                return EINVAL;
-        
+        if(addrlen < 0){
+                return EINVAL;
+        }
         length = (size_t) addrlen;
-        if (length < sizeof(struct sockaddr))
-                return EINVAL;
-
-        switch (addr->sa_family) {
-        case AF_INET:
-                if (length != sizeof(struct sockaddr_in))
-                        return EINVAL;
-                address_in = (struct sockaddr_in *) addr;
-                address_in->sin_port = htons(port);
-                return EOK;
-        case AF_INET6:
-                if (length != sizeof(struct sockaddr_in6))
-                                return EINVAL;
-                address_in6 = (struct sockaddr_in6 *) addr;
-                address_in6->sin6_port = htons(port);
-                return EOK;
-        default:
-                return EAFNOSUPPORT;
-        }
-}
-
-int
-tl_prepare_icmp_packet(int packet_phone, int icmp_phone, packet_t packet,
-    services_t error)
-{
+        if(length < sizeof(struct sockaddr)){
+                return EINVAL;
+        }
+        switch(addr->sa_family){
+                case AF_INET:
+                        if(length != sizeof(struct sockaddr_in)){
+                                return EINVAL;
+                        }
+                        address_in = (struct sockaddr_in *) addr;
+                        address_in->sin_port = htons(port);
+                        return EOK;
+                case AF_INET6:
+                        if(length != sizeof(struct sockaddr_in6)){
+                                return EINVAL;
+                        }
+                        address_in6 = (struct sockaddr_in6 *) addr;
+                        address_in6->sin6_port = htons(port);
+                        return EOK;
+                default:
+                        return EAFNOSUPPORT;
+        }
+}
+
+int tl_prepare_icmp_packet(int packet_phone, int icmp_phone, packet_t packet, services_t error){
         packet_t next;
-        uint8_t *src;
+        uint8_t * src;
         int length;
 
@@ -213 +200 @@
         
         length = packet_get_addr(packet, &src, NULL);
-        if ((length > 0) && (!error) && (icmp_phone >= 0) &&
-            // set both addresses to the source one (avoids the source address
-            // deletion before setting the destination one)
-            (packet_set_addr(packet, src, src, (size_t) length) == EOK)) {
+        if((length > 0)
+                && (! error)
+                && (icmp_phone >= 0)
+        // set both addresses to the source one (avoids the source address deletion before setting the destination one)
+                && (packet_set_addr(packet, src, src, (size_t) length) == EOK)){
                 return EOK;
-        } else
+        }else{
                 pq_release_remote(packet_phone, packet_get_id(packet));
-
+        }
         return ENOENT;
 }
 
-int
-tl_socket_read_packet_data(int packet_phone, packet_ref packet, size_t prefix,
-    const packet_dimension_ref dimension, const struct sockaddr *addr,
-    socklen_t addrlen)
-{
+int tl_socket_read_packet_data(int packet_phone, packet_ref packet, size_t prefix, const packet_dimension_ref dimension, const struct sockaddr * addr, socklen_t addrlen){
         ERROR_DECLARE;
 
@@ -235 +219 @@
         void * data;
 
-        if (!dimension)
-                return EINVAL;
-
+        if(! dimension){
+                return EINVAL;
+        }
         // get the data length
-        if (!async_data_write_receive(&callid, &length))
-                return EINVAL;
-
+        if(! async_data_write_receive(&callid, &length)){
+                return EINVAL;
+        }
         // get a new packet
-        *packet = packet_get_4_remote(packet_phone, length, dimension->addr_len,
-            prefix + dimension->prefix, dimension->suffix);
-        if (!packet)
+        *packet = packet_get_4_remote(packet_phone, length, dimension->addr_len, prefix + dimension->prefix, dimension->suffix);
+        if(! packet){
                 return ENOMEM;
-
+        }
         // allocate space in the packet
         data = packet_suffix(*packet, length);
-        if (!data) {
+        if(! data){
                 pq_release_remote(packet_phone, packet_get_id(*packet));
                 return ENOMEM;
         }
-
         // read the data into the packet
-        if (ERROR_OCCURRED(async_data_write_finalize(callid, data, length)) ||
-            // set the packet destination address
-            ERROR_OCCURRED(packet_set_addr(*packet, NULL, (uint8_t *) addr,
-            addrlen))) {
+        if(ERROR_OCCURRED(async_data_write_finalize(callid, data, length))
+        // set the packet destination address
+                || ERROR_OCCURRED(packet_set_addr(*packet, NULL, (uint8_t *) addr, addrlen))){
                 pq_release_remote(packet_phone, packet_get_id(*packet));
                 return ERROR_CODE;
         }
-
         return (int) length;
 }