Last Call Review of draft-ietf-syslog-dtls-
review-ietf-syslog-dtls-secdir-lc-hanna-2010-05-19-00

Request Review of draft-ietf-syslog-dtls
Requested rev. no specific revision (document currently at 06)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2010-05-18
Requested 2010-05-11
Authors Hongyan Feng, Joseph Salowey, Tom Petch, Rainer Gerhards
Draft last updated 2010-05-19
Completed reviews Secdir Last Call review of -?? by Steve Hanna
Assignment Reviewer Steve Hanna
State Completed
Review review-ietf-syslog-dtls-secdir-lc-hanna-2010-05-19
Review completed: 2010-05-19

Review
review-ietf-syslog-dtls-secdir-lc-hanna-2010-05-19

I have reviewed this document as part of the security directorate's
ongoing effort to review all IETF documents being processed by the IESG.
These comments were written primarily for the benefit of the security
area directors.  Document editors and WG chairs should treat these
comments just like any other comments.

This document defines a DTLS transport for syslog. The document is
well-written, clear, and seems to serve a worthwhile purpose.

Although the security considerations section is brief (mainly just
referring to the security considerations in RFC 5425, RFC 5246,
and RFC 4347), it is largely adequate. I see only one omission.

One difference between the security considerations for syslog over
DTLS and those for syslog over TLS (unnoted in the current Security
Considerations section) is that DTLS does not provide retransmission.
If an attacker can cause a packet to be dropped (especially one
carrying significant information about an attack), the transport
receiver may not consider this a significant event and so the syslog
server may be completely unaware of the occurrence. This contrasts
with syslog over TLS where a dropped packet would be retransmitted
until acknowledged or until the TLS connection goes down (indicating
to the transport sender and receiver and perhaps to the syslog client
and server that a significant event has occurred). Maybe it would be
a good idea to recommend that the transport receiver notice gaps in
the DTLS sequence numbers and notify the syslog server. Still, this
is not as good from a security standpoint as syslog over TLS since
none of the client code will be aware that the dropped message was
not received. At least, there should be a discussion of this issue
in the Security Considerations section of this document.

In addition to this concern, I have noticed a few areas that could
use some clarification and maybe some fixes.

Section 5.3 says "Implementations MUST support the denial of service
countermeasures defined by DTLS." That's good but it's not clear
whether this means that these countermeasures MUST always be enabled.
Since that is not explicitly stated, it seems that a server could
have those countermeasures enabled by default and a client could
have them disabled by default. That would result in a client and
server that would not interoperate until the administrator tracked
down the problem and changed their configuration. I suggest that
the document be changed to require not only that implementations
support these countermeasures but that they be enabled by default.

Section 7 says "The security policies for syslog over DTLS are the
same as those described in [RFC5425]." Does that mean that all the
normative text in section 5 of RFC 5425 applies to implementations
of this document as well? I hope so but if that's the intent, it
should be explicitly stated (for example by adding the text "and
all the normative requirements of section 5 of [RFC5425] apply").

Once these issues are addressed, I'm sure that the document will
be a worthwhile and relatively secure addition to the RFC series.
Congratulations and thanks to the editors/authors for their work.

Thanks,

Steve