wrong usage of snprintf

Project: http://git-wip-us.apache.org/repos/asf/trafodion/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafodion/commit/0ba13fb7
Tree: http://git-wip-us.apache.org/repos/asf/trafodion/tree/0ba13fb7
Diff: http://git-wip-us.apache.org/repos/asf/trafodion/diff/0ba13fb7

Branch: refs/heads/master
Commit: 0ba13fb78f1bdc4ba4798c2d28ddea95637bae57
Parents: 8414249
Author: Liu Ming <ovis_p...@sina.com>
Authored: Tue Jul 17 21:16:58 2018 -0400
Committer: Liu Ming <ovis_p...@sina.com>
Committed: Tue Jul 17 21:16:58 2018 -0400

----------------------------------------------------------------------
 core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafodion/blob/0ba13fb7/core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp
----------------------------------------------------------------------
diff --git a/core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp 
b/core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp
index 92b4928..e3735c8 100644
--- a/core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp
+++ b/core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp
@@ -10993,7 +10993,7 @@ CmpSeabaseDDL::setupHbaseOptions(ElemDDLHbaseOptions * 
hbaseOptionsClause,
           (HBASE_DATA_BLOCK_ENCODING_OPTION) == TRUE)
         {
           numHbaseOptions += 1;
-          snprintf(optionStr, 199, "DATA_BLOCK_ENCODING='%s'|", 
dataBlockEncoding.data());
+          snprintf(optionStr, 200, "DATA_BLOCK_ENCODING='%s'|", 
dataBlockEncoding.data());
           hbaseOptionsStr += optionStr;
         }
     }
@@ -11007,7 +11007,7 @@ CmpSeabaseDDL::setupHbaseOptions(ElemDDLHbaseOptions * 
hbaseOptionsClause,
           (HBASE_MEMSTORE_FLUSH_SIZE_OPTION) == TRUE)
         {
           numHbaseOptions += 1;
-          snprintf(optionStr, 199, "MEMSTORE_FLUSH_SIZE='%s'|", 
flushSize.data());
+          snprintf(optionStr, 200, "MEMSTORE_FLUSH_SIZE='%s'|", 
flushSize.data());
           hbaseOptionsStr += optionStr;
         }
     }
@@ -11021,7 +11021,7 @@ CmpSeabaseDDL::setupHbaseOptions(ElemDDLHbaseOptions * 
hbaseOptionsClause,
           (HBASE_COMPRESSION_OPTION) == TRUE)
         {
           numHbaseOptions += 1;
-          snprintf(optionStr, 199, "COMPRESSION='%s'|", compression.data());
+          snprintf(optionStr, 200, "COMPRESSION='%s'|", compression.data());
           hbaseOptionsStr += optionStr;
         }
     }
@@ -11036,7 +11036,7 @@ CmpSeabaseDDL::setupHbaseOptions(ElemDDLHbaseOptions * 
hbaseOptionsClause,
     hco += "HBASE_OPTIONS=>";
 
     char hbaseOptionsNumCharStr[HBASE_OPTION_MAX_INTEGER_LENGTH];
-    snprintf(hbaseOptionsNumCharStr, HBASE_OPTION_MAX_INTEGER_LENGTH-1, 
"%04d", numHbaseOptions);
+    snprintf(hbaseOptionsNumCharStr, HBASE_OPTION_MAX_INTEGER_LENGTH, "%04d", 
numHbaseOptions);
     hco += hbaseOptionsNumCharStr;
 
     hco += hbaseOptionsStr.data();

Reply via email to