[PATCH] No longer need to escape '+' in a D-Bus address

Michael Witten mfwitten at MIT.EDU
Sat Aug 16 20:54:32 PDT 2008


The character '+' wasn't included as a character
that only needs to be optionally escaped. As a
result, some generated paths wouldn't work (such
as the following that was produced on a Mac OS X
system:

    <listen>unix:tmpdir=/var/folders/9Y/9YL5MvFhGfySA5DvYnUpJE+++TI/-Tmp-/</listen>

Signed-off-by: Michael Witten <mfwitten at mit.edu>
---


    NOTE: I didn't test this in any way, because I already
          chose to circumvent the problem by choosing another
          tmpdir.


 dbus/dbus-address.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/dbus/dbus-address.c b/dbus/dbus-address.c
index 28e8842..85c81cb 100644
--- a/dbus/dbus-address.c
+++ b/dbus/dbus-address.c
@@ -85,6 +85,7 @@ _dbus_set_bad_address (DBusError  *error,
          (((b) >= 'a' && (b) <= 'z') ||                 \
           ((b) >= 'A' && (b) <= 'Z') ||                 \
           ((b) >= '0' && (b) <= '9') ||                 \
+          (b) == '+' ||                                 \
           (b) == '-' ||                                 \
           (b) == '_' ||                                 \
           (b) == '/' ||                                 \
-- 
1.5.6.rc0.52.g58124



More information about the dbus mailing list