[jboss-cvs] JBossCache/src/org/jboss/cache/loader/bdbje ...
Brian Stansberry
brian.stansberry at jboss.com
Mon Sep 18 17:39:48 EDT 2006
User: bstansberry
Date: 06/09/18 17:39:48
Modified: src/org/jboss/cache/loader/bdbje Tag:
Branch_JBossCache_1_4_0 BdbjeCacheLoader.java
Log:
[JBCACHE-769] Make a defensive copy of any map passed to put(Fqn, Map)
Revision Changes Path
No revision
No revision
1.11.2.1 +7 -1 JBossCache/src/org/jboss/cache/loader/bdbje/BdbjeCacheLoader.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: BdbjeCacheLoader.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/loader/bdbje/BdbjeCacheLoader.java,v
retrieving revision 1.11
retrieving revision 1.11.2.1
diff -u -b -r1.11 -r1.11.2.1
--- BdbjeCacheLoader.java 6 May 2006 10:01:54 -0000 1.11
+++ BdbjeCacheLoader.java 18 Sep 2006 21:39:48 -0000 1.11.2.1
@@ -33,7 +33,7 @@
*
* @author Mark Hayes May 16, 2004
* @author Bela Ban
- * @version $Id: BdbjeCacheLoader.java,v 1.11 2006/05/06 10:01:54 msurtani Exp $
+ * @version $Id: BdbjeCacheLoader.java,v 1.11.2.1 2006/09/18 21:39:48 bstansberry Exp $
*/
public class BdbjeCacheLoader implements CacheLoader {
@@ -436,6 +436,9 @@
private void doPut(Transaction txn, Fqn name, Map values)
throws Exception {
+ // JBCACHE-769 -- make a defensive copy
+ values = (values == null ? null : new HashMap(values));
+
/* To update-or-insert, try putNoOverwrite first, then a RMW cycle. */
DatabaseEntry dataEntry = makeDataEntry(values);
DatabaseEntry keyEntry = makeKeyEntry(name);
@@ -466,6 +469,9 @@
private void doPutErase(Transaction txn, Fqn name, Map values)
throws Exception {
+ // JBCACHE-769 -- make a defensive copy
+ values = (values == null ? null : new HashMap(values));
+
DatabaseEntry dataEntry = makeDataEntry(values);
DatabaseEntry keyEntry = makeKeyEntry(name);
Cursor cursor = cacheDb.openCursor(txn, null);
More information about the jboss-cvs-commits
mailing list