Discussion:
[PATCH RFC wayland-protocols v3 1/2] Add secure output protocol
s***@collabora.com
2018-12-03 06:14:49 UTC
Permalink
Hi,

I've come to realise that it's become a bit of a fool's errand to try
and put all of the information related to content security into a single
protocol and have the interface be general yet sane, so I've instead
redesigned this to be general core which can be extended by other protocols.

This takes heavy inspiration from wl_surface "roles" and copies a lot of
the language of that. So each security interface will now have an
"implementation" which currently could be HDCP, the "I don't care" one,
or whatever one that someone decides to extend this with in the future.

The HDCP protocol is basically copied from [1], but I don't really know
a lot of the details of HDCP and it would currently rely on kernel
interfaces not upstream. This protocol is only really here to show what
it could look like.

I also have a basic proof-of-concept implementation of the first
protocol here [2].

Scott

[1] https://gitlab.freedesktop.org/wayland/weston/blob/5700409c4da6512cfdc6ffc0e644853273b455f3/protocol/content-protection.xml
[2] https://gitlab.freedesktop.org/ascent/weston/tree/secure-output
s***@collabora.com
2018-12-03 06:14:50 UTC
Permalink
From: Scott Anderson <***@collabora.com>

This protocol allows a client to ask the compositor to only allow it to
be displayed on a "secure" output.

This is based on a chromium protocol of the same name [1].

This protocol is mostly useful for closed systems, where the client can
trust the compositor, such as set-top boxes. This is not a way to
implement any kind of Digital Rights Management on desktops. The
protocol deliberately doesn't define what a "secure output" is, and the
compositor would be free to lie to the client anyway.

The protocol has been designed to be extendable with other protocols to
implement more specific types of security that a client may care about,
e.g. HDCP. This extensibility is based on the concept of a "role" that a
wl_surface has.

Signed-off-by: Scott Anderson <***@collabora.com>

[1] https://chromium.googlesource.com/chromium/src/+/master/third_party/wayland-protocols/unstable/secure-output/secure-output-unstable-v1.xml
---
Makefile.am | 1 +
unstable/secure-output/README | 4 +
.../secure-output-unstable-v1.xml | 189 ++++++++++++++++++
3 files changed, 194 insertions(+)
create mode 100644 unstable/secure-output/README
create mode 100644 unstable/secure-output/secure-output-unstable-v1.xml

diff --git a/Makefile.am b/Makefile.am
index 345ae6a..4d94747 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -23,6 +23,7 @@ unstable_protocols = \
unstable/xdg-decoration/xdg-decoration-unstable-v1.xml \
unstable/linux-explicit-synchronization/linux-explicit-synchronization-unstable-v1.xml \
unstable/primary-selection/primary-selection-unstable-v1.xml \
+ unstable/secure-output/secure-output-unstable-v1.xml \
$(NULL)

