thread-safety issue: move ByteBuffer construction to synchronized block

mumrah · Jul 5, 2013 · 509d34f · 509d34f
1 parent 38b7da3
commit 509d34f
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/uniq/UniqueId.java b/src/uniq/UniqueId.java
@@ -43,13 +43,13 @@ public byte[] getId() {
                 seq = 0;
             }
             seq++;
+            ByteBuffer bb = tlbb.get();
+            bb.rewind();
+            bb.putLong(time);
+            bb.put(node);
+            bb.putShort((short)seq);
+            return bb.array();    			
         }
-        ByteBuffer bb = tlbb.get();
-        bb.rewind();
-        bb.putLong(time);
-        bb.put(node);
-        bb.putShort((short)seq);
-        return bb.array();    
     }
 
     public String getStringId() {