Ethereal-dev: [Ethereal-dev] [PATCH] crude text-based proto. hier. stats in ethereal

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Khachaturov Vassilii <Vassilii.Khachaturov@xxxxxxxxxxxx>
Date: Tue, 27 Aug 2002 17:01:12 -0400
(Mailing list arhive reader - see the thread cited below for more info)

New files and a patch to the existing ones attached.
(On Windows you'd need the same change as to Makefile.am to be applied
to Makefile.nmake)

Functionality: when Tools->Protocol Hierarchy Stats
is selected, a text version of the stats formatted for easy postprocessing
is printed on the stdout.

The functionality was originally intended to work inside tethereal,
but, as cfile.plist is not filled in there, it only can be invoked in this
crude way now.
Hence the patch file name.

Vassilii

> -----Original Message-----
> From: Khachaturov Vassilii 
> Sent: Tuesday, August 27, 2002 4:53 PM
> To: 'Guy Harris'
> Cc: 'Joerg Mayer'; 'ethereal-dev@xxxxxxxxxxxx'
> Subject: RE: [Ethereal-dev] is cfile.plist at all filled in 
> in tethereal?
> 
> 
> > > Or build the packet list and reuse the current code?
> > 
> > No.  That'd require it to make two passes over the capture file (one
> > pass to build the packet list, and one pass to process the 
> packets in
> > that list), and require extra memory to store the packet 
> > list; it'd also
> > mean that if we ever made Wiretap work, in 
> > sequential-access-only mode,
> > on pipes, you couldn't get the statistics when reading a pipe.
> 
> I agree, but unfortunately I can't commit to doing it soon right now.
> This whole change was triggered by my need to easily post-process
> the protocol stats, and what I have here locally satisfies me for now
> - although it's crude. I'm going to stick with what I have 
> here for the
> next couple of days (because I need to actually collect some stats
> and process them *now* from some captures here)! If I have time
> later, I'll return with a clean implementation. Meanwhile I'll
> submit the crude patch for archival purposes in case
> somebody else is in an urgent position like me.
> 
> If anybody else (Joerg?) wants to change the stats generation
> meanwhile I'll appreciate a note to avoid duplication of effort.
> 
> Patch will now be submitted in a separate email for archival with
> proper [PATCH] ... subject...


Attachment: print_stats.c
Description: Binary data

Attachment: print_stats.h
Description: Binary data

Attachment: test-in-ethereal.patch
Description: Binary data