bus.c 22.2 KB
Newer Older
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28
/* -*- mode: C; c-file-style: "gnu" -*- */
/* bus.c  message bus context object
 *
 * Copyright (C) 2003 Red Hat, Inc.
 *
 * Licensed under the Academic Free License version 1.2
 * 
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 * 
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 *
 */

#include "bus.h"
#include "activation.h"
#include "connection.h"
#include "services.h"
#include "utils.h"
29
#include "policy.h"
30
#include "config-parser.h"
31 32
#include <dbus/dbus-list.h>
#include <dbus/dbus-hash.h>
33 34 35 36 37
#include <dbus/dbus-internals.h>

struct BusContext
{
  int refcount;
38
  char *type;
39
  char *address;
40
  char *pidfile;
41
  DBusLoop *loop;
42
  DBusList *servers;
43 44 45
  BusConnections *connections;
  BusActivation *activation;
  BusRegistry *registry;
46
  BusPolicy *policy;
47
  DBusUserDatabase *user_database;
48 49 50 51 52
  int activation_timeout;        /**< How long to wait for an activation to time out */
  int auth_timeout;              /**< How long to wait for an authentication to time out */
  int max_completed_connections;    /**< Max number of authorized connections */
  int max_incomplete_connections;   /**< Max number of incomplete connections */
  int max_connections_per_user;     /**< Max number of connections auth'd as same user */
53 54
};

55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107
static int server_data_slot = -1;
static int server_data_slot_refcount = 0;

typedef struct
{
  BusContext *context;
} BusServerData;

#define BUS_SERVER_DATA(server) (dbus_server_get_data ((server), server_data_slot))

static dbus_bool_t
server_data_slot_ref (void)
{
  if (server_data_slot < 0)
    {
      server_data_slot = dbus_server_allocate_data_slot ();
      
      if (server_data_slot < 0)
        return FALSE;

      _dbus_assert (server_data_slot_refcount == 0);
    }  

  server_data_slot_refcount += 1;

  return TRUE;
}

static void
server_data_slot_unref (void)
{
  _dbus_assert (server_data_slot_refcount > 0);

  server_data_slot_refcount -= 1;
  
  if (server_data_slot_refcount == 0)
    {
      dbus_server_free_data_slot (server_data_slot);
      server_data_slot = -1;
    }
}

static BusContext*
server_get_context (DBusServer *server)
{
  BusContext *context;
  BusServerData *bd;
  
  if (!server_data_slot_ref ())
    return NULL;

  bd = BUS_SERVER_DATA (server);
  if (bd == NULL)
108 109 110 111
    {
      server_data_slot_unref ();
      return NULL;
    }
112 113 114 115 116 117 118 119

  context = bd->context;

  server_data_slot_unref ();

  return context;
}

120
static dbus_bool_t
121 122 123 124
server_watch_callback (DBusWatch     *watch,
                       unsigned int   condition,
                       void          *data)
{
125
  DBusServer *server = data;
126

127
  return dbus_server_handle_watch (server, watch, condition);
128 129
}

130
static dbus_bool_t
131
add_server_watch (DBusWatch  *watch,
132
                  void       *data)
133
{
134 135 136 137 138
  DBusServer *server = data;
  BusContext *context;
  
  context = server_get_context (server);
  
139 140 141
  return _dbus_loop_add_watch (context->loop,
                               watch, server_watch_callback, server,
                               NULL);
142 143 144 145
}

static void
remove_server_watch (DBusWatch  *watch,
146
                     void       *data)
147
{
148 149 150 151 152
  DBusServer *server = data;
  BusContext *context;
  
  context = server_get_context (server);
  
153 154
  _dbus_loop_remove_watch (context->loop,
                           watch, server_watch_callback, server);
155 156
}

157

158 159 160 161
static void
server_timeout_callback (DBusTimeout   *timeout,
                         void          *data)
{
162
  /* can return FALSE on OOM but we just let it fire again later */
163 164 165 166 167
  dbus_timeout_handle (timeout);
}

static dbus_bool_t
add_server_timeout (DBusTimeout *timeout,
168
                    void        *data)