stable_protocols = \
diff --git a/unstable/secure-output/README b/unstable/secure-output/README
new file mode 100644
index 0000000..3251af9
--- /dev/null
+++ b/unstable/secure-output/README
@@ -0,0 +1,4 @@
+Secure output protocol
+
+Maintainers:
+David Reveman <***@chromium.org>
diff --git a/unstable/secure-output/secure-output-unstable-v1.xml b/unstable/secure-output/secure-output-unstable-v1.xml
new file mode 100644
index 0000000..867a03c
--- /dev/null
+++ b/unstable/secure-output/secure-output-unstable-v1.xml
@@ -0,0 +1,189 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<protocol name="secure_output_unstable_v1">
+
+ <copyright>
+ Copyright 2016 The Chromium Authors.
+ Copyright 2018 Collabora, Ltd.
+
+ Permission is hereby granted, free of charge, to any person obtaining a
+ copy of this software and associated documentation files (the "Software"),
+ to deal in the Software without restriction, including without limitation
+ the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ and/or sell copies of the Software, and to permit persons to whom the
+ Software is furnished to do so, subject to the following conditions:
+
+ The above copyright notice and this permission notice (including the next
+ paragraph) shall be included in all copies or substantial portions of the
+ Software.
+
+ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ DEALINGS IN THE SOFTWARE.
+ </copyright>
+
+ <description summary="Protocol for providing secure output">
+ This protocol specifies a set of interfaces used to prevent surface
+ contents from appearing in screenshots or from being visible on non-secure
+ outputs.
+
+ In order to prevent surface contents from appearing in screenshots or from
+ being visible on non-secure outputs, a client must first bind the global
+ interface "wp_secure_output" which, if a compositor supports secure output,
+ is exposed by the registry. Using the bound global object, the client uses
+ the "get_security" request to instantiate an interface extension for a
+ wl_surface object. This extended interface will then allow surfaces
+ to be marked as only visible on secure outputs.
+
+ Warning! The protocol described in this file is experimental and backward
+ incompatible changes may be made. Backward compatible changes may be added
+ together with the corresponding interface version bump. Backward
+ incompatible changes are done by bumping the version number in the protocol
+ and interface names and resetting the interface version. Once the protocol
+ is to be declared stable, the 'z' prefix and the version number in the
+ protocol and interface names are removed and the interface version number is
+ reset.
+ </description>
+
+ <interface name="zwp_secure_output_v1" version="1">
+ <description summary="secure output">
+ The global interface exposing secure output capabilities is used
+ to instantiate an interface extension for a wl_surface object.
+ This extended interface will then allow surfaces to be marked as
+ as only visible on secure outputs.
+ </description>
+
+ <request name="destroy" type="destructor">
+ <description summary="unbind from the secure output interface">
+ Informs the server that the client will not be using this
+ protocol object anymore. This does not affect any other objects,
+ security objects included.
+ </description>
+ </request>
+
+ <enum name="error">
+ <entry name="security_exists" value="0"
+ summary="the surface already has a security object associated"/>
+ <entry name="implementation" value="1"
+ summary="the wp_security already has an implementation"/>
+ </enum>
+
+ <request name="get_security">
+ <description summary="extend surface interface for security">
+ Instantiate an interface extension for the given wl_surface to
+ provide surface security. If the given wl_surface already has
+ a security object associated, the security_exists protocol error
+ is raised.
+ </description>
+ <arg name="id" type="new_id" interface="zwp_security_v1"
+ summary="the new security interface id"/>
+ <arg name="surface" type="object" interface="wl_surface"
+ summary="the surface"/>
+ </request>
+
+ <request name="set_default">
+ <description summary="set wp_security implementation">
+ This sets the implementation of a wp_security object to the server's
+ default. See the description of wp_security for the details of security
+ implementations.
+
+ This provides no extra interfaces to the wp_security object and would be
+ used by clients that don't care about the details of how they're secured.
+
+ The default implementation is considered a distinct implementation from
+ all other implementations, even if they're implemented in the same way.
+ </description>
+ <arg name="security" type="object" interface="zwp_security_v1"
+ summary="the wp_security object"/>
+ </request>
+ </interface>
+
+ <interface name="zwp_security_v1" version="1">
+ <description summary="security interface to a wl_surface">
+ An additional interface to a wl_surface object, which allows the
+ client to specify that a surface should not appear in screenshots
+ or be visible on non-secure outputs.
+
+ In order for a security object to take effect, the security object must
+ have an implementation set for it.
+
+ A security object can only have one implementation at a time. Initially
+ the security object has no implementation. Once the implementation is
+ set, it is set permanently set for the whole lifetime of the security
+ object. Setting the same interface again is allowed, unless explicitly
+ forbidden by the relevant interface specification.
+
+ Security implementations are set by the requests in other interfaces
+ such as wp_secure_output.set_default. The request should explicitly
+ mention that the request sets the wp_security's implementation.
+ Often, this request also creates a new protocol object that represents
+ the implementation and adds additional functionality to the wp_security.
+ When a client wants to destroy a wp_security, they must destroy this
+ 'implementation object' before the wp_security.
+
+ If the wl_surface associated with the security object is destroyed,
+ the security object becomes inert.
+
+ If the security object is destroyed, the security state is removed
+ from the wl_surface. The change will be applied on the next
+ wl_surface.commit.
+ </description>
+
+ <enum name="error">
+ <entry name="no_implementation" value="0"
+ summary="wp_security has no implementation"/>
+ </enum>
+
+ <request name="destroy" type="destructor">
+ <description summary="remove security from the surface">
+ The associated wl_surface's security state is removed.
+ The change is applied on the next wl_surface.commit.
+ </description>
+ </request>
+
+ <request name="set_constrain_outputs">
+ <description summary="set the only visible on secure output state">
+ Constrain visibility of wl_surface contents to secure outputs.
+ See wp_secure_output for the description.
+
+ A 'no_implementation' protocol error is raised if this is called
+ while the wp_security has no implementation.
+
+ The constrain outputs state is double-buffered, and will be applied
+ on the next wl_surface.commit.
+ </description>
+ <arg name="constrain" type="uint"
+ summary="wl_surface is constrained to secure outputs"/>
+ </request>
+
+ <event name="status">
+ <description summary="security status changed">
+ This event will be emitted when there is a change in the security
+ status.
+
+ If status is zero, then the content was unable to be displayed
+ securely. If status is one, then the content was able to be displayed
+ securely.
+
+ These events will not be sent until the wp_security has an
+ implementation, and the client should assume that the content cannot
+ be shown securely until this event is first sent.
+
+ If the content cannot be shown securely, then it is compositor-defined
+ for what will be displayed for the client's content.
+
+ A possible reason the security status changes is due to a new connector
+ being added or the client attaching a buffer too large for what the
+ security may protect. However, it is not limited to these reasons.
+
+ A client may want to listen to this event and lower the resolution of
+ their content until it can successfully be shown securely.
+ </description>
+ <arg name="status" type="uint" summary="the security status"/>
+ </event>
+ </interface>
+
+</protocol>
--
2.19.2
s***@collabora.com
2018-12-03 06:14:51 UTC
Permalink
This post might be inappropriate. Click to display it.
Continue reading on narkive:
Loading...