On 8 September 2010 16:57, Dave Shield <d.t.shi...@liverpool.ac.uk> wrote:
> As I recall:
>
>   - the initial code just looked for the config token "include" and
>      ignored everything else.   Hence it used an exact 'strcmp' comparison.
>
>  -  my re-working looked for  "includeDir", "includeFile" and "includeSearch"
>      (plus the original "include"), and hence uses a prefix search (strncmp)
>      as an initial filter
>
> I must admit that I'd forgotten about the possibility of other config tokens
> that started with the prefix "include".   Those should really be passed on to
> the appropriate MIB module config handlers.
>
> I haven't had a chance to test this, but my suspicion is that dropping the
> block that warns about an "unknown include token" should be sufficient
> here.   That's probably safer (at least in the long run) than hardcoding an
> exception for "includeAllDisks".   We've no idea what other "includeXxx"
> config tokens may have been defined in other (user) code.


The details were a little more fiddly than this implies,
but the basic idea proved accurate.   The current 5.6.x
and main trunk should now accept *any* registered
tokens of the form "includeXXX",  and issue the usual
warnings for incomplete or unrecognised directives.

Dave

------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store 
http://p.sf.net/sfu/nokia-dev2dev
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to