package/rdesktop: Fix build with OpenSSL 1.1.x
authorBernd Kuhls <bernd.kuhls@t-online.de>
Sat, 25 Aug 2018 09:17:02 +0000 (11:17 +0200)
committerThomas Petazzoni <thomas.petazzoni@bootlin.com>
Sun, 9 Sep 2018 14:01:20 +0000 (16:01 +0200)
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
package/rdesktop/0002-openssl11.patch [new file with mode: 0644]

diff --git a/package/rdesktop/0002-openssl11.patch b/package/rdesktop/0002-openssl11.patch
new file mode 100644 (file)
index 0000000..050d6f6
--- /dev/null
@@ -0,0 +1,130 @@
+From bd6aa6acddf0ba640a49834807872f4cc0d0a773 Mon Sep 17 00:00:00 2001
+From: Jani Hakala <jjhakala@gmail.com>
+Date: Thu, 16 Jun 2016 14:28:15 +0300
+Subject: [PATCH] Fix OpenSSL 1.1 compability issues
+
+Some data types have been made opaque in OpenSSL version 1.1 so
+stack allocation and accessing struct fields directly does not work.
+
+Downloaded from upstream commit
+https://github.com/rdesktop/rdesktop/commit/bd6aa6acddf0ba640a49834807872f4cc0d0a773
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+---
+ ssl.c | 65 ++++++++++++++++++++++++++++++++++++-----------------------
+ 1 file changed, 40 insertions(+), 25 deletions(-)
+
+diff --git a/ssl.c b/ssl.c
+index 48751255..032e9b9e 100644
+--- a/ssl.c
++++ b/ssl.c
+@@ -88,7 +88,7 @@ rdssl_rsa_encrypt(uint8 * out, uint8 * in, int len, uint32 modulus_size, uint8 *
+                 uint8 * exponent)
+ {
+       BN_CTX *ctx;
+-      BIGNUM mod, exp, x, y;
++      BIGNUM *mod, *exp, *x, *y;
+       uint8 inr[SEC_MAX_MODULUS_SIZE];
+       int outlen;
+@@ -98,24 +98,24 @@ rdssl_rsa_encrypt(uint8 * out, uint8 * in, int len, uint32 modulus_size, uint8 *
+       reverse(inr, len);
+       ctx = BN_CTX_new();
+-      BN_init(&mod);
+-      BN_init(&exp);
+-      BN_init(&x);
+-      BN_init(&y);
+-
+-      BN_bin2bn(modulus, modulus_size, &mod);
+-      BN_bin2bn(exponent, SEC_EXPONENT_SIZE, &exp);
+-      BN_bin2bn(inr, len, &x);
+-      BN_mod_exp(&y, &x, &exp, &mod, ctx);
+-      outlen = BN_bn2bin(&y, out);
++      mod = BN_new();
++      exp = BN_new();
++      x = BN_new();
++      y = BN_new();
++
++      BN_bin2bn(modulus, modulus_size, mod);
++      BN_bin2bn(exponent, SEC_EXPONENT_SIZE, exp);
++      BN_bin2bn(inr, len, x);
++      BN_mod_exp(y, x, exp, mod, ctx);
++      outlen = BN_bn2bin(y, out);
+       reverse(out, outlen);
+       if (outlen < (int) modulus_size)
+               memset(out + outlen, 0, modulus_size - outlen);
+-      BN_free(&y);
+-      BN_clear_free(&x);
+-      BN_free(&exp);
+-      BN_free(&mod);
++      BN_free(y);
++      BN_clear_free(x);
++      BN_free(exp);
++      BN_free(mod);
+       BN_CTX_free(ctx);
+ }
+@@ -146,12 +146,20 @@ rdssl_cert_to_rkey(RDSSL_CERT * cert, uint32 * key_len)
+          Kudos to Richard Levitte for the following (. intiutive .) 
+          lines of code that resets the OID and let's us extract the key. */
+-      nid = OBJ_obj2nid(cert->cert_info->key->algor->algorithm);
++
++      X509_PUBKEY *key = NULL;
++      X509_ALGOR *algor = NULL;
++
++      key = X509_get_X509_PUBKEY(cert);
++      algor = X509_PUBKEY_get0_param(NULL, NULL, 0, &algor, key);
++
++      nid = OBJ_obj2nid(algor->algorithm);
++
+       if ((nid == NID_md5WithRSAEncryption) || (nid == NID_shaWithRSAEncryption))
+       {
+               DEBUG_RDP5(("Re-setting algorithm type to RSA in server certificate\n"));
+-              ASN1_OBJECT_free(cert->cert_info->key->algor->algorithm);
+-              cert->cert_info->key->algor->algorithm = OBJ_nid2obj(NID_rsaEncryption);
++              X509_PUBKEY_set0_param(key, OBJ_nid2obj(NID_rsaEncryption),
++                                     0, NULL, NULL, 0);
+       }
+       epk = X509_get_pubkey(cert);
+       if (NULL == epk)
+@@ -201,14 +209,24 @@ rdssl_rkey_get_exp_mod(RDSSL_RKEY * rkey, uint8 * exponent, uint32 max_exp_len,
+ {
+       int len;
+-      if ((BN_num_bytes(rkey->e) > (int) max_exp_len) ||
+-          (BN_num_bytes(rkey->n) > (int) max_mod_len))
++      BIGNUM *e = NULL;
++      BIGNUM *n = NULL;
++
++#if OPENSSL_VERSION_NUMBER < 0x10100000L
++      e = rkey->e;
++      n = rkey->n;
++#else
++      RSA_get0_key(rkey, &e, &n, NULL);
++#endif
++
++      if ((BN_num_bytes(e) > (int) max_exp_len) ||
++          (BN_num_bytes(n) > (int) max_mod_len))
+       {
+               return 1;
+       }
+-      len = BN_bn2bin(rkey->e, exponent);
++      len = BN_bn2bin(e, exponent);
+       reverse(exponent, len);
+-      len = BN_bn2bin(rkey->n, modulus);
++      len = BN_bn2bin(n, modulus);
+       reverse(modulus, len);
+       return 0;
+ }
+@@ -229,8 +247,5 @@ void
+ rdssl_hmac_md5(const void *key, int key_len, const unsigned char *msg, int msg_len,
+              unsigned char *md)
+ {
+-      HMAC_CTX ctx;
+-      HMAC_CTX_init(&ctx);
+       HMAC(EVP_md5(), key, key_len, msg, msg_len, md, NULL);
+-      HMAC_CTX_cleanup(&ctx);
+ }