On Fri, 2013-03-15 at 20:03 +0100, Claus Klein wrote:
> Hi,
> 
> I found the problem and tried to fix it.
> 
> It is not really clear to me why original code does not work, but at  
> least with my patch it works again.
> 
> The snmpEnableAuthenTraps was never saved again.
> Too, most snmp configure scalars does not trigger the snmp_store()  
> when changed since watch handlers are used!
> 
> I am confused why there are 2 implementations of snmp_mib*.{c,h}
>   
>  It seem that this the error is there since V5.5 with this patch: From 
> 32eec65467164c41995feeabff18f934be890674 Mon Sep 17 00:00:00 2001 From: 
> Magnus Fromreide <ma...@users.sourceforge.net> Date: Sat, 3 Jan 2009 00:19:49 
> +0000 Subject: [PATCH] CHANGES: snmpd: Unify all snmpd statistic variables 
> and make the handlers instances of a common handler. git-svn-id: 
> file:///home/hardaker/lib/sf-bkups/net-snmp-convert-svnrepo/trunk/net-snmp@17340
>  06827809-a52a-0410-b366-d66718629ded --- README.agent-mibs | 6 +- 
> agent/mibgroup/mibII.h | 2 +- agent/mibgroup/mibII/snmp_mib.h | 1 + 
> agent/mibgroup/mibII/snmp_mib_5_5.c | 109 +++++++++++++++++ +++++++++ 
> agent/mibgroup/mibII/snmp_mib_5_5.h | 5 ++ 
> agent/mibgroup/snmpv3/snmpMPDStats_5_5.c | 24 ++---- 
> agent/mibgroup/snmpv3/snmpMPDStats_5_5.h | 1 + 
> agent/mibgroup/snmpv3/usmStats_5_5.c | 23 ++---- 
> agent/mibgroup/snmpv3/usmStats_5_5.h | 1 + agent/mibgroup/target.h | 2 +- 
> agent/mibgroup/target/target_counters.h | 2 + 
> agent/mibgroup/target/target_counters_5_5.c | 27 +++++++ 
> agent/mibgroup/target/target_counters_5_5.h | 4 + 
> agent/mibgroup/utilities/snmp_get_statistic.c | 44 +++++++++++ 
> agent/mibgroup/utilities/snmp_get_statistic.h | 20 +++++ 15 files changed, 
> 229 insertions(+), 42 deletions(-) On 11.03.2013, at 23:00, Claus Klein 
> wrote: > I was wondering why the > > system_mib,   (sys_contact, 
> sys_location, ...) > snmp_mib,    (snmpEnableAuthenTraps) > and 
> notification-log-mib (nlmConfigGlobalEntryLimit, > nlmConfigGlobalAgeOut) > > 
> does not call the snmp_store_needed() after commit the configuration. > > 
> Tested with current git source on branch V5-7-patches >

Thanks for reporting this bug.

I am - slowly - looking into it.

Your patch for the issue is sadly not the right one as it attacks at the
wrong level, but thanks for your efforts and a clear bug report.

/MF


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to