169
{
170 171 172 173 174
  DBusServer *server = data;
  BusContext *context;
  
  context = server_get_context (server);

175 176
  return _dbus_loop_add_timeout (context->loop,
                                 timeout, server_timeout_callback, server, NULL);
177 178 179 180
}

static void
remove_server_timeout (DBusTimeout *timeout,
181
                       void        *data)
182
{
183 184 185 186 187
  DBusServer *server = data;
  BusContext *context;
  
  context = server_get_context (server);
  
188 189
  _dbus_loop_remove_timeout (context->loop,
                             timeout, server_timeout_callback, server);
190 191
}

192 193 194 195 196 197 198 199
static void
new_connection_callback (DBusServer     *server,
                         DBusConnection *new_connection,
                         void           *data)
{
  BusContext *context = data;
  
  if (!bus_connections_setup_connection (context->connections, new_connection))
200 201
    {
      _dbus_verbose ("No memory to setup new connection\n");
202

203 204 205 206 207 208 209 210
      /* if we don't do this, it will get unref'd without
       * being disconnected... kind of strange really
       * that we have to do this, people won't get it right
       * in general.
       */
      dbus_connection_disconnect (new_connection);
    }
  
211
  /* on OOM, we won't have ref'd the connection so it will die. */
212 213
}

214 215 216 217 218 219 220 221
static void
free_server_data (void *data)
{
  BusServerData *bd = data;  
  
  dbus_free (bd);
}

222 223 224
static dbus_bool_t
setup_server (BusContext *context,
              DBusServer *server,
225
              char      **auth_mechanisms,
226
              DBusError  *error)
227
{
228
  BusServerData *bd;
229 230 231 232 233 234 235 236 237 238 239 240

  bd = dbus_new0 (BusServerData, 1);
  if (!dbus_server_set_data (server,
                             server_data_slot,
                             bd, free_server_data))
    {
      dbus_free (bd);
      BUS_SET_OOM (error);
      return FALSE;
    }

  bd->context = context;
241
  
242 243 244 245 246 247
  if (!dbus_server_set_auth_mechanisms (server, (const char**) auth_mechanisms))
    {
      BUS_SET_OOM (error);
      return FALSE;
    }
  
248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271
  dbus_server_set_new_connection_function (server,
                                           new_connection_callback,
                                           context, NULL);
  
  if (!dbus_server_set_watch_functions (server,
                                        add_server_watch,
                                        remove_server_watch,
                                        NULL,
                                        server,
                                        NULL))
    {
      BUS_SET_OOM (error);
      return FALSE;
    }

  if (!dbus_server_set_timeout_functions (server,
                                          add_server_timeout,
                                          remove_server_timeout,
                                          NULL,
                                          server, NULL))
    {
      BUS_SET_OOM (error);
      return FALSE;
    }
272
  
273 274 275
  return TRUE;
}

276
BusContext*
277
bus_context_new (const DBusString *config_file,
278
                 int               print_addr_fd,
279
                 DBusError        *error)
