Skip to content
  1. Aug 01, 2016
  2. Jul 31, 2016
  3. Jul 30, 2016
  4. Jul 29, 2016
    • Dr. Stephen Henson's avatar
      print out MAC algorithm · 44c248b5
      Dr. Stephen Henson authored
      
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      44c248b5
    • Dr. Stephen Henson's avatar
      Fix CRL time comparison. · e032117d
      Dr. Stephen Henson authored
      
      
      Thanks to David Benjamin <davidben@google.com> for reporting this bug.
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      e032117d
    • Matt Caswell's avatar
      Update the SSL_set_bio()/SSL_set0_rbio()/SSL_set0_wbio() docs · e040a42e
      Matt Caswell authored
      
      
      Update the documentation for the newly renamed and modified SSL_set0_rbio()
      and SSL_set0_wbio() functions. State that they should be preferred over
      SSL_set_bio(). Attempt to document the ownership rules for SSL_set_bio().
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      e040a42e
    • Matt Caswell's avatar
      Simplify and rename SSL_set_rbio() and SSL_set_wbio() · 65e2d672
      Matt Caswell authored
      
      
      SSL_set_rbio() and SSL_set_wbio() are new functions in 1.1.0 and really
      should be called SSL_set0_rbio() and SSL_set0_wbio(). The old
      implementation was not consistent with what "set0" means though as there
      were special cases around what happens if the rbio and wbio are the same.
      We were only ever taking one reference on the BIO, and checking everywhere
      whether the rbio and wbio are the same so as not to double free.
      
      A better approach is to rename the functions to SSL_set0_rbio() and
      SSL_set0_wbio(). If an existing BIO is present it is *always* freed
      regardless of whether the rbio and wbio are the same or not. It is
      therefore the callers responsibility to ensure that a reference is taken
      for *each* usage, i.e. one for the rbio and one for the wbio.
      
      The legacy function SSL_set_bio() takes both the rbio and wbio in one go
      and sets them both. We can wrap up the old behaviour in the implementation
      of that function, i.e. previously if the rbio and wbio are the same in the
      call to this function then the caller only needed to ensure one reference
      was passed. This behaviour is retained by internally upping the ref count.
      
      This commit was inspired by BoringSSL commit f715c423224.
      
      RT#4572
      
      Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
      65e2d672