Attachment 119348 Details for Bug 75749 – Disallow unexpected replies

[patch] Disallow unexpected replies

0001-Disallow-unexpected-replies.patch (text/plain), 1.63 KB, created by Lars Uebernickel on 2015-11-02 15:13:09 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lars Uebernickel

Created: 2015-11-02 15:13:09 UTC

Size: 1.63 KB

patch

obsolete

>From 0e8e1504235cea81bfde95e07f249848840d55ae Mon Sep 17 00:00:00 2001
>From: Lars Uebernickel <lars@uebernic.de>
>Date: Mon, 2 Nov 2015 15:47:39 +0100
>Subject: [PATCH] Disallow unexpected replies
>
>Allowing to send replies when NO_REPLY_EXPECTED is set is useless in
>practice: Clients need to be careful not to send these replies, because
>bus policy could deny these messages. The spec even mentions that this
>issue exists.
>
>To make this more clear and misbehaving clients less likely, disallow
>sending unexpected replies entirely.
>
>Bug: https://bugs.freedesktop.org/show_bug.cgi?id=75749
>---
> doc/dbus-specification.xml | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
>diff --git a/doc/dbus-specification.xml b/doc/dbus-specification.xml
>index 1e0fe20..45213a0 100644
>--- a/doc/dbus-specification.xml
>+++ b/doc/dbus-specification.xml
>@@ -1383,12 +1383,7 @@
>                   <para>
>                     This message does not expect method return replies or
>                     error replies, even if it is of a type that can
>-                    have a reply; the reply can be omitted as an
>-                    optimization. It is compliant with this specification
>-                    to return the reply despite this flag, although doing
>-                    so on a bus with a non-trivial security policy
>-                    (such as the well-known system bus) may result in
>-                    access denial messages being logged for the reply.
>+                    have a reply; the reply must be omitted.
>                   </para>
>                   <para>
>                     Note that METHOD_CALL is the only message type currently
>-- 
>2.5.0
>

Actions: View | Splinter Review

Attachments on bug 75749: 95087 | 106749 | 119348 | 119435

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.