Project

General

Profile

Actions
Copy link

Task #4117

closed

Remove check-for-self in ChronoSync updates

Added by Nicholas Gordon over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Low
Assignee:
Nicholas Gordon
Target version:
v0.4.0
Start date:
06/02/2017
Due date:
% Done:

0%

Estimated time:

Description

Currently the sync-logic-handler checks whether the received update is from itself. In theory this shouldn't be possible, because:

  1. Broadcast strategy does not send the interest back on the Face it received it on. This does not protect from loops of 3 or more, except
  2. Nonces in the Interest should cause NFD to drop the interest before it gets to this point.

It's a small change, but this code appears to be redundant and removing it would slightly improve performance and readability.

Actions
Copy link

Also available in: Atom PDF