From f01dc8cc343da43c2c6fa65b7f10b2425280bade Mon Sep 17 00:00:00 2001 From: goldsimon Date: Thu, 23 Apr 2015 09:59:15 +0200 Subject: [PATCH] more "const" fixes --- src/core/ipv4/ip4_addr.c | 2 +- src/core/ipv6/ip6_addr.c | 8 ++++---- src/core/ipv6/mld6.c | 2 +- src/core/snmp/mib2.c | 2 +- src/include/lwip/ip_addr.h | 30 +++++++++++++++--------------- 5 files changed, 22 insertions(+), 22 deletions(-) diff --git a/src/core/ipv4/ip4_addr.c b/src/core/ipv4/ip4_addr.c index b69bf27e..5f4f5fa3 100644 --- a/src/core/ipv4/ip4_addr.c +++ b/src/core/ipv4/ip4_addr.c @@ -326,7 +326,7 @@ ip4_2_ip(const ip4_addr_t *ip4addr, ip_addr_t* storage) return NULL; } ip4_addr_copy(storage->addr.ip4, *ip4addr); - IP_SET_TYPE_L(storage, IPADDR_TYPE_V4); + IP_SET_TYPE_VAL(*storage, IPADDR_TYPE_V4); return storage; } #endif /* LWIP_IPV6 */ diff --git a/src/core/ipv6/ip6_addr.c b/src/core/ipv6/ip6_addr.c index 67715131..4fcee07c 100644 --- a/src/core/ipv6/ip6_addr.c +++ b/src/core/ipv6/ip6_addr.c @@ -260,7 +260,7 @@ ip6_2_ip(const ip6_addr_t *ip6addr, ip_addr_t* storage) return NULL; } ip6_addr_copy(storage->addr.ip6, *ip6addr); - IP_SET_TYPE_L(storage, IPADDR_TYPE_V6); + IP_SET_TYPE_VAL(*storage, IPADDR_TYPE_V6); return storage; } @@ -280,20 +280,20 @@ ipaddr_aton(const char *cp, ip_addr_t *addr) if (*c == '.') { /* contains a dot: IPv4 address */ if (addr) { - IP_SET_TYPE_L(addr, IPADDR_TYPE_V4); + IP_SET_TYPE_VAL(*addr, IPADDR_TYPE_V4); } return ip4addr_aton(cp, ip_2_ip4(addr)); } else if (*c == ':') { /* contains a colon: IPv6 address */ if (addr) { - IP_SET_TYPE_L(addr, IPADDR_TYPE_V6); + IP_SET_TYPE_VAL(*addr, IPADDR_TYPE_V6); } return ip6addr_aton(cp, ip_2_ip6(addr)); } } /* nothing found, call ip4addr_aton as fallback */ if (addr) { - IP_SET_TYPE_L(addr, IPADDR_TYPE_V4); + IP_SET_TYPE_VAL(*addr, IPADDR_TYPE_V4); } return ip4addr_aton(cp, ip_2_ip4(addr)); } diff --git a/src/core/ipv6/mld6.c b/src/core/ipv6/mld6.c index 697c010b..e7b7daea 100644 --- a/src/core/ipv6/mld6.c +++ b/src/core/ipv6/mld6.c @@ -527,7 +527,7 @@ mld6_send(struct mld_group *group, u8_t type) { struct mld_header * mld_hdr; struct pbuf * p; - ip6_addr_t * src_addr; + const ip6_addr_t * src_addr; /* Allocate a packet. Size is MLD header + IPv6 Hop-by-hop options header. */ p = pbuf_alloc(PBUF_IP, sizeof(struct mld_header) + sizeof(struct ip6_hbh_hdr), PBUF_RAM); diff --git a/src/core/snmp/mib2.c b/src/core/snmp/mib2.c index 9d184e06..d5919c39 100644 --- a/src/core/snmp/mib2.c +++ b/src/core/snmp/mib2.c @@ -3911,7 +3911,7 @@ udpentry_get_value(struct obj_def *od, u16_t len, void *value) LWIP_UNUSED_ARG(len); snmp_oidtoip(&od->id_inst_ptr[1], ip_2_ip4(&ip)); - IP_SET_TYPE(&ip, IPADDR_TYPE_V4); + IP_SET_TYPE_VAL(ip, IPADDR_TYPE_V4); LWIP_ASSERT("invalid port", (od->id_inst_ptr[5] >= 0) && (od->id_inst_ptr[5] <= 0xffff)); port = (u16_t)od->id_inst_ptr[5]; diff --git a/src/include/lwip/ip_addr.h b/src/include/lwip/ip_addr.h index 85871fee..823efc77 100644 --- a/src/include/lwip/ip_addr.h +++ b/src/include/lwip/ip_addr.h @@ -58,13 +58,13 @@ typedef struct _ip_addr { #define IPADDR_TYPE_V4 0U #define IPADDR_TYPE_V6 6U -#define IPADDR4_INIT(u32val) { { { u32val, 0ul, 0ul, 0ul } }, IPADDR_TYPE_V4 } -#define IPADDR6_INIT(a, b, c, d) { { { a, b, c, d } }, IPADDR_TYPE_V6 } +#define IPADDR4_INIT(u32val) { { { { u32val, 0ul, 0ul, 0ul } } }, IPADDR_TYPE_V4 } +#define IPADDR6_INIT(a, b, c, d) { { { { a, b, c, d } } }, IPADDR_TYPE_V6 } #define IP_IS_V6_VAL(ipaddr) ((ipaddr).type == IPADDR_TYPE_V6) #define IP_IS_V6(ipaddr) (((ipaddr) != NULL) && IP_IS_V6_VAL(*(ipaddr))) -#define IP_SET_TYPE_L(ipaddr, iptype) do { (ipaddr)->type = (iptype); }while(0) -#define IP_SET_TYPE(ipaddr, iptype) do { if((ipaddr) != NULL) { IP_SET_TYPE_L(ipaddr, iptype); }}while(0) +#define IP_SET_TYPE_VAL(ipaddr, iptype) do { (ipaddr).type = (iptype); }while(0) +#define IP_SET_TYPE(ipaddr, iptype) do { if((ipaddr) != NULL) { IP_SET_TYPE_VAL(*(ipaddr), iptype); }}while(0) #define IP_ADDR_PCB_VERSION_MATCH(pcb, ipaddr) (PCB_ISIPV6(pcb) == IP_IS_V6(ipaddr)) @@ -84,17 +84,17 @@ static ip4_addr_t* ip_2_ip4(const ip_addr_t *ipaddr) #endif /* LWIP_ALLOW_STATIC_FN_IN_HEADER */ #define IP_ADDR4(ipaddr,a,b,c,d) do { IP4_ADDR(ip_2_ip4(ipaddr),a,b,c,d); \ - IP_SET_TYPE_L(ipaddr, IPADDR_TYPE_V4); } while(0) + IP_SET_TYPE_VAL(*(ipaddr), IPADDR_TYPE_V4); } while(0) #define IP_ADDR6(ipaddr,idx,a,b,c,d) do { IP6_ADDR(ip_2_ip6(ipaddr),idx,a,b,c,d); \ - IP_SET_TYPE_L(ipaddr, IPADDR_TYPE_V6); } while(0) + IP_SET_TYPE_VAL(*(ipaddr), IPADDR_TYPE_V6); } while(0) #define ip_addr_copy(dest, src) do{if(IP_IS_V6_VAL(src)){ \ - ip6_addr_copy(*ip_2_ip6(&(dest)), *ip_2_ip6(&(src))); IP_SET_TYPE_L(&(dest), IPADDR_TYPE_V6); }else{ \ - ip4_addr_copy(*ip_2_ip4(&(dest)), *ip_2_ip4(&(src))); IP_SET_TYPE_L(&(dest), IPADDR_TYPE_V4); }}while(0) + ip6_addr_copy(*ip_2_ip6(&(dest)), *ip_2_ip6(&(src))); IP_SET_TYPE_VAL(dest, IPADDR_TYPE_V6); }else{ \ + ip4_addr_copy(*ip_2_ip4(&(dest)), *ip_2_ip4(&(src))); IP_SET_TYPE_VAL(dest, IPADDR_TYPE_V4); }}while(0) #define ip_addr_copy_from_ip6(dest, src) do{ \ - ip6_addr_copy(*ip_2_ip6(&(dest)), src); IP_SET_TYPE_L(&(dest), IPADDR_TYPE_V6); }while(0) + ip6_addr_copy(*ip_2_ip6(&(dest)), src); IP_SET_TYPE_VAL(dest, IPADDR_TYPE_V6); }while(0) #define ip_addr_copy_from_ip4(dest, src) do{ \ - ip4_addr_copy(*ip_2_ip4(&(dest)), src); IP_SET_TYPE_L(&(dest), IPADDR_TYPE_V4); }while(0) + ip4_addr_copy(*ip_2_ip4(&(dest)), src); IP_SET_TYPE_VAL(dest, IPADDR_TYPE_V4); }while(0) #define ip_addr_set_ip4_u32(ipaddr, val) do{if(ipaddr){ip4_addr_set_u32(ip_2_ip4(ipaddr), val); \ IP_SET_TYPE(ipaddr, IPADDR_TYPE_V4); }}while(0) #define ip_addr_get_ip4_u32(ipaddr) (((ipaddr) && !IP_IS_V6(ipaddr)) ? \ @@ -122,7 +122,7 @@ static ip4_addr_t* ip_2_ip4(const ip_addr_t *ipaddr) #define ip_addr_netcmp(addr1, addr2, mask) ((IP_IS_V6(addr1) && IP_IS_V6(addr2)) ? \ 0 : \ ip4_addr_netcmp(ip_2_ip4(addr1), ip_2_ip4(addr2), mask)) -#define ip_addr_cmp(addr1, addr2) ((IP_IS_V6(addr1) != IP_IS_V6(addr2)) ? 0 : (IP_IS_V6(addr1) ? \ +#define ip_addr_cmp(addr1, addr2) ((IP_IS_V6_VAL(*(addr1)) != IP_IS_V6_VAL(*(addr2))) ? 0 : (IP_IS_V6_VAL(*(addr1)) ? \ ip6_addr_cmp(ip_2_ip6(addr1), ip_2_ip6(addr2)) : \ ip4_addr_cmp(ip_2_ip4(addr1), ip_2_ip4(addr2)))) #define ip_addr_isany(ipaddr) ((IP_IS_V6(ipaddr)) ? \ @@ -165,7 +165,7 @@ typedef ip4_addr_t ip_addr_t; #define IPADDR4_INIT(u32val) { u32val } #define IP_IS_V6_VAL(ipaddr) 0 #define IP_IS_V6(ipaddr) 0 -#define IP_SET_TYPE_L(ipaddr, iptype) +#define IP_SET_TYPE_VAL(ipaddr, iptype) #define IP_SET_TYPE(ipaddr, iptype) #define ip4_2_ip(ipaddr, unused) (ipaddr) #define ip_2_ip4(ipaddr) (ipaddr) @@ -202,7 +202,7 @@ typedef ip6_addr_t ip_addr_t; #define IPADDR6_INIT(a, b, c, d) { a, b, c, d } #define IP_IS_V6_VAL(ipaddr) 1 #define IP_IS_V6(ipaddr) 1 -#define IP_SET_TYPE_L(ipaddr, iptype) +#define IP_SET_TYPE_VAL(ipaddr, iptype) #define IP_SET_TYPE(ipaddr, iptype) #define ip6_2_ip(ipaddr, unused) (ipaddr) #define ip_2_ip6(ipaddr) (ipaddr) @@ -259,11 +259,11 @@ extern const ip_addr_t ip6_addr_any; /** IP6_ADDR_ANY can be used as a fixed ip_addr_t * for the IPv6 wildcard address */ -#define IP6_ADDR_ANY ((ip_addr_t*)&ip6_addr_any) +#define IP6_ADDR_ANY (&ip6_addr_any) /** IP6_ADDR_ANY6 can be used as a fixed ip6_addr_t * for the IPv6 wildcard address */ -#define IP6_ADDR_ANY6 ((ip6_addr_t*)ip_2_ip6(&ip6_addr_any)) +#define IP6_ADDR_ANY6 (ip_2_ip6(&ip6_addr_any)) #if !LWIP_IPV4 /** Just a little upgrade-helper for IPv6-only configurations: */