On 08/30/2011 12:40 PM, Bart Van Assche wrote: > On Tue, Aug 30, 2011 at 10:24 AM, Jan Safranek <[email protected]> wrote: >> Problem is that you *must* dequeue all requests which are related to the >> closing AgentX session - somewhere deep in these requests there is >> pointer to the AgentX session and the session is being freed when closing. > > That became clear to me too ... Regarding your patch: are you aware > that it changes the return value of > netsnmp_remove_delegated_requests_for_session() such that it always > returns zero ?
It's called only on one place, without checking its return value... But yes, it's probably part of public API, so it might be better to move the loop outside this function and put it to close_agentx_session(). And there are even less possibilities to skip to end of the AgentX's subtree. Jan ------------------------------------------------------------------------------ Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free "Love Thy Logs" t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev _______________________________________________ Net-snmp-coders mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
