Project

General

Profile

Actions

Bug #5047

open

Router should reject /localhost prefix registration

Added by Junxiao Shi about 5 years ago. Updated over 3 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
RIB
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Steps to reproduce:

  1. Connect to a testbed router over TCP.
  2. Send a prefix registration command for /localhost/nfd/rib.
  3. 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.

Actions #1

Updated by Junxiao Shi about 5 years ago

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.

Actions #2

Updated by Eric Newberry almost 5 years ago

  • Status changed from New to In Progress
  • Assignee set to Eric Newberry
Actions #3

Updated by Eric Newberry almost 5 years ago

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.

Actions #4

Updated by Junxiao Shi almost 5 years ago

The best solution to note-3 is to finish #4529 first, so that you can access list of faces directly from RIB management.

Actions #5

Updated by Davide Pesavento almost 5 years ago

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.

Actions #6

Updated by Junxiao Shi almost 5 years ago

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.

Actions #7

Updated by Davide Pesavento almost 5 years ago

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.

Actions #8

Updated by Davide Pesavento over 3 years ago

  • Status changed from In Progress to New
  • Assignee deleted (Eric Newberry)
  • Target version deleted (22.02)
Actions

Also available in: Atom PDF