projects / openssl.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 937a766) | patch
Add length sanity check in SSLv2 n_do_ssl_write()
authorMatt Caswell <matt@openssl.org>
Wed, 29 Apr 2015 15:15:40 +0000 (16:15 +0100)
committerMatt Caswell <matt@openssl.org>
Wed, 29 Apr 2015 16:23:45 +0000 (17:23 +0100)
commitc5f8cd7bc661f90dc012c9d2bae1808a4281985f
treee3a40d82ab76ababc0517e08b36c9148878005a8tree | snapshot
parent937a766982229fd4aa3d9ceb544517f81a193206commit | diff
Add length sanity check in SSLv2 n_do_ssl_write()

Fortify flagged up a problem in n_do_ssl_write() in SSLv2. Analysing the
code I do not believe there is a real problem here. However the logic flows
are complicated enough that a sanity check of |len| is probably worthwhile.

Thanks to Kevin Wojtysiak (Int3 Solutions) and Paramjot Oberoi (Int3
Solutions) for reporting this issue.

Reviewed-by: Rich Salz <rsalz@openssl.org>
ssl/s2_pkt.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.