On September 13, 2019 at 12:08:47 PM, Susan Hares via Datatracker ( noreply@xxxxxxxx) wrote:
[Author hat on.]
Sue:
Hi! Thanks for the review!
...
Issue: Issue/Error in document
However, this document does not specify the error handling if a PIM implementation does
not abid by the MUSTs in section 4 defined as follows:
Unless otherwise specified, all the Flag Bits for each PIM type are
Reserved [RFC8126]. They MUST be set to zero on transmission, and
they MUST be ignored upon receipt. The specification of a new PIM
type, MUST indicate whether the bits should be treated differently.
This issues becomes a serious issue if it negates any error handling or changes
any existing error handling.
I suspect this is not really a serious issue, but a flaw in the write-up.
I suspect that because you set to zero on transmission, and
ignored on receipt you will not engage in any error handling.
However, it is necessary to state that so that machines testing
the PIM and SM-PIM protocols will easily add this to their testing suite.
The Flag Bits are currently Reserved in rfc7761, which describes the field as: "Set to zero on transmission. Ignored upon receipt.” IOW, the behavior doesn’t change at all…really just the name of the field and the per-type applicability.
I’m not sure what you would like to see added, given that there is no change in the behavior specified in rfc7761. Can you suggest something?
2) Issue: Security considerations in general are the same as
[RFC7761] and {RFC3973], but bogus error handling (depending how you do #1)
may impact the security considerations.
3) Minor Issue: Should Section 3 include a change to RFC3973 - section 4.7.1
No. PIM-DM (rfc3973) uses the packet formats (including the Header of course, §4.7.1) defined by PIM-SM…from §4.7: "All PIM-DM packets use the same format as PIM-SM packets. In the event of a discrepancy, PIM-SM [4] should be considered the definitive
specification.” The reference is to rfc2362, the PIM-DM spec at the time, which has now been replaced by rfc7761.
Thanks!
Alvaro.
|