280 281
{
  BusContext *context;
282 283 284 285
  DBusList *link;
  DBusList **addresses;
  BusConfigParser *parser;
  DBusString full_address;
286
  const char *user, *pidfile;
287 288 289
  char **auth_mechanisms;
  DBusList **auth_mechanisms_list;
  int len;
290
  
291
  _DBUS_ASSERT_ERROR_IS_CLEAR (error);
292

293
  if (!_dbus_string_init (&full_address))
294 295 296 297
    {
      BUS_SET_OOM (error);
      return NULL;
    }
298

299 300 301 302 303 304 305
  if (!server_data_slot_ref ())
    {
      BUS_SET_OOM (error);
      _dbus_string_free (&full_address);
      return NULL;
    }
  
306 307
  parser = NULL;
  context = NULL;
308
  auth_mechanisms = NULL;
309 310 311 312
  
  parser = bus_config_load (config_file, error);
  if (parser == NULL)
    goto failed;
313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337

  /* Check for an existing pid file. Of course this is a race;
   * we'd have to use fcntl() locks on the pid file to
   * avoid that. But we want to check for the pid file
   * before overwriting any existing sockets, etc.
   */
  pidfile = bus_config_parser_get_pidfile (parser);
  if (pidfile != NULL)
    {
      DBusString u;
      DBusStat stbuf;
      DBusError tmp_error;
      
      dbus_error_init (&tmp_error);
      _dbus_string_init_const (&u, pidfile);
      
      if (_dbus_stat (&u, &stbuf, &tmp_error))
	{
	  dbus_set_error (error, DBUS_ERROR_FAILED,
			  "The pid file \"%s\" exists, if the message bus is not running, remove this file",
			  pidfile);
	  dbus_error_free (&tmp_error);
	  goto failed;
	}
    }
338 339 340 341 342
  
  context = dbus_new0 (BusContext, 1);
  if (context == NULL)
    {
      BUS_SET_OOM (error);
343
      goto failed;
344 345 346
    }
  
  context->refcount = 1;
347

348 349 350 351 352 353
  /* we need another ref of the server data slot for the context
   * to own
   */
  if (!server_data_slot_ref ())
    _dbus_assert_not_reached ("second ref of server data slot failed");
  
354
#ifdef DBUS_BUILD_TESTS
355
  context->activation_timeout = 6000;  /* 6 seconds */
356
#else
357
  context->activation_timeout = 15000; /* 15 seconds */
358
#endif
359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374

  /* Making this long risks making a DOS attack easier, but too short
   * and legitimate auth will fail.  If interactive auth (ask user for
   * password) is allowed, then potentially it has to be quite long.
   * Ultimately it needs to come from the configuration file.
   */     
  context->auth_timeout = 3000; /* 3 seconds */

  context->max_incomplete_connections = 32;
  context->max_connections_per_user = 128;

  /* Note that max_completed_connections / max_connections_per_user
   * is the number of users that would have to work together to
   * DOS all the other users.
   */
  context->max_completed_connections = 1024;
375 376 377 378 379 380 381

  context->user_database = _dbus_user_database_new ();
  if (context->user_database == NULL)
    {
      BUS_SET_OOM (error);
      goto failed;
    }
382
  
383
  context->loop = _dbus_loop_new ();
384 385 386 387 388 389
  if (context->loop == NULL)
    {
      BUS_SET_OOM (error);
      goto failed;
    }
  
390 391 392 393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418
  /* Build an array of auth mechanisms */
  
  auth_mechanisms_list = bus_config_parser_get_mechanisms (parser);
  len = _dbus_list_get_length (auth_mechanisms_list);

  if (len > 0)
    {
      int i;

      auth_mechanisms = dbus_new0 (char*, len + 1);
      if (auth_mechanisms == NULL)
        goto failed;
      
      i = 0;
      link = _dbus_list_get_first_link (auth_mechanisms_list);
      while (link != NULL)
        {
          auth_mechanisms[i] = _dbus_strdup (link->data);
          if (auth_mechanisms[i] == NULL)
            goto failed;
          link = _dbus_list_get_next_link (auth_mechanisms_list, link);
        }
    }
  else
    {
      auth_mechanisms = NULL;
    }

  /* Listen on our addresses */
419 420 421 422 423 424 425 426 427 428 429
  
  addresses = bus_config_parser_get_addresses (parser);  
  
  link = _dbus_list_get_first_link (addresses);
  while (link != NULL)
    {
      DBusServer *server;
      
      server = dbus_server_listen (link->data, error);
      if (server == NULL)
        goto failed;
430
      else if (!setup_server (context, server, auth_mechanisms, error))
431 432 433 434 435 436 437 438 439 440 441
        goto failed;

      if (!_dbus_list_append (&context->servers, server))
        {
          BUS_SET_OOM (error);
          goto failed;
        }          
      
      link = _dbus_list_get_next_link (addresses, link);
    }

442 443
  /* note that type may be NULL */
  context->type = _dbus_strdup (bus_config_parser_get_type (parser));
444
  
445 446 447 448 449 450 451 452 453 454 455 456 457 458 459 460 461 462 463 464 465 466 467
  /* We have to build the address backward, so that
   * <listen> later in the config file have priority
   */
  link = _dbus_list_get_last_link (&context->servers);
  while (link != NULL)
    {
      char *addr;
      
      addr = dbus_server_get_address (link->data);
      if (addr == NULL)
        {
          BUS_SET_OOM (error);
          goto failed;
        }

      if (_dbus_string_get_length (&full_address) > 0)
        {
          if (!_dbus_string_append (&full_address, ";"))
            {
              BUS_SET_OOM (error);
              goto failed;
            }
        }
468

469 470 471 472 473 474 475 476 477 478 479 480
      if (!_dbus_string_append (&full_address, addr))
        {
          BUS_SET_OOM (error);
          goto failed;
        }

      dbus_free (addr);

      link = _dbus_list_get_prev_link (&context->servers, link);
    }

  if (!_dbus_string_copy_data (&full_address, &context->address))
481 482 483 484
    {
      BUS_SET_OOM (error);
      goto failed;
    }
485

486 487 488 489 490 491 492 493 494 495 496 497 498 499 500 501 502 503 504 505 506 507 508 509 510 511 512 513 514 515 516 517 518 519 520 521 522 523 524 525 526 527
  /* Note that we don't know whether the print_addr_fd is
   * one of the sockets we're using to listen on, or some
   * other random thing. But I think the answer is "don't do
   * that then"
   */
  if (print_addr_fd >= 0)
    {
      DBusString addr;
      const char *a = bus_context_get_address (context);
      int bytes;
      
      _dbus_assert (a != NULL);
      if (!_dbus_string_init (&addr))
        {
          BUS_SET_OOM (error);
          goto failed;
        }
      
      if (!_dbus_string_append (&addr, a) ||
          !_dbus_string_append (&addr, "\n"))
        {
          _dbus_string_free (&addr);
          BUS_SET_OOM (error);
          goto failed;
        }

      bytes = _dbus_string_get_length (&addr);
      if (_dbus_write (print_addr_fd, &addr, 0, bytes) != bytes)
        {
          dbus_set_error (error, DBUS_ERROR_FAILED,
                          "Printing message bus address: %s\n",
                          _dbus_strerror (errno));
          _dbus_string_free (&addr);
          goto failed;
        }

      if (print_addr_fd > 2)
        _dbus_close (print_addr_fd, NULL);

      _dbus_string_free (&addr);
    }
  
528
  /* Create activation subsystem */
529
  
530
  context->activation = bus_activation_new (context, &full_address,
531 532
                                            bus_config_parser_get_service_dirs (parser),
                                            error);
533 534 535 536 537 538 539 540 541 542 543 544 545
  if (context->activation == NULL)
    {
      _DBUS_ASSERT_ERROR_IS_SET (error);
      goto failed;
    }

  context->connections = bus_connections_new (context);
  if (context->connections == NULL)
    {
      BUS_SET_OOM (error);
      goto failed;
    }

546
  context->registry = bus_registry_new (context);
547 548 549 550 551
  if (context->registry == NULL)
    {
      BUS_SET_OOM (error);
      goto failed;
    }
552

553 554 555
  context->policy = bus_config_parser_steal_policy (parser);
  _dbus_assert (context->policy != NULL);
  
556 557 558
  /* Now become a daemon if appropriate */
  if (bus_config_parser_get_fork (parser))
    {
559 560 561 562 563 564
      DBusString u;

      if (pidfile)
        _dbus_string_init_const (&u, pidfile);
      
      if (!_dbus_become_daemon (pidfile ? &u : NULL, error))
565 566
        goto failed;
    }
567 568 569 570 571 572 573 574 575 576 577 578 579
  else
    {
      /* Need to write PID file for ourselves, not for the child process */
      if (pidfile != NULL)
        {
          DBusString u;

          _dbus_string_init_const (&u, pidfile);
          
          if (!_dbus_write_pid_file (&u, _dbus_getpid (), error))
            goto failed;
        }
    }
580 581 582

  /* keep around the pid filename so we can delete it later */
  context->pidfile = _dbus_strdup (pidfile);
583 584 585 586 587 588 589 590 591 592 593 594 595 596 597 598 599 600 601 602 603 604 605 606 607

  /* Here we change our credentials if required,
   * as soon as we've set up our sockets and pidfile
   */
  user = bus_config_parser_get_user (parser);
  if (user != NULL)
    {
      DBusCredentials creds;
      DBusString u;

      _dbus_string_init_const (&u, user);

      if (!_dbus_credentials_from_username (&u, &creds) ||
          creds.uid < 0 ||
          creds.gid < 0)
        {
          dbus_set_error (error, DBUS_ERROR_FAILED,
                          "Could not get UID and GID for username \"%s\"",
                          user);
          goto failed;
        }
      
      if (!_dbus_change_identity (creds.uid, creds.gid, error))
        goto failed;
    }
608
  
609 610
  bus_config_parser_unref (parser);
  _dbus_string_free (&full_address);
611
  dbus_free_string_array (auth_mechanisms);
612
  server_data_slot_unref ();
613 614 615
  
  return context;
  
616
 failed:  
617 618 619 620 621 622 623
  if (parser != NULL)
    bus_config_parser_unref (parser);

  if (context != NULL)
    bus_context_unref (context);

  _dbus_string_free (&full_address);
624
  dbus_free_string_array (auth_mechanisms);
625 626 627

  server_data_slot_unref ();
  
628 629 630
  return NULL;
}

