On Fri, 7 Oct 2005 23:10:29 +0100 Dave wrote:
DS> > Did you intend to set up multiple conf tokens to map to the same value?
DS> 
DS> >     netsnmp_ds_register_config(ASN_OCTET_STR, "snmpd", "agentSecName",
DS> >     netsnmp_ds_register_config(ASN_OCTET_STR, "snmpd", "iquerySecName",
DS> 
DS> 
DS> Yes.
DS> "agentSecName" is for backward compatibility with previous releases.
DS> "iquerySecName" felt to be a more appropriate name.

Then I strongly suggest that some error checking be introduced to check for
both being used, and issue a deprecated warning for the old name.

DS> I also spotted a related change to the CVS code recently, whereby you
DS> seemed to provide a default value for this setting.  Care to explain why?

Sure. Because the code you checked in called get_ds_* for the pointer, but
didn't check if it was NULL before using it. I didn't have time to figure out
the correct fix, so I just made sure the ptr wouldn't be null. ;-)


-- 
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. 


-------------------------------------------------------
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

Reply via email to