On Fri, 04 Mar 2005 10:27:56 +0000 Dave wrote: DS> In particular, you seem to have introduced a new routine DS> netsnmp_check_vb_range() which is basically a duplicate DS> of the previously netsnmp_check_vb_int_range() but without DS> a check that the value actually is an integer. DS> What's the purpose of this new routine?
To check ranges for non-integer varbinds. Or to check additional ranges, where the integer check has already been done. DS> I'm also not clear about the latest change to agent/snmp_vars.c DS> Why shouldn't the agent modules be initialised when running as DS> a subagent? Since these include stuff like 'init_agentx_config()', DS> I'd have thought this was necessary in order to handle the config DS> file correctly? DS> What have I missed? Good catch. I think the change was in response to someone noticing that sub-agents were complaining about lack of usm configuration. Previously, the agent module init was done in init_mib_modules in mib_modules.c (you put it there, stating "Not convinced this is the best place for the agent_module init routines to be called, but it'll do for now.", and thus didn't happen for sub-agents anyways. But it did cause the agent library to depend on the mib library, which is bad for sub-agents. This dependency is the other thing I was trying to fix. I had noticed the agentx config problem, and I think I have a work-around sitting in my local repository. But the correct solution is probably to break the agent modules into agent modules and sub-agent modules, and call the sub-agent modules in init_subagent. -- Robert Story; NET-SNMP Junkie Support: <http://www.net-snmp.org/> <irc://irc.freenode.net/#net-snmp> Archive: <http://sourceforge.net/mailarchive/forum.php?forum=net-snmp-coders> You are lost in a twisty maze of little standards, all different. ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click _______________________________________________ Net-snmp-coders mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
