You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
in bpv7/ipn/ipnfw.c inside the applyRoutingOverride, bindOverride, and main functions, the arguments in the call to ipn_lookupOvrd are not in the the correct order. Argument list SHOULD be dataLabel, destNodeNbr, srcNodeNum, addr, rather than what is currently being used (dataLabel, srcNodeNbr, destNodeNbr, addr).
This bug results in routing overrides not being properly applied. If there are ipn override tests passing, it's likely because the source and destination are also swapped in the ipnadmin configurations.
The easiest fix would just be to use the correct order of arguments. A better (in my opinion) but more time consuming fix would be to make IPN overrides universally have the order dataLabel, srcNodeNbr, destNodeNbr as that seems less likely to produce bugs like this in the future. May not be worth the effort though.
-John
The text was updated successfully, but these errors were encountered:
in bpv7/ipn/ipnfw.c inside the applyRoutingOverride, bindOverride, and main functions, the arguments in the call to ipn_lookupOvrd are not in the the correct order. Argument list SHOULD be dataLabel, destNodeNbr, srcNodeNum, addr, rather than what is currently being used (dataLabel, srcNodeNbr, destNodeNbr, addr).
This bug results in routing overrides not being properly applied. If there are ipn override tests passing, it's likely because the source and destination are also swapped in the ipnadmin configurations.
The easiest fix would just be to use the correct order of arguments. A better (in my opinion) but more time consuming fix would be to make IPN overrides universally have the order dataLabel, srcNodeNbr, destNodeNbr as that seems less likely to produce bugs like this in the future. May not be worth the effort though.
-John
The text was updated successfully, but these errors were encountered: