Bug 5834

Summary: tlclient fails to start when any line in configuration file exceeds 1020 characters
Product: ThinLinc Reporter: Karl Mikaelsson <derfian>
Component: ClientAssignee: Karl Mikaelsson <derfian>
Status: CLOSED FIXED    
Severity: Normal CC: astrand, hean01
Priority: P2 Keywords: hean01_tester, relnotes
Version: 4.5.0   
Target Milestone: 4.9.0   
Hardware: PC   
OS: Unknown   
Acceptance Criteria:

Description Karl Mikaelsson cendio 2016-04-04 14:25:57 CEST
We got a report from a customer that a server name configuration with a lot of trailing periods make tlclient unable to start. I've been able to reproduce and found that tlclient seems to spend a lot of time in ConfigMap::LoadFile when it encounters a long line.
Comment 6 Karl Mikaelsson cendio 2017-08-15 13:20:32 CEST
Tested on Linux, Windows and OS X with configuration files with lines exceeding 1020 characters.
Comment 8 Henrik Andersson cendio 2017-08-21 13:17:29 CEST
Inspected and ran the unit tests, looks all good to me.
Comment 9 Henrik Andersson cendio 2017-08-21 16:25:58 CEST
Verified on Windows 10, Fedora 26 and Mac OSX. Works as expected.