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

Failed to load addins error when using RDP file which contains drivetoredirect:s #9989

Closed
DasHammett opened this issue Mar 17, 2024 · 1 comment · Fixed by #9991
Closed

Comments

@DasHammett
Copy link

When connecting to a Windows machine using an RDP file that has been generated to use in Windows environment and has the option drivestoredirect:s: with drives in Windows format (ie, drivestoredirect:s:c), Freerdp exits with error:
[ERROR][com.freerdp.client.common] - [freerdp_client_load_channels]: Failed to load addins [00000000]

Steps to reproduce the behavior:

  1. Create an RDP file with the line drivestoredirect:s:c
  2. Execute xfreerdp <rdp_file.rdp>
  3. See error

Expected behavior
A better error message to point out that the format of drivestoredirect:s is not compatible with Linux, as the current error message points to a missing or broken configuration (as in #9591)
or
Instead of an error and exit, issue a warning, ignore the drivetoredirects:s line in the rdp file and continue to connect to remote machine

Application details

  • FreeRDP version xfreerdp all versions
  • Command line used: xfreerdp <rdp_file.rdp>
  • Output of xfreerdp /buildconfig: Any build configuration will fail on this
  • OS version connecting to (server side): Any Windows version

Environment (please complete the following information):

  • OS: Linux
  • Version/Distribution: Any distro will fail
  • Architecture: amd64
@floppym
Copy link
Contributor

floppym commented Mar 17, 2024

Downstream bug: https://bugs.gentoo.org/926879

akallabeth added a commit to akallabeth/FreeRDP that referenced this issue Mar 18, 2024
Fixes FreeRDP#9989: ignore invalid drive (=path) redirections from
configuration, but notify the user with a warning message.
@akallabeth akallabeth added this to the next-3.0 milestone Mar 18, 2024
akallabeth added a commit that referenced this issue Mar 18, 2024
Fixes #9989: ignore invalid drive (=path) redirections from
configuration, but notify the user with a warning message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants