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

Reply via email to