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