From 45eb21b1354b6925fc058f834f505a9699d1bbbe Mon Sep 17 00:00:00 2001
From: pgamba <pgamba@localhost>
Date: Wed, 10 Jun 2009 08:43:50 +0000
Subject: [PATCH] External Changelog - first step - related issues 495, 519
---
opends/src/server/org/opends/server/replication/protocol/ReplServerStartMsg.java | 12 ++++++++----
1 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/opends/src/server/org/opends/server/replication/protocol/ReplServerStartMsg.java b/opends/src/server/org/opends/server/replication/protocol/ReplServerStartMsg.java
index f1bdb60..0201146 100644
--- a/opends/src/server/org/opends/server/replication/protocol/ReplServerStartMsg.java
+++ b/opends/src/server/org/opends/server/replication/protocol/ReplServerStartMsg.java
@@ -22,7 +22,7 @@
* CDDL HEADER END
*
*
- * Copyright 2006-2008 Sun Microsystems, Inc.
+ * Copyright 2006-2009 Sun Microsystems, Inc.
*/
package org.opends.server.replication.protocol;
@@ -167,9 +167,13 @@
pos += length + 1;
}
- /*
- * read the ServerState
- */
+ // Read the ServerState
+ // Caution: ServerState MUST be the last field. Because ServerState can
+ // contain null character (string termination of sererid string ..) it
+ // cannot be decoded using getNextLength() like the other fields. The
+ // only way is to rely on the end of the input buffer : and that forces
+ // the ServerState to be the last. This should be changed and we want to
+ // have more than one ServerState field.
serverState = new ServerState(in, pos, in.length - 1);
} catch (UnsupportedEncodingException e)
{
--
Gitblit v1.10.0