631 632 633
static void
shutdown_server (BusContext *context,
                 DBusServer *server)
634
{
635 636
  if (server == NULL ||
      !dbus_server_get_is_connected (server))
637 638
    return;
  
639
  if (!dbus_server_set_watch_functions (server,
640
                                        NULL, NULL, NULL,
641 642 643 644
                                        context,
                                        NULL))
    _dbus_assert_not_reached ("setting watch functions to NULL failed");
  
645
  if (!dbus_server_set_timeout_functions (server,
646
                                          NULL, NULL, NULL,
647 648 649 650
                                          context,
                                          NULL))
    _dbus_assert_not_reached ("setting timeout functions to NULL failed");
  
651 652 653 654 655 656 657 658 659 660 661 662 663 664 665
  dbus_server_disconnect (server);
}

void
bus_context_shutdown (BusContext  *context)
{
  DBusList *link;

  link = _dbus_list_get_first_link (&context->servers);
  while (link != NULL)
    {
      shutdown_server (context, link->data);

      link = _dbus_list_get_next_link (&context->servers, link);
    }
666 667 668 669 670 671 672 673 674 675 676 677 678 679 680 681 682
}

void
bus_context_ref (BusContext *context)
{
  _dbus_assert (context->refcount > 0);
  context->refcount += 1;
}

