ospf6d: replace OSPF6_LSA_HEADER_END macro

Replacing the macro with an inline function allows the compiler to
check the parameter type.

Use the replacement function consistently to reduce the number of
open coded pointer cast plus offset calculations.

use tools/indent.py to reformat all occurences of its use.

Signed-off-by: Andrew Cooks <acooks.at.bda@gmail.com>

diff --git a/ospf6d/ospf6_abr.c b/ospf6d/ospf6_abr.c
index d3ff759d33ed45e468cdeee6e59143c914afe5a7..aec947a6988b294c46205b1d982016a7fb953e7b 100644 (file)
--- a/ospf6d/ospf6_abr.c
+++ b/ospf6d/ospf6_abr.c
@@ -553,9 +553,8 @@ int ospf6_abr_originate_summary_to_area(struct ospf6_route *route,
        lsa_header = (struct ospf6_lsa_header *)buffer;
 
        if (route->type == OSPF6_DEST_TYPE_ROUTER) {
-               router_lsa = (struct ospf6_inter_router_lsa
-                                     *)((caddr_t)lsa_header
-                                        + sizeof(struct ospf6_lsa_header));
+               router_lsa = (struct ospf6_inter_router_lsa *)
+                       ospf6_lsa_header_end(lsa_header);
                p = (caddr_t)router_lsa + sizeof(struct ospf6_inter_router_lsa);
 
                /* Fill Inter-Area-Router-LSA */
@@ -566,9 +565,8 @@ int ospf6_abr_originate_summary_to_area(struct ospf6_route *route,
                router_lsa->router_id = ADV_ROUTER_IN_PREFIX(&route->prefix);
                type = htons(OSPF6_LSTYPE_INTER_ROUTER);
        } else {
-               prefix_lsa = (struct ospf6_inter_prefix_lsa
-                                     *)((caddr_t)lsa_header
-                                        + sizeof(struct ospf6_lsa_header));
+               prefix_lsa = (struct ospf6_inter_prefix_lsa *)
+                       ospf6_lsa_header_end(lsa_header);
                p = (caddr_t)prefix_lsa + sizeof(struct ospf6_inter_prefix_lsa);
 
                /* Fill Inter-Area-Prefix-LSA */
@@ -1018,9 +1016,8 @@ void ospf6_abr_examin_summary(struct ospf6_lsa *lsa, struct ospf6_area *oa)
                                   oa->name);
                }
 
-               prefix_lsa =
-                       (struct ospf6_inter_prefix_lsa *)OSPF6_LSA_HEADER_END(
-                               lsa->header);
+               prefix_lsa = (struct ospf6_inter_prefix_lsa *)
+                       ospf6_lsa_header_end(lsa->header);
                prefix.family = AF_INET6;
                prefix.prefixlen = prefix_lsa->prefix.prefix_length;
                ospf6_prefix_in6_addr(&prefix.u.prefix6, prefix_lsa,
@@ -1039,11 +1036,9 @@ void ospf6_abr_examin_summary(struct ospf6_lsa *lsa, struct ospf6_area *oa)
                                   oa->name);
                }
 
-               router_lsa =
-                       (struct ospf6_inter_router_lsa *)OSPF6_LSA_HEADER_END(
-                               lsa->header);
-               ospf6_linkstate_prefix(router_lsa->router_id, htonl(0),
-                                      &prefix);
+               router_lsa = (struct ospf6_inter_router_lsa *)
+                       ospf6_lsa_header_end(lsa->header);
+               ospf6_linkstate_prefix(router_lsa->router_id, htonl(0), &prefix);
                if (is_debug)
                        inet_ntop(AF_INET, &router_lsa->router_id, buf,
                                  sizeof(buf));
