On Tue, 2005-10-18 at 06:58 -0700, Wes Hardaker wrote: > Dave> The proposed patch looks reasonably straightforward, so ought to > Dave> be safe enough. I'm not sure whether it introduces a memory leak > Dave> (should the PDU be freed or not?), but that's less of a problem than > Dave> an agent crash. > > Well, FYI, I had the free in there originally but it crashed in the > free so took it out.
That's pretty much what I expected. (The only doubt in my mind was over who put it in/took it out!) > It's one of those cases that I was sure it was > the right thing to do and now I simply feel insecure again. Oh, don't feel insecure, Wes - we love you really. Quick everybody - group hug..... :-) I'd suggest that the more important thing here is avoiding an agent crash. We can live with a minor memory leak in the short term. (We've done so plenty of times before!) Dave ------------------------------------------------------- This SF.Net email is sponsored by: Power Architecture Resource Center: Free content, downloads, discussions, and more. http://solutions.newsforge.com/ibmarch.tmpl _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
