LGTM https://codereview.appspot.com/305460043/diff/1/external_tests/ssl_gtest/ssl_... File external_tests/ssl_gtest/ssl_auth_unittest.cc (right): https://codereview.appspot.com/305460043/diff/1/external_tests/ssl_gtest/ssl_... external_tests/ssl_gtest/ssl_auth_unittest.cc:76: uint8_t sig_alg = capture->buffer().data()[offset + 1]; I would use buffer().Read() here rather than split the two. https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3con.c File lib/ssl/ssl3con.c (right): https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3con.c#newcode7939 lib/ssl/ssl3con.c:7939: SignatureScheme sigScheme = ssl_sig_none; You can set this like with hashAlg. https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3ecc.c File lib/ssl/ssl3ecc.c (right): https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3ecc.c#newcode628 lib/ssl/ssl3ecc.c:628: SignatureScheme sigScheme = ssl_sig_none; follow pattern of hashAlg here too
https://codereview.appspot.com/305460043/diff/1/external_tests/ssl_gtest/ssl_... File external_tests/ssl_gtest/ssl_auth_unittest.cc (right): https://codereview.appspot.com/305460043/diff/1/external_tests/ssl_gtest/ssl_... external_tests/ssl_gtest/ssl_auth_unittest.cc:76: uint8_t sig_alg = capture->buffer().data()[offset + 1]; On 2016/09/13 00:40:57, mt wrote: > I would use buffer().Read() here rather than split the two. Done. https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3con.c File lib/ssl/ssl3con.c (right): https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3con.c#newcode7939 lib/ssl/ssl3con.c:7939: SignatureScheme sigScheme = ssl_sig_none; On 2016/09/13 00:40:57, mt wrote: > You can set this like with hashAlg. Done. https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3ecc.c File lib/ssl/ssl3ecc.c (right): https://codereview.appspot.com/305460043/diff/1/lib/ssl/ssl3ecc.c#newcode628 lib/ssl/ssl3ecc.c:628: SignatureScheme sigScheme = ssl_sig_none; On 2016/09/13 00:40:58, mt wrote: > follow pattern of hashAlg here too Done.