Attachment 19870 Details for Bug 18229 – Possible fix

[patch] Possible fix

0001-send_requested_reply-only-works-for-replies.patch (text/plain), 1.28 KB, created by Joachim Breitner on 2008-10-26 08:59:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Joachim Breitner

Created: 2008-10-26 08:59:02 UTC

Size: 1.28 KB

patch

obsolete

>From e56a58547797533dab9760d3a5658eaab46bc5f9 Mon Sep 17 00:00:00 2001
>From: Joachim Breitner <mail@joachim-breitner.de>
>Date: Sun, 26 Oct 2008 16:47:44 +0100
>Subject: [PATCH] send_requested_reply only works for replies
>
>This should fix 18229
>---
> bus/policy.c |   16 ++++++++++++++++
> 1 files changed, 16 insertions(+), 0 deletions(-)
>
>diff --git a/bus/policy.c b/bus/policy.c
>index caa544e..9a05f8b 100644
>--- a/bus/policy.c
>+++ b/bus/policy.c
>@@ -928,6 +928,14 @@ bus_client_policy_check_can_send (BusClientPolicy *policy,
>               continue;
>             }
>         }
>+      else
>+        {
>+        if (rule->d.send.requested_reply)
>+          {
>+              _dbus_verbose ("  (policy) skipping send_requested_reply rule for non-reply message\n");
>+              continue;
>+          }
>+        }
>       
>       if (rule->d.send.path != NULL)
>         {
>@@ -1124,6 +1132,14 @@ bus_client_policy_check_can_receive (BusClientPolicy *policy,
>               continue;
>             }
>         }
>+      else
>+        {
>+        if (rule->d.receive.requested_reply)
>+          {
>+              _dbus_verbose ("  (policy) skipping receive_requested_reply rule for non-reply message\n");
>+              continue;
>+          }
>+        }
>       
>       if (rule->d.receive.path != NULL)
>         {
>-- 
>1.5.6.5
>

Actions: View | Splinter Review

Attachments on bug 18229: 19870 | 20193 | 20397 | 20817 | 20899 | 20924 | 20928 | 20942

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.