Skip to content

Initial transport layer implementation#1

Merged
awakecoding merged 9 commits intoFreeRDP:masterfrom
llyzs:network
Jul 3, 2011
Merged

Initial transport layer implementation#1
awakecoding merged 9 commits intoFreeRDP:masterfrom
llyzs:network

Conversation

@llyzs
Copy link
Member

@llyzs llyzs commented Jul 3, 2011

With unit testing, it's already ok to send the first X.224 Connection Request PDU, and received X.224 Connection Confirm PDU from server.

awakecoding added a commit that referenced this pull request Jul 3, 2011
Initial transport layer implementation
@awakecoding awakecoding merged commit 3a9e3e5 into FreeRDP:master Jul 3, 2011
@fedsed fedsed mentioned this pull request Nov 22, 2011
@ghost ghost mentioned this pull request Jan 30, 2012
akallabeth pushed a commit that referenced this pull request Apr 8, 2024
found by coverity

705        if (transferDir == USBD_TRANSFER_DIRECTION_OUT)
706        {
      18. Condition !Stream_CheckAndLogRequiredLengthEx("com.freerdp.channels.urbdrc.client" /* "com.freerdp.channels.urbdrc.client" */, 3, s, OutputBufferSize, 1, "%s(%s:%zu)" /* "%s(%s:%zu)" */, <anonymous>, "/home/runner/work/FreeRDP/FreeRDP/channels/urbdrc/client/data_transfer.c", 707UL /* (size_t)707 */), taking true branch.
707                if (!Stream_CheckAndLogRequiredLength(TAG, s, OutputBufferSize))

CID 424733: (#1 of 1): Resource leak (RESOURCE_LEAK)
19. leaked_storage: Variable out going out of scope leaks the storage it points to.
708                        return ERROR_INVALID_DATA;
709                Stream_Copy(s, out, OutputBufferSize);
710        }
akallabeth pushed a commit that referenced this pull request Apr 8, 2024
found by coverity

655        if (Stream_Write_UTF16_String_From_UTF8(s, tmpDirCharLen - 1, tempDirectory->szTempDir,
656                                                ARRAYSIZE(tempDirectory->szTempDir), TRUE) < 0)

CID 424748: (#1 of 1): Resource leak (RESOURCE_LEAK)
9. leaked_storage: Variable s going out of scope leaks the storage it points to.
657                return ERROR_INTERNAL_ERROR;
658        /* Path must be 260 UTF16 characters with '\0' termination.
659         * ensure this here */
akallabeth pushed a commit that referenced this pull request Apr 8, 2024
found by coverity

128                if (winpr_asprintf(&headers, &size, post_header_fmt, path, hostname, blen) < 0)

CID 424888: (#1 of 1): Resource leak (RESOURCE_LEAK)
15. leaked_storage: Variable hostname going out of scope leaks the storage it points to.
129                        return FALSE;
130        }
qmfrederik pushed a commit to qmfrederik/FreeRDP that referenced this pull request Nov 10, 2024
Fix hanging RDP connection when opposite side is down and thru forwarder
@zero-wsh zero-wsh mentioned this pull request Aug 21, 2025
@KT-lcz KT-lcz mentioned this pull request Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants