Patch 3058388:
Changes in 'include' support in config file parsing seem to have broken
includeAllDisks. I now get the following error:
/etc/snmp/snmpd.conf: line 5: Warning: Unknown include token: includeAllDisks.
No disks are included. Looks like it was caused by changes to
snmplib/read_config.c to add config file inclusion. Attached patch seems
to fix it.
I'm actually wondering why we're not doing a full strcasecmp and why
we're using the 'n' version? Robert?
--- snmplib/read_config.c 2010-09-02 14:05:40.730618036 -0700
+++ snmplib/read_config.c.new 2010-09-02 14:05:27.385754910 -0700
@@ -820,7 +820,7 @@
*/
cptr = copy_nword(cptr, token, sizeof(token));
}
- } else if ((token[0] == 'i') && (strncasecmp(token,"include", 7
)==0)) {
+ } else if ((token[0] == 'i') && (strncasecmp(token,"include", 7
)==0) && (strncasecmp(token, "includeAllDisks", 15) != 0)) {
if ( strcasecmp( token, "include" )==0) {
if (when != PREMIB_CONFIG &&
!netsnmp_ds_get_boolean(NETSNMP_DS_LIBRARY_ID,
--
Wes Hardaker
Please mail all replies to net-snmp-coders@lists.sourceforge.net
------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:
Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
