On Wed, Mar 17, 2010 at 10:09 AM, Dave Shield <[email protected]> wrote: > I've had a quick look at the code for winExtDLL.c > (in response to bug #2971257), and there's something > that confuses (/concerns) me. > > Within the var_winExtDLL handler, there's the usual > (for request = requests; request; request = request->next ) > loop, to work through all the relevant varbinds for this call. > So far, so good. > > But then immediately within this block is another loop, > starting from the 'requestvb' field of this first request > and walking through the 'next_variable' links. > > That's not something which a handler should normally > be concerned about - this list is really for the benefit > of the main agent processing code. > The handler should be working with the "requests" > list, and accessing each 'request->requestvb'
A fix for this issue is available and can be reviewed by inspecting the output of the following command: svn diff -r18309:18348 https://net-snmp.svn.sourceforge.net/svnroot/net-snmp/branches/V5-4-patches/net-snmp Does this issue count as a show-stopper, or is voting necessary to decide whether or not it should be included in the next 5.4 release candidate ? Bart. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Net-snmp-coders mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
