[ net-snmp-Bugs-426710 ] Route_Scan_Reload loads inactive routes

[<no...@so...>]

Bugs item #426710, was opened at 2001-05-23 11:26
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=112694&aid=426710&group_id=12694

Category: agent
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
>Assigned to: John Naylon (jbpn)
Summary: Route_Scan_Reload loads inactive routes

Initial Comment:
On a linux operating the system, in file mibII/var_route.c, the Route_Scan_Reload
function reads all entries in the /proc/net/route table. If a route is added and then 
deleted with the route command, the entry will remain in the /proc/net/route table,
with the RTF_UP flag cleared (not up.)

The Route_Scan_Reload function does not check if the route is 'up'. Therefore, it will 
list deleted routes.

The fix is to add a check in Route_Scan_Reload if the route is active:

     /* We want only active routes. Check the flags to verify the
       * route is up. If not, ignore it.
       */
      if (!(flags &  RTF_UP))
              continue;

pjweber

----------------------------------------------------------------------

>Comment By: John Naylon (jbpn)
Date: 2001-07-30 06:20

Message:
Logged In: YES 
user_id=93926

I think it would be more accurate to have ipRouteType equal
to invalid(2) for this case; RFC 1213 says:

   "Setting this object to the value invalid(2) has
    the effect of invalidating the corresponding entry
    in the ipRouteTable object.  That is, it
    effectively dissasociates the destination
    identified with said entry from the route
    identified with said entry.  It is an
    implementation-specific matter as to whether the
    agent removes an invalidated entry from the table.
    Accordingly, management stations must be prepared
    to receive tabular information from agents that
    corresponds to entries not currently in use.
    Proper interpretation of such entries requires
    examination of the relevant ipRouteType object."

Indeed, the code attempts to do this on some platforms,
although it seems to have bugs.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=112694&aid=426710&group_id=12694