diff options
author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-06-13 19:10:32 +0200 |
---|---|---|
committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-06-13 19:10:32 +0200 |
commit | e6fda497d36365ef9b5adcbb9f59c397d0c7f5c5 (patch) | |
tree | b07d9bd58b55ef5b9268d75e3127827573181be0 /ospfd | |
parent | Merge branch 'stable/3.0' (diff) | |
download | frr-e6fda497d36365ef9b5adcbb9f59c397d0c7f5c5.tar.xz frr-e6fda497d36365ef9b5adcbb9f59c397d0c7f5c5.zip |
*: Clean up call into inet_aton
In the few places where we were not checking the return code
for inet_aton, do so.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'ospfd')
-rw-r--r-- | ospfd/ospf_ri.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/ospfd/ospf_ri.c b/ospfd/ospf_ri.c index 4723f4a91..bd2506d2d 100644 --- a/ospfd/ospf_ri.c +++ b/ospfd/ospf_ri.c @@ -1192,7 +1192,11 @@ DEFUN (router_info, /* Check and get Area value if present */ if (area) { - inet_aton (area, &OspfRI.area_id); + if (!inet_aton (area, &OspfRI.area_id)) + { + vty_out (vty, "Please specifya valid Area ID%s", VTY_NEWLINE); + return CMD_WARNING; + } scope = OSPF_OPAQUE_AREA_LSA; } else |