Michael Tuexen via RT wrote:
the attached patch fixes a bug where a single user message
was distributed over multiple DTLS records.

Dear Michael,

thanks for the patch. My app runs smoothly now.

I'm wondering if we can get rid of the redundant if statement that checks

if (len > SSL3_RT_MAX_PLAIN_LENGTH)

.
dtls1_write_app_data_bytes and dtls1_write_bytes both perform this check whereas dtls1_write_app_data_bytes calls dtls1_write_bytes. Let's remove this sanity check from dtls1_write_app_data_bytes because it'll get checked anyway further down the call stack. What do you think?

-Daniel

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       openssl-dev@openssl.org
Automated List Manager                           majord...@openssl.org

Reply via email to