Re: missing call to snmp_store_needed() for some MIBs

[Claus Klein]

Hi,

I found the problem and tried to fix it.



The snmpEnableAuthenTraps was never saved again.


I am confused why there are 2 implementations of snmp_mib*.{c,h}

 

It is not really clear to me why original code does not work, but at least with my patch it works again. Too, most snmp configure scalars does not trigger the snmp_store() when changed since watch handlers are used!

fix_snmp_enableauthentraps_store.patch
Description: Binary data

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

 the handlers instances of a common handler.


---
 README.agent-mibs                             |   6 +-
 agent/mibgroup/mibII.h                        |   2 +-
 agent/mibgroup/mibII/snmp_mib.h               |   1 +

 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:

Subject: [PATCH] CHANGES: snmpd: Unify all snmpd statistic variables and make git-svn-id: file:///home/hardaker/lib/sf-bkups/net-snmp-convert-svnrepo/trunk/net-snmp@17340 06827809-a52a-0410-b366-d66718629ded agent/mibgroup/mibII/snmp_mib_5_5.c | 109 +++++++++++++++++ +++++++++

I was wondering why the

system_mib,     (sys_contact, sys_location, ...)
snmp_mib,       (snmpEnableAuthenTraps)


does not call the snmp_store_needed() after commit the configuration.

Tested with current git source on branch V5-7-patches

and notification-log-mib (nlmConfigGlobalEntryLimit, nlmConfigGlobalAgeOut)

------------------------------------------------------------------------------
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