Fix thinko in open_address

if (!uri && uri[0] = '/') is obviously never going to be true ...
Fix by just giving up in the !uri case.

Strangely enough, I'm not hitting this in testing on the tablet --
perhaps gcc is optimizing out the test?

diff --git a/dbus-server-bindings.c b/dbus-server-bindings.c
index 3c7718e..b29b1ab 100644 (file)
--- a/dbus-server-bindings.c
+++ b/dbus-server-bindings.c
@@ -47,7 +47,11 @@ static void open_address(const char *uri) {
        char *new_uri;
        size_t new_uri_len;
 
        char *new_uri;
        size_t new_uri_len;
 

-       if (!uri && uri[0] == '/') {
+       if (!uri)
+               /* Not much to do in this case ... */
+               return;
+
+       if (uri[0] == '/') {

                new_uri_len = strlen("file://") + strlen(uri) + 1;
                if (!(new_uri = calloc(new_uri_len, sizeof(char))))
                        exit(1);
                new_uri_len = strlen("file://") + strlen(uri) + 1;
                if (!(new_uri = calloc(new_uri_len, sizeof(char))))
                        exit(1);