OSPFv3: Add/remove some bounds checks

Remove two bounds checks now useless with GET_ macros uses.
The check on 'lshp->ls_router' is needed before ipaddr_string() call.

diff --git a/print-ospf6.c b/print-ospf6.c
index c734b4d20ebe264608591b14ec93e6b6bc426f72..cec67e1d91f9a2b69dcfc1605a5b4118f9405115 100644 (file)
--- a/print-ospf6.c
+++ b/print-ospf6.c
@@ -388,8 +388,7 @@ ospf6_print_lshdr(netdissect_options *ndo,
 {
        if ((const u_char *)(lshp + 1) > dataend)
                goto trunc;
 {
        if ((const u_char *)(lshp + 1) > dataend)
                goto trunc;

-       ND_TCHECK_2(lshp->ls_type);
-       ND_TCHECK_4(lshp->ls_seq);
+       ND_TCHECK_4(lshp->ls_router);

 
        ND_PRINT("\n\t  Advertising Router %s, seq 0x%08x, age %us, length %u",
                ipaddr_string(ndo, lshp->ls_router),
 
        ND_PRINT("\n\t  Advertising Router %s, seq 0x%08x, age %us, length %u",
                ipaddr_string(ndo, lshp->ls_router),