Bug 4764 - Escaping of % and \ doesn't work correct in TLDC Web interface
Summary: Escaping of % and \ doesn't work correct in TLDC Web interface
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: TLDC (Desktop Customizer) (show other bugs)
Version: 4.1.0
Hardware: PC Linux Ubuntu
: P2 Major
Target Milestone: MediumPrio
Assignee: Pierre Ossman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-09 10:45 CEST by Jens Maus
Modified: 2013-08-13 10:57 CEST (History)
0 users

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Jens Maus 2013-08-09 10:45:15 CEST 
When using the Web Administration interface to enter a command string in the "Applications (Manual)" part of the TLDC Web administration interface the "%" and "\" characters are not correctly escaped or converted to the generated *.desktop file in /opt/thinlinc/desktops/xdg_data_dir/applications/thinlinc

For example, when entering a path like

%ProgramFiles(x86)%\Adobe\Adobe Photoshop CS2\Photoshop.exe

in the "Command (required)" field of the web administration the web interface afterwards presents the final command string like:

tl-run-winapp-seamless "%ProgramFiles(x86)%\\Adobe\\Adobe" Photoshop "CS2\\Photoshop.exe"

Even more severe, the final *.desktop file has a broken Exec line like

Exec=tl-run-winapp-seamless "%%ProgramFiles(x86)%%\\\\Adobe\\\\Adobe" Photoshop "CS2\\\\Photoshop.exe"

As a workaround, however, one can perfectly modify the *desktop file by hand and use the following Exec line to correctly Escape the % characters

Exec=tl-run-winapp-seamless "%%ProgramFiles(x86)%%\Adobe\\\\Adobe Photoshop CS2\Photoshop.exe"
Comment 1 Jens Maus 2013-08-09 10:46:40 CEST 
Please note, that the last "correct" Exec line should be

Exec=tl-run-winapp-seamless "%%ProgramFiles(x86)%%\Adobe\Adobe Photoshop
CS2\Photoshop.exe"

That was wrong in my initial report.
Note You need to log in before you can comment on or make changes to this bug.