1. 31 Aug, 2015 2 commits
  2. 16 Aug, 2015 1 commit
  3. 01 Aug, 2015 1 commit
  4. 31 Jul, 2015 1 commit
  5. 30 Jul, 2015 1 commit
  6. 29 Jul, 2015 1 commit
  7. 13 Jul, 2015 1 commit
  8. 10 Jul, 2015 2 commits
  9. 09 Jul, 2015 1 commit
  10. 06 Jul, 2015 1 commit
  11. 02 Jul, 2015 1 commit
    • Dr. Stephen Henson's avatar
      Fix PSK handling. · 1392c238
      Dr. Stephen Henson authored
      
      
      The PSK identity hint should be stored in the SSL_SESSION structure
      and not in the parent context (which will overwrite values used
      by other SSL structures with the same SSL_CTX).
      
      Use BUF_strndup when copying identity as it may not be null terminated.
      
      Reviewed-by: default avatarTim Hudson <tjh@openssl.org>
      (cherry picked from commit 3c66a669)
      1392c238
  12. 25 Jun, 2015 1 commit
  13. 16 Jun, 2015 2 commits
  14. 11 Jun, 2015 8 commits
  15. 10 Jun, 2015 2 commits
  16. 08 Jun, 2015 1 commit
  17. 04 Jun, 2015 4 commits
    • Matt Caswell's avatar
      Remove misleading comment · bb82db1c
      Matt Caswell authored
      
      
      Remove a comment that suggested further clean up was required.
      DH_free() performs the necessary cleanup.
      
      With thanks to the Open Crypto Audit Project for reporting this issue.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit f3d88952)
      bb82db1c
    • Matt Caswell's avatar
      Clean premaster_secret for GOST · 470446db
      Matt Caswell authored
      
      
      Ensure OPENSSL_cleanse() is called on the premaster secret value calculated for GOST.
      
      With thanks to the Open Crypto Audit Project for reporting this issue.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit b7ee4815)
      
      Conflicts:
      	ssl/s3_srvr.c
      470446db
    • Matt Caswell's avatar
      Clean Kerberos pre-master secret · 91e64e14
      Matt Caswell authored
      
      
      Ensure the Kerberos pre-master secret has OPENSSL_cleanse called on it.
      
      With thanks to the Open Crypto Audit Project for reporting this issue.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit 4e3dbe37)
      91e64e14
    • Matt Caswell's avatar
      Fix off-by-one error in BN_bn2hex · 0d3a7e7c
      Matt Caswell authored
      
      
      A BIGNUM can have the value of -0. The function BN_bn2hex fails to account
      for this and can allocate a buffer one byte too short in the event of -0
      being used, leading to a one byte buffer overrun. All usage within the
      OpenSSL library is considered safe. Any security risk is considered
      negligible.
      
      With thanks to Mateusz Kocielski (LogicalTrust), Marek Kroemeke and
      Filip Palian for discovering and reporting this issue.
      
      Reviewed-by: default avatarTim Hudson <tjh@openssl.org>
      (cherry picked from commit c5635307)
      
      Conflicts:
      	crypto/bn/bn_print.c
      0d3a7e7c
  18. 02 Jun, 2015 3 commits
    • Richard Levitte's avatar
      Add the macro OPENSSL_SYS_WIN64 · a85eef72
      Richard Levitte authored
      
      
      This is for consistency.
      Additionally, have its presence define OPENSSL_SYS_WINDOWS as well.
      
      Reviewed-by: default avatarMatt Caswell <matt@openssl.org>
      (cherry picked from commit 3f131556)
      
      Conflicts:
      	e_os2.h
      a85eef72
    • Matt Caswell's avatar
      Fix race condition in NewSessionTicket · 0ae3473e
      Matt Caswell authored
      If a NewSessionTicket is received by a multi-threaded client when
      attempting to reuse a previous ticket then a race condition can occur
      potentially leading to a double free of the ticket data.
      
      CVE-2015-1791
      
      This also fixes RT#3808 where a session ID is changed for a session already
      in the client session cache. Since the session ID is the key to the cache
      this breaks the cache access.
      
      Parts of this patch were inspired by this Akamai change:
      https://github.com/akamai/openssl/commit/c0bf69a791239ceec64509f9f19fcafb2461b0d3
      
      
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit 27c76b9b)
      
      Conflicts:
      	ssl/ssl.h
      	ssl/ssl_err.c
      0ae3473e
    • Matt Caswell's avatar
      Clear state in DTLSv1_listen · 98377858
      Matt Caswell authored
      This is a backport of commit e83ee04b
      
       from
      the master branch (and this has also been applied to 1.0.2). In 1.0.2 this
      was CVE-2015-0207. For other branches there is no known security issue, but
      this is being backported as a precautionary measure.
      
      The DTLSv1_listen function is intended to be stateless and processes
      the initial ClientHello from many peers. It is common for user code to
      loop over the call to DTLSv1_listen until a valid ClientHello is received
      with an associated cookie. A defect in the implementation of DTLSv1_listen
      means that state is preserved in the SSL object from one invokation to the
      next.
      
      Reviewed-by: default avatarRichard Levitte <levitte@openssl.org>
      (cherry picked from commit cce3e4ad)
      98377858
  19. 28 May, 2015 2 commits
  20. 23 May, 2015 4 commits
    • Richard Levitte's avatar
      Have mkerr.pl treat already existing multiline string defs properly · 079495ca
      Richard Levitte authored
      
      
      Since source reformat, we ended up with some error reason string
      definitions that spanned two lines.  That in itself is fine, but we
      sometimes edited them to provide better strings than what could be
      automatically determined from the reason macro, for example:
      
          {ERR_REASON(SSL_R_NO_GOST_CERTIFICATE_SENT_BY_PEER),
           "Peer haven't sent GOST certificate, required for selected ciphersuite"},
      
      However, mkerr.pl didn't treat those two-line definitions right, and
      they ended up being retranslated to whatever the macro name would
      indicate, for example:
      
          {ERR_REASON(SSL_R_NO_GOST_CERTIFICATE_SENT_BY_PEER),
           "No gost certificate sent by peer"},
      
      Clearly not what we wanted.  This change fixes this problem.
      
      Reviewed-by: default avatarMatt Caswell <matt@openssl.org>
      (cherry picked from commit 2cfdfe09)
      079495ca
    • Richard Levitte's avatar
      Fix update and depend in engines/ · 591c819c
      Richard Levitte authored
      
      
      The update: target in engines/ didn't recurse into engines/ccgost.
      The update: and depend: targets in engines/ccgost needed a fixup.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit 8b822d25)
      591c819c
    • Richard Levitte's avatar
      Missed a couple of spots in the update change · 439c1934
      Richard Levitte authored
      
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit 6f45032f)
      
      Conflicts:
      	apps/Makefile
      439c1934
    • Richard Levitte's avatar
      Fix the update target and remove duplicate file updates · 1b840388
      Richard Levitte authored
      
      
      We had updates of certain header files in both Makefile.org and the
      Makefile in the directory the header file lived in.  This is error
      prone and also sometimes generates slightly different results (usually
      just a comment that differs) depending on which way the update was
      done.
      
      This removes the file update targets from the top level Makefile, adds
      an update: target in all Makefiles and has it depend on the depend: or
      local_depend: targets, whichever is appropriate, so we don't get a
      double run through the whole file tree.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      (cherry picked from commit 0f539dc1)
      
      Conflicts:
      	Makefile.org
      	apps/Makefile
      	test/Makefile
      	crypto/cmac/Makefile
      	crypto/srp/Makefile
      1b840388