inex / IXP-Manager

Full stack web application powering peering at over 200 Internet Exchange Points (IXPs) globally.

Home Page:https://www.ixpmanager.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Missing check for 32bit routeserver asn?

jbemmel opened this issue · comments

Just wondering - I noticed

while

if routeserverasn > 65535 || peerasn > 65535 then
uses

 # it's unwise to conduct a 32-bit check on a 16-bit value
    if routeserverasn > 65535 || peerasn > 65535 then
            return true;

Shouldn't the first one do the same?

Actually, #842 so I guess this is a moot point