Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audio data sent when client has requested none #3019

Open
matt335672 opened this issue Mar 28, 2024 · 2 comments
Open

Audio data sent when client has requested none #3019

matt335672 opened this issue Mar 28, 2024 · 2 comments
Labels

Comments

@matt335672
Copy link
Member

xrdp version

All

Detailed xrdp version, build options

Argument 0 - xrdp
Argument 1 - -v
xrdp 0.10.80
  A Remote Desktop Protocol Server.
  Copyright (C) 2004-2024 Jay Sorg, Neutrino Labs, and all contributors.
  See https://github.com/neutrinolabs/xrdp for more information.

  Configure options:
      --enable-devel-all
      --enable-fuse
      --enable-pixman
      --enable-ipv6
      --enable-painter
      --enable-jpeg
      --with-imlib2
      --enable-vsock
      --with-freetype2
      --enable-utmp
      --with-libpcsclite
      --enable-neutrinordp
      --enable-ibus
      CC=gcc
      CFLAGS=-g -fvar-tracking -Wl,-z,now -Wno-error=deprecated-declarations

  Compiled with OpenSSL 3.0.2 15 Mar 2022

Operating system & version

Ubuntu 22.04

Installation method

Doesn't matter

Which backend do you use?

Either

What desktop environment do you use?

Any

Environment xrdp running on

VM

What's your client?

mstsc.exe

Area(s) with issue?

Audio redirection

Steps to reproduce

  • Start a session with audio output disabled
  • Load xrdp audio module
  • Set breakpoint in chansrv process_pcm_message()
  • Play audio

✔️ Expected Behavior

process_pcm_message() is not called

❌ Actual Behavior

process_pcm_message() is called, and sends data to client.

Anything else?

No response

@matt335672 matt335672 added the bug label Mar 28, 2024
@matt335672
Copy link
Member Author

I've opened this following a conversation on Gitter with @derekschrock who has observed this.

We don't seem to be observing the INFO_NOAUDIOPLAYBACK flag in the TS_INFO_PACKET. This is probably quite fiddly to fix as we need to get the flag from xrdp to chansrv.

@matt335672
Copy link
Member Author

To add to my comment above, the area which needs some attention is possibly xrdp_mm_trans_send_channel_setup() in xrdp_mm.c. The virtual channel rdpsnd is only used for audio output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant