1
0
mirror of https://github.com/mfontanini/libtins synced 2026-01-23 02:35:57 +01:00

Add flag to Stream to know whether it was attached

This commit is contained in:
Matias Fontanini
2016-10-30 10:31:16 -07:00
parent 5d6431d2d9
commit df7e7b391d
3 changed files with 18 additions and 1 deletions

View File

@@ -390,6 +390,10 @@ public:
} }
#endif // TINS_HAVE_TCP_STREAM_CUSTOM_DATA #endif // TINS_HAVE_TCP_STREAM_CUSTOM_DATA
/**
* Indicates whether this is a stream that we attached to after it had actually started
*/
bool is_attached() const;
private: private:
static Flow extract_client_flow(const PDU& packet); static Flow extract_client_flow(const PDU& packet);
static Flow extract_server_flow(const PDU& packet); static Flow extract_server_flow(const PDU& packet);
@@ -416,6 +420,7 @@ private:
timestamp_type last_seen_; timestamp_type last_seen_;
bool auto_cleanup_client_; bool auto_cleanup_client_;
bool auto_cleanup_server_; bool auto_cleanup_server_;
bool is_attached_;
#ifdef TINS_HAVE_TCP_STREAM_CUSTOM_DATA #ifdef TINS_HAVE_TCP_STREAM_CUSTOM_DATA
boost::any user_data_; boost::any user_data_;

View File

@@ -61,12 +61,16 @@ namespace TCPIP {
Stream::Stream(PDU& packet, const timestamp_type& ts) Stream::Stream(PDU& packet, const timestamp_type& ts)
: client_flow_(extract_client_flow(packet)), : client_flow_(extract_client_flow(packet)),
server_flow_(extract_server_flow(packet)), create_time_(ts), server_flow_(extract_server_flow(packet)), create_time_(ts),
last_seen_(ts), auto_cleanup_client_(true), auto_cleanup_server_(true) { last_seen_(ts), auto_cleanup_client_(true), auto_cleanup_server_(true),
is_attached_(false) {
const EthernetII* eth = packet.find_pdu<EthernetII>(); const EthernetII* eth = packet.find_pdu<EthernetII>();
if (eth) { if (eth) {
client_hw_addr_ = eth->src_addr(); client_hw_addr_ = eth->src_addr();
server_hw_addr_ = eth->dst_addr(); server_hw_addr_ = eth->dst_addr();
} }
const TCP& tcp = packet.rfind_pdu<TCP>();
// If this is not the first packet of a stream (SYN), then we've attached to it
is_attached_ = tcp.flags() != TCP::SYN;
} }
void Stream::process_packet(PDU& packet, const timestamp_type& ts) { void Stream::process_packet(PDU& packet, const timestamp_type& ts) {
@@ -268,6 +272,10 @@ bool Stream::ack_tracking_enabled() const {
return client_flow().ack_tracking_enabled() && server_flow().ack_tracking_enabled(); return client_flow().ack_tracking_enabled() && server_flow().ack_tracking_enabled();
} }
bool Stream::is_attached() const {
return is_attached_;
}
void Stream::on_client_flow_data(const Flow& /*flow*/) { void Stream::on_client_flow_data(const Flow& /*flow*/) {
if (on_client_data_callback_) { if (on_client_data_callback_) {
on_client_data_callback_(*this); on_client_data_callback_(*this);

View File

@@ -376,6 +376,7 @@ TEST_F(FlowTest, StreamFollower_ThreeWayHandshake) {
EXPECT_EQ(Flow::ESTABLISHED, stream.server_flow().state()); EXPECT_EQ(Flow::ESTABLISHED, stream.server_flow().state());
EXPECT_EQ(61U, stream.server_flow().sequence_number()); EXPECT_EQ(61U, stream.server_flow().sequence_number());
EXPECT_FALSE(stream.is_attached());
} }
TEST_F(FlowTest, StreamFollower_TCPOptions) { TEST_F(FlowTest, StreamFollower_TCPOptions) {
@@ -528,6 +529,9 @@ TEST_F(FlowTest, StreamFollower_AttachToStreams) {
} }
EXPECT_EQ(packets.size(), stream_client_payload_chunks.size()); EXPECT_EQ(packets.size(), stream_client_payload_chunks.size());
EXPECT_EQ(payload, merge_chunks(stream_client_payload_chunks)); EXPECT_EQ(payload, merge_chunks(stream_client_payload_chunks));
Stream& stream = follower.find_stream(IPv4Address("1.2.3.4"), 22, IPv4Address("4.3.2.1"), 25);
EXPECT_TRUE(stream.is_attached());
} }
TEST_F(FlowTest, StreamFollower_AttachToStreams_PacketsInBothDirections) { TEST_F(FlowTest, StreamFollower_AttachToStreams_PacketsInBothDirections) {