www.cendio.com
Bug 4933 - Add dynamic string allocation to rdp_in_unistr()
: Add dynamic string allocation to rdp_in_unistr()
Status: CLOSED FIXED
: ThinLinc
rdesktop
: 4.1.1
: PC Unknown
: P2 Normal
: 4.2.0
Assigned To:
:
:
:
: 4920
  Show dependency treegraph
 
Reported: 2013-12-06 10:06 by
Modified: 2014-05-05 14:52 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From cendio 2013-12-06 10:06:28
Change declaration as example below and use iconv to get the
correct string size then allocate the string.

rdp_in_unistr(STREAM s, int in_len, char **string, int *str_size)
------- Comment #1 From cendio 2013-12-10 13:02:05 -------
Commits r1773, r1774 adds the dynamic string allocation, commits r1775, r1776
cleans up related issues were used.
------- Comment #2 From cendio 2013-12-12 13:27:12 -------
Tester should test related functions using the rdp_in_unistr():

 - local printer rediretion
 - local drive redirection

 and server redirection in a farm.
------- Comment #3 From cendio 2013-12-12 16:30:08 -------
Vendordrop of upstream in commit 28197
------- Comment #4 From cendio 2014-05-05 14:52:19 -------
(In reply to comment #2)
> Tester should test related functions using the rdp_in_unistr():
> 
>  - local printer rediretion
>  - local drive redirection
> 
>  and server redirection in a farm.

All this tested on other bugs.