[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]

 



On Tue, 2012-04-17 at 10:17 +0200, Peter Meerwald wrote:
> 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

The patch is good. I'll push it today when I get home. Thanks!

-- 
Tanu

> ---------- Forwarded message ----------
> Date: Fri,  6 Apr 2012 11:44:57 +0200
> From: Peter Meerwald <pmeerw at pmeerw.net>
> 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: [pulseaudio-discuss] [PATCH] dbus: fix dbus argument type in
>     iface-stream.c handle_move()
> 
> From: Peter Meerwald <p.meerwald at bct-electronic.com>
> 
> 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);



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

  Powered by Linux