]> git.tuebingen.mpg.de Git - paraslash.git/commitdiff
Merge branch 'refs/heads/t/rm_oaep'
authorAndre Noll <maan@tuebingen.mpg.de>
Sat, 15 Apr 2017 13:51:38 +0000 (15:51 +0200)
committerAndre Noll <maan@tuebingen.mpg.de>
Sat, 15 Apr 2017 13:53:16 +0000 (15:53 +0200)
The open-coded OAEP padding removed in this series is no longer needed
since all supported libgcrypt versions support OAEP padding.

The series also contains a few follow up cleanups for gcrypt.c.

Cooking for three months.

* refs/heads/t/rm_oaep:
  gcrypt: Simplify init_random_seed_or_die().
  gcrypt: Rename E_OEAP to E_RSA_DECODE.
  gcrypt: Remove open-coded OAEP padding.

1  2 
NEWS.md
error.h
gcrypt.c

diff --combined NEWS.md
index 771e12df19c0a72efe530b5d880ab375404d44e6,d9326db76cfa80b50a8a268fef1b29f966cdeb09..fb6665293bbd45acc8a1cf892f4773e7c0d86156
+++ b/NEWS.md
@@@ -1,28 -1,9 +1,30 @@@
  NEWS
  ====
  
 +------------------------------------
 +0.6.0 (to be announced) "fuzzy flux"
 +------------------------------------
 +- Support for Mac OS X has been removed.
 +- On Linux systems, glibc-2.17 or newer is required to build the
 +  source tree.
 +- Support for RSA public keys in ASN format (as generated by openssl
 +  genrsa) has been removed. These keys have been deprecated since
 +  2011, so users should have long switched to keys generated with
 +  ssh-keygen(1).
++- If libgcrypt is used as the crypto library, we now require version
++  1.5.0 (released in 2011) or later.
 +
 +Downloads:
 +[tarball](./releases/paraslash-git.tar.bz2),
 +
  -------------------------------------
  0.5.7 (2016-12-31) "semantic density"
  -------------------------------------
 +
 +Mostly a bug fix release, and a bunch of internal improvements.
 +The only user-visible changes are the sanity checks for the touch
 +command and the new options to the ls command.
 +
  - Speedup of the base64 decoder.
  - One of the two source browsers has been removed from the web pages.
    The doxygen API reference still contains an HTML version of each
diff --combined error.h
index f2b15a4ea7d6ce3eb09fba6c8a6baf71b5af8737,3daee6af196dcadda49bb51bdad6337c31055693..fad417297df865267937ea48e8ba2f217cbfe652
+++ b/error.h
@@@ -12,6 -12,7 +12,6 @@@
        PARA_ERROR(AACDEC_INIT, "failed to init aac decoder"), \
        PARA_ERROR(AAC_DECODE, "aac decode error"), \
        PARA_ERROR(ACL_PERM, "access denied by acl"), \
 -      PARA_ERROR(ADD_CALLBACK, "can not add callback"), \
        PARA_ERROR(ADDRESS_LOOKUP, "can not resolve requested address"),\
        PARA_ERROR(AFH_RECV_BAD_FILENAME, "no file name given"), \
        PARA_ERROR(AFS_SHORT_READ, "short read from afs socket"), \
@@@ -87,6 -88,7 +87,6 @@@
        PARA_ERROR(CREATE_OPUS_DECODER, "could not create opus decoder"), \
        PARA_ERROR(DCCP_OVERRUN, "dccp output buffer buffer overrun"), \
        PARA_ERROR(DECRYPT, "decrypt error"), \
 -      PARA_ERROR(DEFAULT_COMP, "can not find default audio output component"), \
        PARA_ERROR(DUMMY_ROW, "attempted to access blob dummy object"), \
        PARA_ERROR(DUP_PIPE, "exec error: can not create pipe"), \
        PARA_ERROR(EMPTY, "file is empty"), \
        PARA_ERROR(MP3_INFO, "could not read mp3 info"), \
        PARA_ERROR(MP4ASC, "audio spec config error"), \
        PARA_ERROR(MP4V2, "mp4v2 library error"), \
