Fixed review remarks.
authorArmin Novak <armin.novak@thincast.com>
Fri, 8 Feb 2019 11:01:33 +0000 (12:01 +0100)
committerArmin Novak <armin.novak@thincast.com>
Fri, 5 Apr 2019 07:14:35 +0000 (09:14 +0200)
libfreerdp/core/gateway/rdg.c
libfreerdp/crypto/certificate.c

index 79d9e19..3c67ea9 100644 (file)
@@ -614,8 +614,7 @@ static BOOL rdg_skip_seed_payload(rdpTls* tls, SSIZE_T lastResponseLength)
        BYTE seed_payload[10];
        const size_t size = sizeof(seed_payload);
 
-       if (size > SSIZE_MAX)
-               return FALSE;
+       assert(size < SSIZE_MAX);
 
        /* Per [MS-TSGU] 3.3.5.1 step 4, after final OK response RDG server sends
         * random "seed" payload of limited size. In practice it's 10 bytes.
index 1898060..ad8fd67 100644 (file)
@@ -622,7 +622,7 @@ BOOL certificate_data_print(rdpCertificateStore* certificate_store,
        int rc;
        HANDLE fp;
        char* tdata;
-       UINT64 size;
+       size_t size;
        DWORD written;
        /* reopen in append mode */
        /* Assure POSIX style paths, CreateFile expects either '/' or '\\' */