On 10/06/2017 02:35 PM, Phillip Hallam-Baker wrote: > Reviewer: Phillip Hallam-Baker > Review result: Ready > > Given the design constraints in which the protocol operates, it is hard to see > how this could be done differently. > > I have two sets of security concerns. One is that implementations need to be > designed so as to avoid buffer overrun conditions and also to prevent such > conditions leading to a breach. Compression formats such as are inevitably used > in video and image applications tend to make promiscuous use of nested length > encoding formats that commonly lead to security vulnerabilities. > > This document does not have such a warning, having a reference on most of the > security issues, a warning on this issue should appear in: > https://tools.ietf.org/html/draft-ietf-rtcweb-security-08 > > The other security concern is that giving control over the host browser to run > pretty much arbitrary code was always going to be a security disaster but there > isn't much that can be done at this point. > Participant pushback, I'm neither a WG chair or a document editor: Was this intended as a review of a different document? The concern about compression formats seems to be something that belongs in compression format specifications, such as those referenced by PAYLOAD et al. As such, it would reasonably belong in -rtcweb-security, which pulls in security concerns from a number of fields. The generic concern about running Javascript in the browser seems to belong to rtcweb-overview if it belongs anywhere except in a generic architecture critique of the browser ecosystem. If there are concerns specific to JSEP, and the handling of SDP that is described in JSEP, it seems appropriate to document them here. Generic architectural issues and common security best practices don't seem to have the right home in this document. -- Surveillance is pervasive. Go Dark.