-       PARA_ERROR(MPI_PRINT, "could not convert multi-precision integer"), \
        PARA_ERROR(MPI_SCAN, "could not scan multi-precision integer"), \
        PARA_ERROR(NAME_TOO_LONG, "name too long for struct sockaddr_un"), \
        PARA_ERROR(NO_AFHI, "audio format handler info required"), \
        PARA_ERROR(NOT_PLAYING, "not playing"), \
        PARA_ERROR(NO_VALID_FILES, "no valid file found in playlist"), \
        PARA_ERROR(NO_WMA, "asf/wma format not recognized"), \
-       PARA_ERROR(OEAP, "error during oeap (un)padding"), \
        PARA_ERROR(OGGDEC_BADHEADER, "invalid vorbis bitstream header"), \
        PARA_ERROR(OGGDEC_BADLINK, "invalid stream section or requested link corrupt"), \
        PARA_ERROR(OGGDEC_FAULT, "bug or heap/stack corruption"), \
        PARA_ERROR(OGG_PACKET_IN, "ogg_stream_packetin() failed"), \
        PARA_ERROR(OGG_STREAM_FLUSH, "ogg_stream_flush() failed"), \
        PARA_ERROR(OGG_SYNC, "internal ogg storage overflow"), \
 -      PARA_ERROR(OPEN_COMP, "OpenAComponent() error"), \
        PARA_ERROR(OPUS_COMMENT, "invalid or corrupted opus comment"), \
        PARA_ERROR(OPUS_DECODE, "opus decode error"), \
        PARA_ERROR(OPUS_HEADER, "invalid opus header"), \
        PARA_ERROR(PLAY_SYNTAX, "para_play: syntax error"), \
        PARA_ERROR(PREBUFFER_SUCCESS, "prebuffering complete"), \
        PARA_ERROR(PRIVATE_KEY, "can not read private key"), \
 -      PARA_ERROR(PUBLIC_KEY, "can not read public key"), \
        PARA_ERROR(QUEUE, "packet queue overrun"), \
        PARA_ERROR(READ_PATTERN, "did not read expected pattern"), \
        PARA_ERROR(RECV_EOF, "end of file"), \
        PARA_ERROR(REGEX, "regular expression error"), \
        PARA_ERROR(RESAMPLE_EOF, "resample filter: end of file"), \
        PARA_ERROR(RSA, "RSA error"), \
+       PARA_ERROR(RSA_DECODE, "RSA decoding error"), \
        PARA_ERROR(SB_PACKET_SIZE, "invalid sideband packet size or protocol error"), \
        PARA_ERROR(SCM_CREDENTIALS, "did not receive SCM credentials"), \
        PARA_ERROR(SENDER_CMD, "command not supported by this sender"), \
        PARA_ERROR(STAT_ITEM_PARSE, "failed to parse status item"), \
        PARA_ERROR(STATUS_TIMEOUT, "status item timeout"), \
        PARA_ERROR(STCO, "did not find stco atom"), \
 -      PARA_ERROR(STREAM_FORMAT, "could not set stream format"), \
        PARA_ERROR(STREAM_PACKETIN, "ogg stream packet-in error"), \
        PARA_ERROR(STREAM_PACKETOUT, "ogg stream packet-out error"), \
        PARA_ERROR(STREAM_PAGEIN, "ogg stream page-in error"), \
        PARA_ERROR(TOO_MANY_CLIENTS, "maximal number of stat clients exceeded"), \
        PARA_ERROR(UCRED_PERM, "permission denied"), \
        PARA_ERROR(UDP_OVERRUN, "output buffer overrun"), \
 -      PARA_ERROR(UNIT_INIT, "AudioUnitInitialize() error"), \
 -      PARA_ERROR(UNIT_START, "AudioUnitStart() error"), \
        PARA_ERROR(UNKNOWN_STAT_ITEM, "status item not recognized"), \
        PARA_ERROR(UNSUPPORTED_AUDIO_FORMAT, "given audio format not supported"), \
        PARA_ERROR(UNSUPPORTED_FILTER, "given filter not supported"), \
