You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2010/11/03 03:28:36 UTC
svn commit: r1030315 -
/cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java
Author: jbellis
Date: Wed Nov 3 02:28:35 2010
New Revision: 1030315
URL: http://svn.apache.org/viewvc?rev=1030315&view=rev
Log:
add assert to prevent garbage column writes. patch by jbellis
Modified:
cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java
Modified: cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java
URL: http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java?rev=1030315&r1=1030314&r2=1030315&view=diff
==============================================================================
--- cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java (original)
+++ cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/db/ColumnSerializer.java Wed Nov 3 02:28:35 2010
@@ -21,20 +21,27 @@ package org.apache.cassandra.db;
*/
-import java.io.*;
+import java.io.DataInput;
+import java.io.DataOutput;
+import java.io.IOException;
+import java.nio.ByteBuffer;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.cassandra.io.ICompactSerializer2;
import org.apache.cassandra.utils.FBUtilities;
-import java.nio.ByteBuffer;
-
public class ColumnSerializer implements ICompactSerializer2<IColumn>
{
+ private static final Logger logger = LoggerFactory.getLogger(ColumnSerializer.class);
+
public final static int DELETION_MASK = 0x01;
public final static int EXPIRATION_MASK = 0x02;
public void serialize(IColumn column, DataOutput dos)
{
+ assert column.name().remaining() > 0;
FBUtilities.writeShortByteArray(column.name(), dos);
try
{