]> git.puffer.fish Git - matthieu/frr.git/commit
bfdd: remove unnecessary receive timer restart
authorIgor Ryzhov <iryzhov@nfware.com>
Wed, 17 Nov 2021 23:20:43 +0000 (02:20 +0300)
committerIgor Ryzhov <iryzhov@nfware.com>
Mon, 6 Dec 2021 16:48:25 +0000 (19:48 +0300)
commita0f9b0c7c860f4b1c8f09723b51cd79298066fb7
tree176bd2b112c1c48538c22ad5ed3adb6438407a6b
parent2e38d79e646a2e37af717ed5e3a3b5365501fbc8
bfdd: remove unnecessary receive timer restart

When the detection time expires, we put the session down and restart the
timer. As the comment in the code says, it's needed to zero the remote
discriminator after the second expiration.

But the RFC clearly says that this must be done on the first expiration:

   bfd.RemoteDiscr

      The remote discriminator for this BFD session.  This is the
      discriminator chosen by the remote system, and is totally opaque
      to the local system.  This MUST be initialized to zero.  If a
      period of a Detection Time passes without the receipt of a valid,
      authenticated BFD packet from the remote system, this variable
      MUST be set to zero.

And we actually already do it in `ptm_bfd_sess_dn`, so there's no need
to reset the timer and wait for it twice.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
bfdd/bfd.c