From 2be014e4f501dff18023450ca0b0ab39023ba290 Mon Sep 17 00:00:00 2001 From: winlin Date: Thu, 9 Oct 2014 14:24:52 +0800 Subject: [PATCH] remove the dead code for bug #180 --- trunk/src/rtmp/srs_protocol_handshake.cpp | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/trunk/src/rtmp/srs_protocol_handshake.cpp b/trunk/src/rtmp/srs_protocol_handshake.cpp index 12438bba5..89a5d3a32 100644 --- a/trunk/src/rtmp/srs_protocol_handshake.cpp +++ b/trunk/src/rtmp/srs_protocol_handshake.cpp @@ -1263,17 +1263,12 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs { int ret = ERROR_SUCCESS; - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - SrsHandshakeBytes* ptr = hs_bytes; - ssize_t nsize; // complex handshake if ((ret = hs_bytes->create_c0c1()) != ERROR_SUCCESS) { return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // sign c1 c1s1 c1; @@ -1282,8 +1277,6 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs return ret; } c1.dump(hs_bytes->c0c1 + 1); - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // verify c1 bool is_valid; @@ -1291,23 +1284,17 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs ret = ERROR_RTMP_TRY_SIMPLE_HS; return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); if ((ret = io->write(hs_bytes->c0c1, 1537, &nsize)) != ERROR_SUCCESS) { srs_warn("write c0c1 failed. ret=%d", ret); return ret; } srs_verbose("write c0c1 success."); - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // s0s1s2 if ((ret = hs_bytes->read_s0s1s2(io)) != ERROR_SUCCESS) { return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // plain text required. if (hs_bytes->s0s1s2[0] != 0x03) { @@ -1315,16 +1302,12 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs srs_warn("handshake failed, plain text required. ret=%d", ret); return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // verify s1s2 c1s1 s1; if ((ret = s1.parse(hs_bytes->s0s1s2 + 1, c1.schema)) != ERROR_SUCCESS) { return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); // never verify the s1, // for if forward to nginx-rtmp, verify s1 will failed, @@ -1334,15 +1317,11 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs if ((ret = hs_bytes->create_c2()) != ERROR_SUCCESS) { return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); c2s2 c2; if ((ret = c2.c2_create(&s1)) != ERROR_SUCCESS) { return ret; } - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); c2.dump(hs_bytes->c2); if ((ret = io->write(hs_bytes->c2, 1536, &nsize)) != ERROR_SUCCESS) { @@ -1350,8 +1329,6 @@ int SrsComplexHandshake::handshake_with_server(SrsHandshakeBytes* hs_bytes, ISrs return ret; } srs_verbose("complex handshake write c2 success."); - // TODO: FIXME: check memory corrupt, for https://github.com/winlinvip/simple-rtmp-server/issues/180 - srs_assert(ptr == hs_bytes); srs_trace("complex handshake success.");