void
bus_context_unref (BusContext *context)
{
  _dbus_assert (context->refcount > 0);
  context->refcount -= 1;

  if (context->refcount == 0)
    {
683 684
      DBusList *link;
      
685 686
      _dbus_verbose ("Finalizing bus context %p\n", context);
      
687
      bus_context_shutdown (context);
688 689 690 691 692 693

      if (context->connections)
        {
          bus_connections_unref (context->connections);
          context->connections = NULL;
        }
694
      
695
      if (context->registry)
696 697 698 699 700
        {
          bus_registry_unref (context->registry);
          context->registry = NULL;
        }
      
701
      if (context->activation)
702 703 704 705
        {
          bus_activation_unref (context->activation);
          context->activation = NULL;
        }
706 707 708

      link = _dbus_list_get_first_link (&context->servers);
      while (link != NULL)
709
        {
710 711 712
          dbus_server_unref (link->data);
          
          link = _dbus_list_get_next_link (&context->servers, link);
713
        }
714
      _dbus_list_clear (&context->servers);
715

716
      if (context->policy)
717
        {
718 719
          bus_policy_unref (context->policy);
          context->policy = NULL;
720
        }
721
      
722 723
      if (context->loop)
        {
724
          _dbus_loop_unref (context->loop);
725 726 727
          context->loop = NULL;
        }
      
728
      dbus_free (context->type);
729
      dbus_free (context->address);
730 731 732 733 734 735 736 737 738 739 740 741 742 743

      if (context->pidfile)
	{
          DBusString u;
          _dbus_string_init_const (&u, context->pidfile);

          /* Deliberately ignore errors here, since there's not much
	   * we can do about it, and we're exiting anyways.
	   */
	  _dbus_delete_file (&u, NULL);

          dbus_free (context->pidfile); 
	}

744 745
      _dbus_user_database_unref (context->user_database);
      
746
      dbus_free (context);
747 748

      server_data_slot_unref ();
749 750 751
    }
}

