author | mickeyl <mickeyl> | 2003-05-08 08:16:14 (UTC) |
---|---|---|
committer | mickeyl <mickeyl> | 2003-05-08 08:16:14 (UTC) |
commit | eb1dba5550e63ab7b915d55048175e2992ab5447 (patch) (side-by-side diff) | |
tree | 52b37945352c56effd58cacac071cba4f410bef2 /libopie2 | |
parent | 0950f27fa7136eb59f7e259cb4477ca10c89640d (diff) | |
download | opie-eb1dba5550e63ab7b915d55048175e2992ab5447.zip opie-eb1dba5550e63ab7b915d55048175e2992ab5447.tar.gz opie-eb1dba5550e63ab7b915d55048175e2992ab5447.tar.bz2 |
decouple dump files from live capture to shift control over 'what' is dumped to applications
-rw-r--r-- | libopie2/opienet/opcap.cpp | 42 | ||||
-rw-r--r-- | libopie2/opienet/opcap.h | 21 |
2 files changed, 48 insertions, 15 deletions
diff --git a/libopie2/opienet/opcap.cpp b/libopie2/opienet/opcap.cpp index bef9182..1de7124 100644 --- a/libopie2/opienet/opcap.cpp +++ b/libopie2/opienet/opcap.cpp @@ -865,4 +865,15 @@ bool OPacketCapturer::blocking() const +void OPacketCapturer::closeDumpFile() +{ + if ( _pcd ) + { + pcap_dump_close( _pcd ); + _pcd = 0; + } + pcap_close( _pch ); +} + + void OPacketCapturer::close() { @@ -874,11 +885,6 @@ void OPacketCapturer::close() delete _sn; } - if ( _pcd ) - { - pcap_dump_close( _pcd ); - _pcd = 0; - } - pcap_close( _pch ); - _open = false; + closeDumpFile(); + _open = false; } @@ -898,4 +904,15 @@ int OPacketCapturer::dataLink() const +void OPacketCapturer::dump( OPacket* p ) +{ + if ( !_pcd ) + { + qWarning( "OPacketCapturer::dump() - cannot dump without open capture file!" ); + return; + } + pcap_dump( (u_char*) _pcd, &p->_hdr, p->_data ); +} + + int OPacketCapturer::fileno() const { @@ -916,6 +933,4 @@ OPacket* OPacketCapturer::next() const unsigned char* pdata = pcap_next( _pch, &header ); qDebug( "<== OPacketCapturer::next()" ); - if ( _pcd ) - pcap_dump( (u_char*) _pcd, &header, pdata ); if ( pdata && header.len ) @@ -939,5 +954,5 @@ OPacket* OPacketCapturer::next() -bool OPacketCapturer::open( const QString& name, const QString& filename ) +bool OPacketCapturer::open( const QString& name ) { if ( _open ) @@ -976,5 +991,10 @@ bool OPacketCapturer::open( const QString& name, const QString& filename ) } - // if requested, open a dump + return true; +} + + +bool OPacketCapturer::openDumpFile( const QString& filename ) +{ pcap_dumper_t* dump = pcap_dump_open( _pch, const_cast<char*>( (const char*) filename ) ); if ( !dump ) diff --git a/libopie2/opienet/opcap.h b/libopie2/opienet/opcap.h index ad5b07c..9119972 100644 --- a/libopie2/opienet/opcap.h +++ b/libopie2/opienet/opcap.h @@ -116,4 +116,6 @@ class OPacket : public QObject Q_OBJECT + friend class OPacketCapturer; + public: OPacket( int datalink, packetheaderstruct, const unsigned char*, QObject* parent ); @@ -525,5 +527,5 @@ class OPacketCapturer : public QObject ~OPacketCapturer(); /** - * Setting the packet capturer to use blocking IO calls can be useful when + * Set the packet capturer to use blocking or non-blocking IO. This can be useful when * not using the socket notifier, e.g. without an application object. */ @@ -534,8 +536,12 @@ class OPacketCapturer : public QObject bool blocking() const; /** - * Closes the packet capturer. This is automatically done in the destructor. + * Close the packet capturer. This is automatically done in the destructor. */ void close(); /** + * Close the output capture file. + */ + void closeDumpFile(); + /** * @returns the data link type. * @see <pcap.h> for possible values. @@ -543,4 +549,8 @@ class OPacketCapturer : public QObject int dataLink() const; /** + * Dump a packet to the output capture file. + */ + void dump( OPacket* ); + /** * @returns the file descriptor of the packet capturer. This is only useful, if * not using the socket notifier, e.g. without an application object. @@ -554,7 +564,6 @@ class OPacketCapturer : public QObject /** * Open the packet capturer to capture packets in live-mode from @a interface. - * If a @a filename is given, all captured packets are output to a tcpdump-compatible capture file. */ - bool open( const QString& interface, const QString& filename = QString::null ); + bool open( const QString& interface ); /** * Open the packet capturer to capture packets in offline-mode from @a file. @@ -562,4 +571,8 @@ class OPacketCapturer : public QObject bool open( const QFile& file ); /** + * Open a prerecorded tcpdump compatible capture file for use with @ref dump() + */ + bool openDumpFile( const QString& filename ); + /** * @returns true if the packet capturer is open */ |