Bug #5047
open
Router should reject /localhost prefix registration
Added by Junxiao Shi about 5 years ago.
Updated over 3 years ago.
Description
Steps to reproduce:
- Connect to a testbed router over TCP.
- Send a prefix registration command for
/localhost/nfd/rib
.
- Disconnect.
Expected: the prefix registration command should be rejected because /localhost
Interest cannot be forwarded to a remote router.
Actual: the prefix registration command succeeds; after disconnecting, HTTP status page stops working and shows 504 error.
Although the reason this registration could succeed is #2856, this issue is not a duplicate of #2856.
Management should prevent /localhost
registration to a non-local nexthop.
Management should prevent its own prefixes from being registered to somewhere else or being unregistered.
- Status changed from New to In Progress
- Assignee set to Eric Newberry
One issue presenting itself as I work on this is determining the face scope from the RIB. Since the RIB manager runs in a separate thread from the rest of management, we shouldn't directly use the FaceTable from the RIB manager, as can be done in the FIB manager to get this kind of face information. Any suggestions on how to proceed? One idea I had was issuing a request for face scope information from the RIB manager to the Face manager and using this information to accept or deny the request.
The best solution to note-3 is to finish #4529 first, so that you can access list of faces directly from RIB management.
Junxiao Shi wrote:
The best solution to note-3 is to finish #4529 first, so that you can access list of faces directly from RIB management.
FaceTable is not/will not be in the mgmt thread, so I'm not sure how #4529 would help.
Reply to note-5:
Management thread can lock data structure of main thread and read it directly. FaceTable is lockable and readable through this mechanism.
Junxiao Shi wrote:
Management thread can lock data structure of main thread and read it directly. FaceTable is lockable and readable through this mechanism.
yeah, I wouldn't call that "directly accessing the table"... Also if you just want to add locking you can do that without #4529.
- Status changed from In Progress to New
- Assignee deleted (
Eric Newberry)
- Target version deleted (
22.02)
Also available in: Atom
PDF