diff --combined gcrypt.c
index 1ae7cfb0deb4fb591dc97bcb5491b7cc54ec920a,5202c9b713eba8eba300d9ef336ffeae801ba03b..0ba8d526a3dd224135a73991e5b149c4a4eef1fb
+++ b/gcrypt.c
@@@ -19,9 -19,6 +19,6 @@@
  
  //#define GCRYPT_DEBUG 1
  
- static bool libgcrypt_has_oaep;
- static const char *rsa_decrypt_sexp;
  #ifdef GCRYPT_DEBUG
  static void dump_buffer(const char *msg, unsigned char *buf, int len)
  {
@@@ -63,134 -60,25 +60,25 @@@ void get_random_bytes_or_die(unsigned c
   * don't have to initialize any random seed here, but we must initialize the
   * gcrypt library. This task is performed by gcry_check_version() which can
   * also check that the gcrypt library version is at least the minimal required
-  * version. This function also tells us whether we have to use our own OAEP
-  * padding code.
+  * version.
   */
  void init_random_seed_or_die(void)
  {
-       const char *ver, *req_ver;
-       ver = gcry_check_version(NULL);
-       req_ver = "1.4.0";
-       if (!gcry_check_version(req_ver)) {
-               PARA_EMERG_LOG("fatal: need at least libgcrypt-%s, have: %s\n",
-                       req_ver, ver);
-               exit(EXIT_FAILURE);
-       }
-       req_ver = "1.5.0";
-       if (gcry_check_version(req_ver)) {
-               libgcrypt_has_oaep = true;
-               rsa_decrypt_sexp = "(enc-val(flags oaep)(rsa(a %m)))";
-       } else {
-               libgcrypt_has_oaep = false;
-               rsa_decrypt_sexp = "(enc-val(rsa(a %m)))";
-       }
+       const char *req_ver = "1.5.0";
+       if (gcry_check_version(req_ver))
+               return;
+       PARA_EMERG_LOG("fatal: need at least libgcrypt-%s, have: %s\n",
+               req_ver, gcry_check_version(NULL));
+       exit(EXIT_FAILURE);
  }
  
  /** S-expression for the public part of an RSA key. */
  #define RSA_PUBKEY_SEXP "(public-key (rsa (n %m) (e %m)))"
  /** S-expression for a private RSA key. */
  #define RSA_PRIVKEY_SEXP "(private-key (rsa (n %m) (e %m) (d %m) (p %m) (q %m) (u %m)))"
- /* rfc 3447, appendix B.2 */
- static void mgf1(unsigned char *seed, size_t seed_len, unsigned result_len,
-               unsigned char *result)
- {
-       gcry_error_t gret;
-       gcry_md_hd_t handle;
-       size_t n;
-       unsigned char *md;
-       unsigned char octet_string[4], *rp = result, *end = rp + result_len;
-       assert(result_len / HASH_SIZE < 1ULL << 31);
-       gret = gcry_md_open(&handle, GCRY_MD_SHA1, 0);
-       assert(gret == 0);
-       for (n = 0; rp < end; n++) {
-               gcry_md_write(handle, seed, seed_len);
-               octet_string[0] = (unsigned char)((n >> 24) & 255);
-               octet_string[1] = (unsigned char)((n >> 16) & 255);
-               octet_string[2] = (unsigned char)((n >> 8)) & 255;
-               octet_string[3] = (unsigned char)(n & 255);
-               gcry_md_write(handle, octet_string, 4);
-               gcry_md_final(handle);
-               md = gcry_md_read(handle, GCRY_MD_SHA1);
-               memcpy(rp, md, PARA_MIN(HASH_SIZE, (int)(end - rp)));
-               rp += HASH_SIZE;
-               gcry_md_reset(handle);
-       }
-       gcry_md_close(handle);
- }
- /** The sha1 hash of an empty file. */
- static const unsigned char empty_hash[HASH_SIZE] =
-       "\xda" "\x39" "\xa3" "\xee" "\x5e"
-       "\x6b" "\x4b" "\x0d" "\x32" "\x55"
-       "\xbf" "\xef" "\x95" "\x60" "\x18"
-       "\x90" "\xaf" "\xd8" "\x07" "\x09";
- /* rfc3447, section 7.1.1 */
- static void pad_oaep(unsigned char *in, size_t in_len, unsigned char *out,
-               size_t out_len)
- {
-       size_t ps_len = out_len - in_len - 2 * HASH_SIZE - 2;
-       size_t n, mask_len = out_len - HASH_SIZE - 1;
-       unsigned char *seed = out + 1, *db = seed + HASH_SIZE,
-               *ps = db + HASH_SIZE, *one = ps + ps_len;
-       unsigned char *db_mask, seed_mask[HASH_SIZE];
-       assert(in_len <= out_len - 2 - 2 * HASH_SIZE);
-       assert(out_len > 2 * HASH_SIZE + 2);
-       PARA_DEBUG_LOG("padding %zu byte input -> %zu byte output\n",
-               in_len, out_len);
-       dump_buffer("unpadded buffer", in, in_len);
-       out[0] = '\0';
-       get_random_bytes_or_die(seed, HASH_SIZE);
-       memcpy(db, empty_hash, HASH_SIZE);
-       memset(ps, 0, ps_len);
-       *one = 0x01;
-       memcpy(one + 1, in, in_len);
-       db_mask = para_malloc(mask_len);
-       mgf1(seed, HASH_SIZE, mask_len, db_mask);
-       for (n = 0; n < mask_len; n++)
-               db[n] ^= db_mask[n];
-       mgf1(db, mask_len, HASH_SIZE, seed_mask);
-       for (n = 0; n < HASH_SIZE; n++)
-               seed[n] ^= seed_mask[n];
-       free(db_mask);
-       dump_buffer("padded buffer", out, out_len);
- }
- /* rfc 3447, section 7.1.2 */
- static int unpad_oaep(unsigned char *in, size_t in_len, unsigned char *out,
-               size_t *out_len)
- {
-       unsigned char *masked_seed = in + 1;
-       unsigned char *db = in + 1 + HASH_SIZE;
-       unsigned char seed[HASH_SIZE], seed_mask[HASH_SIZE];
-       unsigned char *db_mask, *p;
-       size_t n, mask_len = in_len - HASH_SIZE - 1;
-       mgf1(db, mask_len, HASH_SIZE, seed_mask);
-       for (n = 0; n < HASH_SIZE; n++)
-               seed[n] = masked_seed[n] ^ seed_mask[n];
-       db_mask = para_malloc(mask_len);
-       mgf1(seed, HASH_SIZE, mask_len, db_mask);
-       for (n = 0; n < mask_len; n++)
-               db[n] ^= db_mask[n];
-       free(db_mask);
-       if (memcmp(db, empty_hash, HASH_SIZE))
-               return -E_OEAP;
-       for (p = db + HASH_SIZE; p < in + in_len - 1; p++)
-               if (*p != '\0')
-                       break;
-       if (p >= in + in_len - 1)
-               return -E_OEAP;
-       p++;
-       *out_len = in + in_len - p;
-       memcpy(out, p, *out_len);
-       return 1;
- }
+ /** S-expression for decryption. */
+ #define RSA_DECRYPT_SEXP "(enc-val(flags oaep)(rsa(a %m)))"
  
  struct asymmetric_key {
        gcry_sexp_t sexp;
@@@ -301,6 -189,64 +189,6 @@@ static inline int get_long_form_num_len
        return c & 0x7f;
  }
  
 -static int find_pubkey_bignum_offset(const unsigned char *data, int len)
 -{
 -      const unsigned char *p = data, *end = data + len;
 -
 -      /* the whole thing starts with one sequence */
 -      if (*p != ASN1_TYPE_SEQUENCE)
 -              return -E_ASN1_PARSE;
 -      p++;
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      if (is_short_form(*p))
 -              p++;
 -      else
 -              p += 1 + get_long_form_num_length_bytes(*p);
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      /* another sequence containing the object id, skip it */
 -      if (*p != ASN1_TYPE_SEQUENCE)
 -              return -E_ASN1_PARSE;
 -      p++;
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      if (!is_short_form(*p))
 -              return -E_ASN1_PARSE;
 -      p += 1 + get_short_form_length(*p);
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      /* all numbers are wrapped in a bit string object that follows */
 -      if (*p != ASN1_TYPE_BIT_STRING)
 -              return -E_ASN1_PARSE;
 -      p++;
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      if (is_short_form(*p))
 -              p++;
 -      else
 -              p += 1 + get_long_form_num_length_bytes(*p);
 -      p++; /* skip number of unused bits in the bit string */
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -
 -      /* next, we have a sequence of two integers (n and e) */
 -      if (*p != ASN1_TYPE_SEQUENCE)
 -              return -E_ASN1_PARSE;
 -      p++;
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      if (is_short_form(*p))
 -              p++;
 -      else
 -              p += 1 + get_long_form_num_length_bytes(*p);
 -      if (p >= end)
 -              return -E_ASN1_PARSE;
 -      if (*p != ASN1_TYPE_INTEGER)
 -              return -E_ASN1_PARSE;
 -      return p - data;
 -}
 -
  /*
   * Returns: Number of bytes scanned. This may differ from the value returned via
   * bn_bytes because the latter does not include the ASN.1 prefix and a leading
@@@ -402,7 -348,6 +290,7 @@@ static int get_private_key(const char *
        gcry_sexp_t sexp;
        struct asymmetric_key *key;
  
 +      *result = NULL;
        ret = decode_key(key_file, PRIVATE_KEY_HEADER, PRIVATE_KEY_FOOTER,
                &blob);
        if (ret < 0)
@@@ -481,6 -426,65 +369,6 @@@ free_blob
        return ret;
  }
  
 -/** Public keys start with this header. */
 -#define PUBLIC_KEY_HEADER "-----BEGIN PUBLIC KEY-----"
 -/** Public keys end with this footer. */
 -#define PUBLIC_KEY_FOOTER "-----END PUBLIC KEY-----"
 -
 -static int get_asn_public_key(const char *key_file, struct asymmetric_key **result)
 -{
 -      gcry_mpi_t n = NULL, e = NULL;
 -      unsigned char *blob, *cp, *end;
 -      int blob_size, ret, n_size;
 -      gcry_error_t gret;
 -      size_t erroff;
 -      gcry_sexp_t sexp;
 -      struct asymmetric_key *key;
 -
 -      ret = decode_key(key_file, PUBLIC_KEY_HEADER, PUBLIC_KEY_FOOTER,
 -              &blob);
 -      if (ret < 0)
 -              return ret;
 -      blob_size = ret;
 -      end = blob + blob_size;
 -      ret = find_pubkey_bignum_offset(blob, blob_size);
 -      if (ret < 0)
 -              goto free_blob;
 -      PARA_DEBUG_LOG("decoding public RSA params at offset %d\n", ret);
 -      cp = blob + ret;
 -
 -      ret = read_bignum(cp, end, &n, &n_size);
 -      if (ret < 0)
 -              goto free_blob;
 -      cp += ret;
 -
 -      ret = read_bignum(cp, end, &e, NULL);
 -      if (ret < 0)
 -              goto release_n;
 -
 -      gret = gcry_sexp_build(&sexp, &erroff, RSA_PUBKEY_SEXP, n, e);
 -      if (gret) {
 -              PARA_ERROR_LOG("offset %zu: %s\n", erroff,
 -                      gcry_strerror(gcry_err_code(gret)));
 -              ret = -E_SEXP_BUILD;
 -              goto release_e;
 -      }
 -      key = para_malloc(sizeof(*key));
 -      key->sexp = sexp;
 -      key->num_bytes = n_size;
 -      *result = key;
 -      ret = n_size;
 -      PARA_INFO_LOG("successfully read %d bit asn public key\n", n_size * 8);
 -
 -release_e:
 -      gcry_mpi_release(e);
 -release_n:
 -      gcry_mpi_release(n);
 -free_blob:
 -      free(blob);
 -      return ret;
 -}
 -
  static int get_ssh_public_key(unsigned char *data, int size, gcry_sexp_t *result)
  {
        int ret;
@@@ -542,7 -546,8 +430,7 @@@ free_blob
        return ret;
  }
  
 -int get_asymmetric_key(const char *key_file, int private,
 -              struct asymmetric_key **result)
 +int get_public_key(const char *key_file, struct asymmetric_key **result)
  {
        int ret, ret2;
        void *map;
        gcry_sexp_t sexp;
        struct asymmetric_key *key;
  
 -      if (private)
 -              return get_private_key(key_file, result);
        ret = mmap_full_file(key_file, O_RDONLY, &map, &map_size, NULL);
        if (ret < 0)
                return ret;
        ret = is_ssh_rsa_key(map, map_size);
        if (!ret) {
 -              ret = para_munmap(map, map_size);
 -              if (ret < 0)
 -                      return ret;
 -              return get_asn_public_key(key_file, result);
 +              para_munmap(map, map_size);
 +              return -E_SSH_PARSE;
        }
        start = map + ret;
        end = map + map_size;
@@@ -578,7 -587,7 +466,7 @@@ unmap
        return ret;
  }
  
 -void free_asymmetric_key(struct asymmetric_key *key)
 +void free_public_key(struct asymmetric_key *key)
  {
        if (!key)
                return;
        free(key);
  }
  
- static int decode_rsa(gcry_sexp_t sexp, int key_size, unsigned char *outbuf,
-               size_t *nbytes)
+ static int decode_rsa(gcry_sexp_t sexp, unsigned char *outbuf, size_t *nbytes)
  {
-       int ret;
-       gcry_error_t gret;
-       unsigned char oaep_buf[512];
-       gcry_mpi_t out_mpi;
-       if (libgcrypt_has_oaep) {
-               const char *p = gcry_sexp_nth_data(sexp, 1, nbytes);
-               if (!p) {
-                       PARA_ERROR_LOG("could not get data from list\n");
-                       return -E_OEAP;
-               }
-               memcpy(outbuf, p, *nbytes);
-               return 1;
-       }
-       out_mpi = gcry_sexp_nth_mpi(sexp, 0, GCRYMPI_FMT_USG);
-       if (!out_mpi)
-               return -E_SEXP_FIND;
-       gret = gcry_mpi_print(GCRYMPI_FMT_USG, oaep_buf, sizeof(oaep_buf),
-               nbytes, out_mpi);
-       if (gret) {
-               PARA_ERROR_LOG("mpi_print: %s\n", gcrypt_strerror(gret));
-               ret = -E_MPI_PRINT;
-               goto out_mpi_release;
-       }
-       /*
-        * An oaep-encoded buffer always starts with at least one zero byte.
-        * However, leading zeroes in an mpi are omitted in the output of
-        * gcry_mpi_print() when using the GCRYMPI_FMT_USG format. The
-        * alternative, GCRYMPI_FMT_STD, does not work either because here the
-        * leading zero(es) might also be omitted, depending on the value of
-        * the second byte.
-        *
-        * To circumvent this, we shift the oaep buffer to the right. But first
-        * we check that the buffer actually started with a zero byte, i.e. that
-        * nbytes < key_size. Otherwise a decoding error occurred.
-        */
-       ret = -E_SEXP_DECRYPT;
-       if (*nbytes >= key_size)
-               goto out_mpi_release;
-       memmove(oaep_buf + key_size - *nbytes, oaep_buf, *nbytes);
-       memset(oaep_buf, 0, key_size - *nbytes);
-       PARA_DEBUG_LOG("decrypted buffer before unpad (%d bytes):\n",
-               key_size);
-       dump_buffer("non-unpadded decrypted buffer", oaep_buf, key_size);
-       ret = unpad_oaep(oaep_buf, key_size, outbuf, nbytes);
-       if (ret < 0)
-               goto out_mpi_release;
-       PARA_DEBUG_LOG("decrypted buffer after unpad (%zu bytes):\n",
-               *nbytes);
-       dump_buffer("unpadded decrypted buffer", outbuf, *nbytes);
-       ret = 1;
- out_mpi_release:
-       gcry_mpi_release(out_mpi);
-       return ret;
+       const char *p = gcry_sexp_nth_data(sexp, 1, nbytes);
+       if (!p)
+               return -E_RSA_DECODE;
+       memcpy(outbuf, p, *nbytes);
+       return 1;
  }
  
  int priv_decrypt(const char *key_file, unsigned char *outbuf,
                unsigned char *inbuf, int inlen)
  {
        gcry_error_t gret;
-       int ret, key_size;
+       int ret;
        struct asymmetric_key *priv;
        gcry_mpi_t in_mpi = NULL;
        gcry_sexp_t in, out, priv_key;
        size_t nbytes;
  
 -      ret = check_key_file(key_file, true);
 +      ret = check_private_key_file(key_file);
        if (ret < 0)
                return ret;
        PARA_INFO_LOG("decrypting %d byte input\n", inlen);
        ret = get_private_key(key_file, &priv);
        if (ret < 0)
                return ret;
-       key_size = ret / 8;
  
        /* asymmetric key priv -> sexp priv_key */
        ret = -E_SEXP_FIND;
                goto key_release;
        }
        /* in_mpi -> in sexp */
-       gret = gcry_sexp_build(&in, NULL, rsa_decrypt_sexp, in_mpi);
+       gret = gcry_sexp_build(&in, NULL, RSA_DECRYPT_SEXP, in_mpi);
        if (gret) {
                PARA_ERROR_LOG("%s\n", gcrypt_strerror(gret));
                ret = -E_SEXP_BUILD;
                ret = -E_SEXP_DECRYPT;
                goto in_release;
        }
-       ret = decode_rsa(out, key_size, outbuf, &nbytes);
+       ret = decode_rsa(out, outbuf, &nbytes);
        if (ret < 0)
                goto out_release;
        PARA_INFO_LOG("successfully decrypted %zu byte message\n", nbytes);
@@@ -710,8 -667,7 +546,8 @@@ in_mpi_release
  key_release:
        gcry_sexp_release(priv_key);
  free_key:
 -      free_asymmetric_key(priv);
 +      gcry_sexp_release(priv->sexp);
 +      free(priv);
        return ret;
  }
  
@@@ -730,18 -686,7 +566,7 @@@ int pub_encrypt(struct asymmetric_key *
        pub_key = gcry_sexp_find_token(pub->sexp, "public-key", 0);
        if (!pub_key)
                return -E_SEXP_FIND;
-       if (libgcrypt_has_oaep) {
-               gret = gcry_sexp_build(&in, NULL,
-                       "(data(flags oaep)(value %b))", len, inbuf);
-       } else {
-               unsigned char padded_input[256];
-               const size_t pad_size = 256;
-               /* inbuf -> padded inbuf */
-               pad_oaep(inbuf, len, padded_input, pad_size);
-               /* padded inbuf -> in sexp */
-               gret = gcry_sexp_build(&in, NULL,
-                       "(data(flags raw)(value %b))", pad_size, padded_input);
-       }
+       gret = gcry_sexp_build(&in, NULL, "(data(flags oaep)(value %b))", len, inbuf);
        if (gret) {
                PARA_ERROR_LOG("%s\n", gcrypt_strerror(gret));
                ret = -E_SEXP_BUILD;