[PATCH] dbus: fix dbus argument type in iface-stream.c handle_move() (fwd)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Tanu,

can you please have a look at below patch/bug report?

I'm trying to move a stream from one device to another using dbus; I 
think the parameter types do not match between the API and the C implementation

thanks, p.

---------- Forwarded message ----------
Date: Fri,  6 Apr 2012 11:44:57 +0200
From: Peter Meerwald <pmeerw@xxxxxxxxxx>
Reply-To: General PulseAudio Discussion
    <pulseaudio-discuss at lists.freedesktop.org>
To: General PulseAudio Discussion <pulseaudio-discuss at lists.freedesktop.org>
Cc: Peter Meerwald <p.meerwald at bct-electronic.com>
Subject: [PATCH] dbus: fix dbus argument type in
    iface-stream.c handle_move()

From: Peter Meerwald <p.meerwald@xxxxxxxxxxxxxxxxxx>

BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=48376
Signed-off-by: Peter Meerwald <p.meerwald at bct-electronic.com>
---
 src/modules/dbus/iface-stream.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/modules/dbus/iface-stream.c b/src/modules/dbus/iface-stream.c
index ecd0ff3..c6dc1c9 100644
--- a/src/modules/dbus/iface-stream.c
+++ b/src/modules/dbus/iface-stream.c
@@ -611,7 +611,7 @@ static void handle_move(DBusConnection *conn, DBusMessage *msg, void *userdata)
     pa_assert(msg);
     pa_assert(s);
 
-    pa_assert_se(dbus_message_get_args(msg, NULL, DBUS_TYPE_STRING, &device, DBUS_TYPE_INVALID));
+    pa_assert_se(dbus_message_get_args(msg, NULL, DBUS_TYPE_OBJECT_PATH, &device, DBUS_TYPE_INVALID));
 
     if (s->type == STREAM_TYPE_PLAYBACK) {
         pa_sink *sink = pa_dbusiface_core_get_sink(s->core, device);
-- 
1.7.4.1

_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux