From db77f058c92ee3d0b531c474093d6a3d1d81cd25 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 26 May 2014 12:17:41 +0000
Subject: [PATCH] Ensured no accidental protocol break can happen by renaming all ByteArrayBuilder.append*() methods and getting rid of all method overloading.

---
 opends/src/server/org/opends/server/replication/protocol/ModifyMsg.java |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/opends/src/server/org/opends/server/replication/protocol/ModifyMsg.java b/opends/src/server/org/opends/server/replication/protocol/ModifyMsg.java
index 33ec848..e37158e 100644
--- a/opends/src/server/org/opends/server/replication/protocol/ModifyMsg.java
+++ b/opends/src/server/org/opends/server/replication/protocol/ModifyMsg.java
@@ -176,7 +176,7 @@
   public byte[] getBytes_V1()
   {
     final ByteArrayBuilder builder = encodeHeader_V1(MSG_TYPE_MODIFY_V1);
-    builder.append(encodedMods);
+    builder.appendByteArray(encodedMods);
     return builder.toByteArray();
   }
 
@@ -186,7 +186,7 @@
   {
     final ByteArrayBuilder builder =
         encodeHeader(MSG_TYPE_MODIFY, ProtocolVersion.REPLICATION_PROTOCOL_V3);
-    builder.append(encodedMods);
+    builder.appendByteArray(encodedMods);
     return builder.toByteArray();
   }
 
@@ -196,10 +196,10 @@
   {
     final ByteArrayBuilder builder =
         encodeHeader(MSG_TYPE_MODIFY, protocolVersion);
-    builder.appendUTF8(encodedMods.length);
-    builder.append(encodedMods);
-    builder.appendUTF8(encodedEclIncludes.length);
-    builder.append(encodedEclIncludes);
+    builder.appendIntUTF8(encodedMods.length);
+    builder.appendByteArray(encodedMods);
+    builder.appendIntUTF8(encodedEclIncludes.length);
+    builder.appendByteArray(encodedEclIncludes);
     return builder.toByteArray();
   }
 

--
Gitblit v1.10.0