If you are interested in the cvs, subscribe to ntop-dev - that's where the
commit messages are sent (and where you should be posted about ntop
development)
-----Burton 

-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of
Naman Latif
Sent: Friday, February 04, 2005 10:50 AM
To: [email protected]
Cc: Luca Deri
Subject: RE: [Ntop] NTOP 3.1 dies on Reset_Stats - PRQ5CVD8B

Any possibility that it is fixed in the latest CVS release ?

\\ Naman

-----Original Message-----
From: Burton Strauss [mailto:[EMAIL PROTECTED]
Sent: Friday, January 21, 2005 9:31 AM
To: [email protected]
Cc: 'Luca Deri'
Subject: RE: [Ntop] NTOP 3.1 dies on Reset_Stats - PRQ5CVD8B

That's it... The reset is freeing the structure

void resetDevice(int devIdx) {
...
  if(myGlobals.device[devIdx].netflowGlobals != NULL)
    free(myGlobals.device[devIdx].netflowGlobals);
  myGlobals.device[devIdx].netflowGlobals = NULL; ...
  createDeviceIpProtosList(devIdx);
} 

And the netFlow main loop isn't testing the pointer.  So the 1st reference
after the reset bombs.


Since Luca rewrote the plugin for 3.1 I'm not sure what the right fix really
is.  It may be as simple as zeroing the counters instead of freeing the
structure, or maybe he wants to split it into freeable and permanent pieces
or just whatever...


I'm going to bounce this to him...



-----Burton


_______________________________________________
Ntop mailing list
[email protected]
http://listgateway.unipi.it/mailman/listinfo/ntop

_______________________________________________
Ntop mailing list
[email protected]
http://listgateway.unipi.it/mailman/listinfo/ntop

Reply via email to