752 753 754 755 756 757 758
/* type may be NULL */
const char*
bus_context_get_type (BusContext *context)
{
  return context->type;
}

759 760 761 762 763 764
const char*
bus_context_get_address (BusContext *context)
{
  return context->address;
}

765 766 767 768 769 770 771 772 773 774 775 776 777 778 779 780 781
BusRegistry*
bus_context_get_registry (BusContext  *context)
{
  return context->registry;
}

BusConnections*
bus_context_get_connections (BusContext  *context)
{
  return context->connections;
}

BusActivation*
bus_context_get_activation (BusContext  *context)
{
  return context->activation;
}
782

783
DBusLoop*
784 785 786 787 788
bus_context_get_loop (BusContext *context)
{
  return context->loop;
}

789 790 791 792 793 794
DBusUserDatabase*
bus_context_get_user_database (BusContext *context)
{
  return context->user_database;
}

795 796 797 798
dbus_bool_t
bus_context_allow_user (BusContext   *context,
                        unsigned long uid)
{
799 800 801
  return bus_policy_allow_user (context->policy,
                                context->user_database,
                                uid);
802 803
}

804 805 806
BusClientPolicy*
bus_context_create_client_policy (BusContext      *context,
                                  DBusConnection  *connection)
807
{
808
  return bus_policy_create_client_policy (context->policy, connection);
809
}
810 811 812 813 814 815 816

int
bus_context_get_activation_timeout (BusContext *context)
{
  
  return context->activation_timeout;
}
817 818 819 820 821 822 823 824 825 826 827 828 829 830 831 832 833 834 835 836 837 838 839 840 841 842 843 844 845 846 847 848 849 850 851 852 853 854 855 856 857 858 859 860 861 862 863 864 865 866 867 868 869 870 871 872 873 874 875 876 877 878 879

dbus_bool_t
bus_context_check_security_policy (BusContext     *context,
                                   DBusConnection *sender,
                                   DBusConnection *recipient,
                                   DBusMessage    *message,
                                   DBusError      *error)
{
  BusClientPolicy *sender_policy;
  BusClientPolicy *recipient_policy;

  /* NULL sender/receiver means the bus driver */
  
  if (sender != NULL)
    {
      _dbus_assert (dbus_connection_get_is_authenticated (sender));
      sender_policy = bus_connection_get_policy (sender);
    }
  else
    sender_policy = NULL;

  if (recipient != NULL)
    {
      _dbus_assert (dbus_connection_get_is_authenticated (recipient));
      recipient_policy = bus_connection_get_policy (recipient);
    }
  else
    recipient_policy = NULL;

  if (sender_policy &&
      !bus_client_policy_check_can_send (sender_policy,
                                         context->registry, recipient,
                                         message))
    {
      const char *dest = dbus_message_get_service (message);
      dbus_set_error (error, DBUS_ERROR_ACCESS_DENIED,
                      "A security policy in place prevents this sender "
                      "from sending this message to this recipient, "
                      "see message bus configuration file (rejected message "
                      "had name \"%s\" destination \"%s\")",
                      dbus_message_get_name (message),
                      dest ? dest : DBUS_SERVICE_DBUS);
      return FALSE;
    }

  if (recipient_policy &&
      !bus_client_policy_check_can_receive (recipient_policy,
                                            context->registry, sender,
                                            message))
    {
      const char *dest = dbus_message_get_service (message);
      dbus_set_error (error, DBUS_ERROR_ACCESS_DENIED,
                      "A security policy in place prevents this recipient "
                      "from receiving this message from this sender, "
                      "see message bus configuration file (rejected message "
                      "had name \"%s\" destination \"%s\")",
                      dbus_message_get_name (message),
                      dest ? dest : DBUS_SERVICE_DBUS);
      return FALSE;
    }

  return TRUE;
}