-
Patch
-
Resolution: Unresolved
-
Minor
-
None
-
JBossAS-5.1.0.GA
-
None
-
Low
RequestHandlerImpl.getValue() for includes logic for determining how to express a JMX attribute in terms of an SNMP type. It supports Long, Integer, and String, as well as values of the raw SnmpTimeTicks and SnmpObjectId types.
I think this should be widened, by removing the check for SnmpTimeTicks and SnmpObjectId, and replacing it with a single check for the SnmpSyntax interface. This would allow beans to return the full range of SNMP types. Also, an explicit check for null should be performed, and represented by the SnmpNull type.
Note that setValueFor() is already more restrictive than getValueFor(), so we wouldn't be introducing any new asymmetry.
Patch for RequestHandlerImpl.java to follow.