[jboss-cvs] JBossAS SVN: r110472 - branches/snmp4j-int/varia/src/main/java/org/jboss/jmx/adaptor/snmp/agent.

[jboss-cvs-commits at lists.jboss.org]

Author: thauser at redhat.com
Date: 2011-01-27 11:06:35 -0500 (Thu, 27 Jan 2011)
New Revision: 110472

Modified:
   branches/snmp4j-int/varia/src/main/java/org/jboss/jmx/adaptor/snmp/agent/RequestHandlerImpl.java
Log:
implement object vs. instance checking. 

Modified: branches/snmp4j-int/varia/src/main/java/org/jboss/jmx/adaptor/snmp/agent/RequestHandlerImpl.java
===================================================================
--- branches/snmp4j-int/varia/src/main/java/org/jboss/jmx/adaptor/snmp/agent/RequestHandlerImpl.java	2011-01-27 14:56:12 UTC (rev 110471)
+++ branches/snmp4j-int/varia/src/main/java/org/jboss/jmx/adaptor/snmp/agent/RequestHandlerImpl.java	2011-01-27 16:06:35 UTC (rev 110472)
@@ -504,7 +504,8 @@
 					return response;
 				}
 				catch (Exception e){
-					log.debug("snmpReceivedSet: catastrophe!!! General variable validation error." + e);
+					log.debug("snmpReceivedSet: catastrophe!!! General variable validation error." + " " + e);
+					e.printStackTrace();
 					undoSets(modified);
 					makeErrorPdu(response, pdu, counter, PDU.genErr);
 					return response;
@@ -714,12 +715,13 @@
 	
 	/** test method **/
 	private void addObjectEntry(String oid){
-		if (objectKeys.contains(oid))
+		OID coid = new OID(oid);
+		if (objectKeys.contains(coid))
 			log.debug("duplicate object " + oid + SKIP_ENTRY);
 		
 		if (oid == null)
 			log.debug("null oid for object");
-		objectKeys.add(oid);
+		objectKeys.add(coid);
 				
 	}