@@ -1433,9 +1428,8 @@ static char *ospf6_inter_area_prefix_lsa_get_prefix_str(struct ospf6_lsa *lsa,
        char tbuf[16];
 
        if (lsa != NULL) {
-               prefix_lsa =
-                       (struct ospf6_inter_prefix_lsa *)OSPF6_LSA_HEADER_END(
-                               lsa->header);
+               prefix_lsa = (struct ospf6_inter_prefix_lsa *)
+                       ospf6_lsa_header_end(lsa->header);
 
                ospf6_prefix_in6_addr(&in6, prefix_lsa, &prefix_lsa->prefix);
                if (buf) {
@@ -1457,7 +1451,7 @@ static int ospf6_inter_area_prefix_lsa_show(struct vty *vty,
        struct ospf6_inter_prefix_lsa *prefix_lsa;
        char buf[INET6_ADDRSTRLEN];
 
-       prefix_lsa = (struct ospf6_inter_prefix_lsa *)OSPF6_LSA_HEADER_END(
+       prefix_lsa = (struct ospf6_inter_prefix_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        if (use_json) {
@@ -1494,9 +1488,8 @@ static char *ospf6_inter_area_router_lsa_get_prefix_str(struct ospf6_lsa *lsa,
        struct ospf6_inter_router_lsa *router_lsa;
 
        if (lsa != NULL) {
-               router_lsa =
-                       (struct ospf6_inter_router_lsa *)OSPF6_LSA_HEADER_END(
-                               lsa->header);
+               router_lsa = (struct ospf6_inter_router_lsa *)
+                       ospf6_lsa_header_end(lsa->header);
 
 
                if (buf)
@@ -1514,7 +1507,7 @@ static int ospf6_inter_area_router_lsa_show(struct vty *vty,
        struct ospf6_inter_router_lsa *router_lsa;
        char buf[64];
 
-       router_lsa = (struct ospf6_inter_router_lsa *)OSPF6_LSA_HEADER_END(
+       router_lsa = (struct ospf6_inter_router_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        ospf6_options_printbuf(router_lsa->options, buf, sizeof(buf));

diff --git a/ospf6d/ospf6_asbr.c b/ospf6d/ospf6_asbr.c
index 701704cdc80db722ad46bcd99863bf8ef1be2036..2065527c9377bb3b5214f8584a3f08e55e08d650 100644 (file)
--- a/ospf6d/ospf6_asbr.c
+++ b/ospf6d/ospf6_asbr.c
@@ -102,9 +102,8 @@ struct ospf6_lsa *ospf6_as_external_lsa_originate(struct ospf6_route *route,
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       as_external_lsa = (struct ospf6_as_external_lsa
-                                  *)((caddr_t)lsa_header
-                                     + sizeof(struct ospf6_lsa_header));
+       as_external_lsa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+               lsa_header);
        p = (caddr_t)((caddr_t)as_external_lsa
                      + sizeof(struct ospf6_as_external_lsa));
 
@@ -217,7 +216,7 @@ static route_tag_t ospf6_as_external_lsa_get_tag(struct ospf6_lsa *lsa)
        if (!lsa)
                return 0;
 
-       external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        if (!CHECK_FLAG(external->bits_metric, OSPF6_ASBR_BIT_T))
@@ -521,7 +520,7 @@ void ospf6_asbr_lsa_add(struct ospf6_lsa *lsa)
        type = ntohs(lsa->header->type);
        oa = lsa->lsdb->data;
 
-       external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        if (IS_OSPF6_DEBUG_EXAMIN(AS_EXTERNAL))
@@ -726,7 +725,7 @@ void ospf6_asbr_lsa_remove(struct ospf6_lsa *lsa,
        int type;
        bool debug = false;
 
-       external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        if (IS_OSPF6_DEBUG_EXAMIN(AS_EXTERNAL) || (IS_OSPF6_DEBUG_NSSA))
@@ -2425,7 +2424,7 @@ static char *ospf6_as_external_lsa_get_prefix_str(struct ospf6_lsa *lsa,
        char tbuf[16];
 
        if (lsa) {
-               external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+               external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                        lsa->header);
 
                if (pos == 0) {
@@ -2460,7 +2459,7 @@ static int ospf6_as_external_lsa_show(struct vty *vty, struct ospf6_lsa *lsa,
        char buf[64];
 
        assert(lsa->header);
-       external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        /* bits */
@@ -3028,8 +3027,8 @@ ospf6_originate_summary_lsa(struct ospf6 *ospf6,
                        return;
                }
 
-               external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END
-                                       (aggr_lsa->header);
+               external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+                       aggr_lsa->header);
                metric = (unsigned long)OSPF6_ASBR_METRIC(external);
                tag = ospf6_as_external_lsa_get_tag(aggr_lsa);
                mtype = CHECK_FLAG(external->bits_metric,
@@ -3177,8 +3176,7 @@ ospf6_handle_external_aggr_modify(struct ospf6 *ospf6,
                return OSPF6_FAILURE;
        }
 
-       asel = (struct ospf6_as_external_lsa *)
-               OSPF6_LSA_HEADER_END(lsa->header);
+       asel = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(lsa->header);
        metric = (unsigned long)OSPF6_ASBR_METRIC(asel);
        tag = ospf6_as_external_lsa_get_tag(lsa);
        mtype = CHECK_FLAG(asel->bits_metric,
@@ -3367,9 +3365,8 @@ static void ospf6_handle_aggregated_exnl_rt(struct ospf6 *ospf6,
        lsa = ospf6_lsdb_lookup(htons(OSPF6_LSTYPE_AS_EXTERNAL),
                                htonl(info->id), ospf6->router_id, ospf6->lsdb);
        if (lsa) {
-               ext_lsa = (struct ospf6_as_external_lsa
-                       *)((char *)(lsa->header)
-                       + sizeof(struct ospf6_lsa_header));
+               ext_lsa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+                       lsa->header);
 
                if (rt->prefix.prefixlen != ext_lsa->prefix.prefix_length)
                        return;

diff --git a/ospf6d/ospf6_gr.c b/ospf6d/ospf6_gr.c
index 34cb9706bd387771612b7e449e1aa33a856bb10b..ab119a4ea4eb5e0edd35932c2e16c4789d563167 100644 (file)
--- a/ospf6d/ospf6_gr.c
+++ b/ospf6d/ospf6_gr.c
@@ -54,9 +54,7 @@ static int ospf6_gr_lsa_originate(struct ospf6_interface *oi,
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       grace_lsa =
-               (struct ospf6_grace_lsa *)((caddr_t)lsa_header
-                                          + sizeof(struct ospf6_lsa_header));
+       grace_lsa = (struct ospf6_grace_lsa *)ospf6_lsa_header_end(lsa_header);
 
        /* Put grace period. */
        grace_lsa->tlv_period.header.type = htons(GRACE_PERIOD_TYPE);

diff --git a/ospf6d/ospf6_intra.c b/ospf6d/ospf6_intra.c
index cb036752e8a38b10085bb05cf8e899f0c4b70c8b..48a7012adac796654cc8d239f931918263e3da2d 100644 (file)
--- a/ospf6d/ospf6_intra.c
+++ b/ospf6d/ospf6_intra.c
@@ -242,9 +242,7 @@ void ospf6_router_lsa_originate(struct event *thread)
 
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       router_lsa =
-               (struct ospf6_router_lsa *)((caddr_t)lsa_header
-                                           + sizeof(struct ospf6_lsa_header));
+       router_lsa = (struct ospf6_router_lsa *)ospf6_lsa_header_end(lsa_header);
 
        ospf6_router_lsa_options_set(oa, router_lsa);
 
@@ -305,10 +303,8 @@ void ospf6_router_lsa_originate(struct event *thread)
                        /* Reset Buffer to fill next Router LSA */
                        memset(buffer, 0, sizeof(buffer));
                        lsa_header = (struct ospf6_lsa_header *)buffer;
-                       router_lsa =
-                               (struct ospf6_router_lsa
-                                        *)((caddr_t)lsa_header
-                                           + sizeof(struct ospf6_lsa_header));
+                       router_lsa = (struct ospf6_router_lsa *)
+                               ospf6_lsa_header_end(lsa_header);
 
                        ospf6_router_lsa_options_set(oa, router_lsa);
 
@@ -568,16 +564,14 @@ void ospf6_network_lsa_originate(struct event *thread)
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
        network_lsa =
-               (struct ospf6_network_lsa *)((caddr_t)lsa_header
-                                            + sizeof(struct ospf6_lsa_header));
+               (struct ospf6_network_lsa *)ospf6_lsa_header_end(lsa_header);
 
        /* Collect the interface's Link-LSAs to describe
           network's optional capabilities */
        type = htons(OSPF6_LSTYPE_LINK);
        for (ALL_LSDB_TYPED(oi->lsdb, type, lsa)) {
-               link_lsa = (struct ospf6_link_lsa
-                                   *)((caddr_t)lsa->header
-                                      + sizeof(struct ospf6_lsa_header));
+               link_lsa = (struct ospf6_link_lsa *)ospf6_lsa_header_end(
+                       lsa->header);
                network_lsa->options[0] |= link_lsa->options[0];
                network_lsa->options[1] |= link_lsa->options[1];
                network_lsa->options[2] |= link_lsa->options[2];
@@ -800,8 +794,7 @@ void ospf6_link_lsa_originate(struct event *thread)
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       link_lsa = (struct ospf6_link_lsa *)((caddr_t)lsa_header
-                                            + sizeof(struct ospf6_lsa_header));
+       link_lsa = (struct ospf6_link_lsa *)ospf6_lsa_header_end(lsa_header);
 
        /* Fill Link-LSA */
        link_lsa->priority = oi->priority;
@@ -1042,9 +1035,8 @@ void ospf6_intra_prefix_lsa_originate_stub(struct event *thread)
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa
-                                   *)((caddr_t)lsa_header
-                                      + sizeof(struct ospf6_lsa_header));
+       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa *)ospf6_lsa_header_end(
+               lsa_header);
 
        /* Fill Intra-Area-Prefix-LSA */
        intra_prefix_lsa->ref_type = htons(OSPF6_LSTYPE_ROUTER);
@@ -1159,10 +1151,8 @@ void ospf6_intra_prefix_lsa_originate_stub(struct event *thread)
                        /* Prepare next buffer */
                        memset(buffer, 0, sizeof(buffer));
                        lsa_header = (struct ospf6_lsa_header *)buffer;
-                       intra_prefix_lsa =
-                               (struct ospf6_intra_prefix_lsa
-                                        *)((caddr_t)lsa_header
-                                           + sizeof(struct ospf6_lsa_header));
+                       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa *)
+                               ospf6_lsa_header_end(lsa_header);
 
                        /* Fill Intra-Area-Prefix-LSA */
                        intra_prefix_lsa->ref_type = htons(OSPF6_LSTYPE_ROUTER);
@@ -1269,9 +1259,8 @@ void ospf6_intra_prefix_lsa_originate_transit(struct event *thread)
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa
-                                   *)((caddr_t)lsa_header
-                                      + sizeof(struct ospf6_lsa_header));
+       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa *)ospf6_lsa_header_end(
+               lsa_header);
 
        /* Fill Intra-Area-Prefix-LSA */
        intra_prefix_lsa->ref_type = htons(OSPF6_LSTYPE_NETWORK);
@@ -1668,7 +1657,8 @@ void ospf6_intra_prefix_route_ecmp_path(struct ospf6_area *oa,
                                }
                                intra_prefix_lsa =
                                        (struct ospf6_intra_prefix_lsa *)
-                                       OSPF6_LSA_HEADER_END(lsa->header);
+                                               ospf6_lsa_header_end(
+                                                       lsa->header);
 
                                if (intra_prefix_lsa->ref_adv_router
                                     == oa->ospf6->router_id) {
@@ -1749,9 +1739,8 @@ void ospf6_intra_prefix_lsa_add(struct ospf6_lsa *lsa)
 
        oa = OSPF6_AREA(lsa->lsdb->data);
 
-       intra_prefix_lsa =
-               (struct ospf6_intra_prefix_lsa *)OSPF6_LSA_HEADER_END(
-                       lsa->header);
+       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa *)ospf6_lsa_header_end(
+               lsa->header);
        if (intra_prefix_lsa->ref_type == htons(OSPF6_LSTYPE_ROUTER) ||
            intra_prefix_lsa->ref_type == htons(OSPF6_LSTYPE_NETWORK))
                ospf6_linkstate_prefix(intra_prefix_lsa->ref_adv_router,
@@ -1979,9 +1968,8 @@ void ospf6_intra_prefix_lsa_remove(struct ospf6_lsa *lsa)
 
        oa = OSPF6_AREA(lsa->lsdb->data);
 
-       intra_prefix_lsa =
-               (struct ospf6_intra_prefix_lsa *)OSPF6_LSA_HEADER_END(
-                       lsa->header);
+       intra_prefix_lsa = (struct ospf6_intra_prefix_lsa *)ospf6_lsa_header_end(
+               lsa->header);
 
        prefix_num = ntohs(intra_prefix_lsa->prefix_num);
        start = (caddr_t)intra_prefix_lsa

diff --git a/ospf6d/ospf6_lsa.c b/ospf6d/ospf6_lsa.c
index bc39579653c0e262a29f8e09ce9beeb35cb0c306..24643164c5fd6004e8a2d71dc883368679621c39 100644 (file)
--- a/ospf6d/ospf6_lsa.c
+++ b/ospf6d/ospf6_lsa.c
@@ -63,10 +63,10 @@ struct ospf6 *ospf6_get_by_lsdb(struct ospf6_lsa *lsa)
 static int ospf6_unknown_lsa_show(struct vty *vty, struct ospf6_lsa *lsa,
                                  json_object *json_obj, bool use_json)
 {
-       uint8_t *start, *end, *current;
+       char *start, *end, *current;
 
-       start = (uint8_t *)lsa->header + sizeof(struct ospf6_lsa_header);
-       end = (uint8_t *)lsa->header + ntohs(lsa->header->length);
+       start = ospf6_lsa_header_end(lsa->header);
+       end = (char *)lsa->header + ntohs(lsa->header->length);
 
        if (use_json) {
                json_object_string_add(json_obj, "lsaType", "unknown");
@@ -234,8 +234,8 @@ int ospf6_lsa_is_changed(struct ospf6_lsa *lsa1, struct ospf6_lsa *lsa2)
        if (length <= 0)
                return 0;
 
-       return memcmp(OSPF6_LSA_HEADER_END(lsa1->header),
-                     OSPF6_LSA_HEADER_END(lsa2->header), length);
+       return memcmp(ospf6_lsa_header_end(lsa1->header),
+                     ospf6_lsa_header_end(lsa2->header), length);
 }
 
 /* ospf6 age functions */

diff --git a/ospf6d/ospf6_lsa.h b/ospf6d/ospf6_lsa.h
index c9ac27df88e9fa298d59fc845b58006dddb8df19..0fa632449f59dec6f18b890cb422a20919548c22 100644 (file)
--- a/ospf6d/ospf6_lsa.h
+++ b/ospf6d/ospf6_lsa.h
@@ -87,10 +87,15 @@ struct ospf6_lsa_header {
        uint16_t length;     /* LSA length */
 };
 
-#define OSPF6_LSA_HEADER_END(h) ((caddr_t)(h) + sizeof(struct ospf6_lsa_header))
 #define OSPF6_LSA_SIZE(h) (ntohs(((struct ospf6_lsa_header *)(h))->length))
 #define OSPF6_LSA_END(h)                                                       \
        ((caddr_t)(h) + ntohs(((struct ospf6_lsa_header *)(h))->length))
+
+static inline char *ospf6_lsa_header_end(struct ospf6_lsa_header *header)
+{
+       return (char *)header + sizeof(struct ospf6_lsa_header);
+}
+
 #define OSPF6_LSA_IS_TYPE(t, L)                                                \
        ((L)->header->type == htons(OSPF6_LSTYPE_##t) ? 1 : 0)
 #define OSPF6_LSA_IS_SAME(L1, L2)                                              \

diff --git a/ospf6d/ospf6_lsdb.c b/ospf6d/ospf6_lsdb.c
index c9cbdf8e928bf0110bc71f3024c3d9f0fe96a721..9aca5550a671068dc8f7f574d85960e0143d9297 100644 (file)
--- a/ospf6d/ospf6_lsdb.c
+++ b/ospf6d/ospf6_lsdb.c
@@ -229,9 +229,8 @@ struct ospf6_lsa *ospf6_find_inter_prefix_lsa(struct ospf6 *ospf6,
                struct ospf6_inter_prefix_lsa *prefix_lsa;
                struct prefix prefix;
 
-               prefix_lsa =
-                       (struct ospf6_inter_prefix_lsa *)OSPF6_LSA_HEADER_END(
-                               lsa->header);
+               prefix_lsa = (struct ospf6_inter_prefix_lsa *)
+                       ospf6_lsa_header_end(lsa->header);
                prefix.family = AF_INET6;
                prefix.prefixlen = prefix_lsa->prefix.prefix_length;
                ospf6_prefix_in6_addr(&prefix.u.prefix6, prefix_lsa,

diff --git a/ospf6d/ospf6_nssa.c b/ospf6d/ospf6_nssa.c
index 405ae9052893f15386427a62ec92e2966888e2cc..ea2be20cf355ac35d11633e9c01535e3eafb696f 100644 (file)
--- a/ospf6d/ospf6_nssa.c
+++ b/ospf6d/ospf6_nssa.c
@@ -52,10 +52,8 @@ static int ospf6_abr_nssa_am_elected(struct ospf6_area *oa)
 
        /* Verify all the router LSA to compare the router ID */
        for (ALL_LSDB_TYPED(oa->lsdb, type, lsa)) {
-
-               router_lsa = (struct ospf6_router_lsa
-                                     *)((caddr_t)lsa->header
-                                        + sizeof(struct ospf6_lsa_header));
+               router_lsa = (struct ospf6_router_lsa *)ospf6_lsa_header_end(
+                       lsa->header);
 
                /* ignore non-ABR routers */
                if (!CHECK_FLAG(router_lsa->bits, OSPF6_ROUTER_BIT_B))
@@ -416,7 +414,7 @@ static struct ospf6_lsa *ospf6_lsa_translated_nssa_new(struct ospf6_area *area,
        }
 
        /* find the translated Type-5 for this Type-7 */
-       nssa = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       nssa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                type7->header);
        prefix.family = AF_INET6;
        prefix.prefixlen = nssa->prefix.prefix_length;
@@ -437,12 +435,10 @@ static struct ospf6_lsa *ospf6_lsa_translated_nssa_new(struct ospf6_area *area,
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       extnew = (struct ospf6_as_external_lsa
-                         *)((caddr_t)lsa_header
-                            + sizeof(struct ospf6_lsa_header));
-       ext = (struct ospf6_as_external_lsa
-                      *)((caddr_t)(type7->header)
-                         + sizeof(struct ospf6_lsa_header));
+       extnew = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+               lsa_header);
+       ext = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+               type7->header);
        old_ptr =
                (caddr_t)((caddr_t)ext + sizeof(struct ospf6_as_external_lsa));
        new_ptr = (caddr_t)((caddr_t)extnew
@@ -550,7 +546,7 @@ struct ospf6_lsa *ospf6_translated_nssa_refresh(struct ospf6_area *area,
                                "%s: try to find translated Type-5 LSA for %s",
                                __func__, type7->name);
 
-               ext_lsa = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+               ext_lsa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                        type7->header);
                prefix.family = AF_INET6;
                prefix.prefixlen = ext_lsa->prefix.prefix_length;
@@ -618,7 +614,7 @@ static void ospf6_abr_translate_nssa(struct ospf6_area *area,
        struct ospf6 *ospf6;
 
        ospf6 = area->ospf6;
-       nssa_lsa = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       nssa_lsa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
 
        if (!CHECK_FLAG(nssa_lsa->prefix.prefix_options,
@@ -1244,9 +1240,8 @@ void ospf6_nssa_lsa_originate(struct ospf6_route *route,
        /* prepare buffer */
        memset(buffer, 0, sizeof(buffer));
        lsa_header = (struct ospf6_lsa_header *)buffer;
-       as_external_lsa = (struct ospf6_as_external_lsa
-                                  *)((caddr_t)lsa_header
-                                     + sizeof(struct ospf6_lsa_header));
+       as_external_lsa = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
+               lsa_header);
        p = (caddr_t)((caddr_t)as_external_lsa
                      + sizeof(struct ospf6_as_external_lsa));
 

diff --git a/ospf6d/ospf6_spf.c b/ospf6d/ospf6_spf.c
index e39ae504a23853e907988347019fcf5a178a1875..93120df99915b2ea6674f66e0832bfa0ee9a24aa 100644 (file)
--- a/ospf6d/ospf6_spf.c
+++ b/ospf6d/ospf6_spf.c
@@ -130,10 +130,10 @@ static struct ospf6_vertex *ospf6_vertex_create(struct ospf6_lsa *lsa)
        v->lsa = lsa;
 
        /* capability bits + options */
-       v->capability = *(uint8_t *)(OSPF6_LSA_HEADER_END(lsa->header));
-       v->options[0] = *(uint8_t *)(OSPF6_LSA_HEADER_END(lsa->header) + 1);
-       v->options[1] = *(uint8_t *)(OSPF6_LSA_HEADER_END(lsa->header) + 2);
-       v->options[2] = *(uint8_t *)(OSPF6_LSA_HEADER_END(lsa->header) + 3);
+       v->capability = *(uint8_t *)(ospf6_lsa_header_end(lsa->header));
+       v->options[0] = *(uint8_t *)(ospf6_lsa_header_end(lsa->header) + 1);
+       v->options[1] = *(uint8_t *)(ospf6_lsa_header_end(lsa->header) + 2);
+       v->options[2] = *(uint8_t *)(ospf6_lsa_header_end(lsa->header) + 3);
 
        v->nh_list = list_new();
        v->nh_list->cmp = (int (*)(void *, void *))ospf6_nexthop_cmp;
@@ -206,7 +206,7 @@ static char *ospf6_lsdesc_backlink(struct ospf6_lsa *lsa, caddr_t lsdesc,
        size = (OSPF6_LSA_IS_TYPE(ROUTER, lsa)
                        ? sizeof(struct ospf6_router_lsdesc)
                        : sizeof(struct ospf6_network_lsdesc));
-       for (backlink = OSPF6_LSA_HEADER_END(lsa->header) + 4;
+       for (backlink = ospf6_lsa_header_end(lsa->header) + 4;
             backlink + size <= OSPF6_LSA_END(lsa->header); backlink += size) {
                assert(!(OSPF6_LSA_IS_TYPE(NETWORK, lsa)
                         && VERTEX_IS_TYPE(NETWORK, v)));
@@ -290,7 +290,7 @@ static void ospf6_nexthop_calc(struct ospf6_vertex *w, struct ospf6_vertex *v,
                               != lsa->header->id)
                        continue;
 
-               link_lsa = (struct ospf6_link_lsa *)OSPF6_LSA_HEADER_END(
+               link_lsa = (struct ospf6_link_lsa *)ospf6_lsa_header_end(
                        lsa->header);
                if (IS_OSPF6_DEBUG_SPF(PROCESS)) {
                        inet_ntop(AF_INET6, &link_lsa->linklocal_addr, buf,
@@ -510,7 +510,7 @@ void ospf6_spf_calculation(uint32_t router_id,
                size = (VERTEX_IS_TYPE(ROUTER, v)
                                ? sizeof(struct ospf6_router_lsdesc)
                                : sizeof(struct ospf6_network_lsdesc));
-               for (lsdesc = OSPF6_LSA_HEADER_END(v->lsa->header) + 4;
+               for (lsdesc = ospf6_lsa_header_end(v->lsa->header) + 4;
                     lsdesc + size <= OSPF6_LSA_END(v->lsa->header);
                     lsdesc += size) {
                        lsa = ospf6_lsdesc_lsa(lsdesc, v);
@@ -1063,7 +1063,7 @@ struct ospf6_lsa *ospf6_create_single_router_lsa(struct ospf6_area *area,
                }
 
                if (IS_OSPF6_DEBUG_SPF(PROCESS)) {
-                       lsd = OSPF6_LSA_HEADER_END(rtr_lsa->header) + 4;
+                       lsd = ospf6_lsa_header_end(rtr_lsa->header) + 4;
                        interface_id = ROUTER_LSDESC_GET_IFID(lsd);
                        inet_ntop(AF_INET, &interface_id, ifbuf, sizeof(ifbuf));
                        zlog_debug(
@@ -1074,7 +1074,7 @@ struct ospf6_lsa *ospf6_create_single_router_lsa(struct ospf6_area *area,
 
                /* Append Next Link State ID LSA */
                lsa_header = rtr_lsa->header;
-               memcpy(new_header, (OSPF6_LSA_HEADER_END(rtr_lsa->header) + 4),
+               memcpy(new_header, (ospf6_lsa_header_end(rtr_lsa->header) + 4),
                       (ntohs(lsa_header->length) - lsa_length));
                new_header += (ntohs(lsa_header->length) - lsa_length);
                num_lsa--;
@@ -1137,7 +1137,7 @@ int ospf6_ase_calculate_route(struct ospf6 *ospf6, struct ospf6_lsa *lsa,
                return 0;
        }
 
-       external = (struct ospf6_as_external_lsa *)OSPF6_LSA_HEADER_END(
+       external = (struct ospf6_as_external_lsa *)ospf6_lsa_header_end(
                lsa->header);
        prefix.family = AF_INET6;
        prefix.prefixlen = external->prefix.prefix_length;