On Mon, Jul 24, 2017 at 5:08 PM, Hangbin Liu <liuhang...@gmail.com> wrote: > But what we want in inet6_rtm_getroute() and rt6_dump_route() is to > get/dump the route info. So we should get the info even it's unreachable or > prohibit.
If you want to dump prohibit/blackhole entry, then you have to check for null_entry, and rt->dst.error check is still needed because we could return error on other normal entries too, IOW, your v2 is correct if dumping prohibit/blackhole is expected. I thought we don't dump them. I am not sure about the behavior either.