mirror of
https://gitlab.freedesktop.org/dbus/dbus.git
synced 2026-01-04 01:30:22 +01:00
Add test-case for the same situation as fd.o #90952
This does not directly test the code in the previous commit, but it does confirm that calling dbus_connection_set_route_peer_messages() is enough to fix the observed bug. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=90952 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk> [smcv: re-worded commit message in response to review]
This commit is contained in:
parent
d9ee040d0b
commit
27a6538d5b
1 changed files with 24 additions and 0 deletions
|
|
@ -454,6 +454,8 @@ become_monitor (Fixture *f)
|
|||
int i;
|
||||
dbus_uint32_t zero = 0;
|
||||
|
||||
dbus_connection_set_route_peer_messages (f->monitor, TRUE);
|
||||
|
||||
if (f->config != NULL && f->config->match_rules != NULL)
|
||||
match_rules = f->config->match_rules;
|
||||
else
|
||||
|
|
@ -928,6 +930,28 @@ test_method_call (Fixture *f,
|
|||
|
||||
become_monitor (f);
|
||||
|
||||
/* regression test for
|
||||
* https://bugs.freedesktop.org/show_bug.cgi?id=90952 */
|
||||
m = dbus_message_new_method_call (f->recipient_name, "/foo",
|
||||
DBUS_INTERFACE_PEER, "Ping");
|
||||
dbus_connection_send (f->sender, m, NULL);
|
||||
dbus_message_unref (m);
|
||||
|
||||
while (g_queue_get_length (&f->monitored) < 2)
|
||||
test_main_context_iterate (f->ctx, TRUE);
|
||||
|
||||
m = g_queue_pop_head (&f->monitored);
|
||||
assert_method_call (m, f->sender_name, f->recipient_name, "/foo",
|
||||
DBUS_INTERFACE_PEER, "Ping", "");
|
||||
dbus_message_unref (m);
|
||||
|
||||
m = g_queue_pop_head (&f->monitored);
|
||||
assert_method_reply (m, f->recipient_name, f->sender_name, "");
|
||||
dbus_message_unref (m);
|
||||
|
||||
m = g_queue_pop_head (&f->monitored);
|
||||
g_assert (m == NULL);
|
||||
|
||||
m = dbus_message_new_method_call (f->recipient_name, "/foo", "com.example.bar",
|
||||
"Call1");
|
||||
dbus_connection_send (f->sender, m, NULL);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue