Fixed compiler warnings10.6-monty

- Usage of strncpy that didn't set an end \0
- Initialized some variables that caused compiler warnings
- Increased buffer that caused warnings from strncpy

One warning from InnoDB, others from Connect.

6 files changed, 19 insertions, 20 deletions

diff --git a/storage/connect/filamdbf.cpp b/storage/connect/filamdbf.cpp
index c8571cea559..af121470d48 100644
--- a/storage/connect/filamdbf.cpp
+++ b/storage/connect/filamdbf.cpp
@@ -732,7 +732,7 @@ bool DBFFAM::AllocateBuffer(PGLOBAL g)
               return true;
             } // endswitch c
       
-          strncpy(descp->Name, cdp->GetName(), 11);
+          *strncpy(descp->Name, cdp->GetName(), sizeof(descp->Name)-1)= 0;
           descp->Type = c;
           descp->Length = (uchar)cdp->GetLong();
           } // endif Flags
diff --git a/storage/connect/jsonudf.cpp b/storage/connect/jsonudf.cpp
index c633993863c..73ab4a8dfde 100644
--- a/storage/connect/jsonudf.cpp
+++ b/storage/connect/jsonudf.cpp
@@ -4758,7 +4758,7 @@ char *jbin_array(UDF_INIT *initid, UDF_ARGS *args, char *result,
 			bsp = NULL;
 
 		if (!bsp && (bsp = JbinAlloc(g, args, initid->max_length, NULL)))
-			strncpy(bsp->Msg, g->Message, BMX);
+			*strncpy(bsp->Msg, g->Message, BMX-1)= 0;
 
 		// Keep result of constant function
 		g->Xchk = (initid->const_item) ? bsp : NULL;
@@ -4829,7 +4829,7 @@ char *jbin_array_add_values(UDF_INIT *initid, UDF_ARGS *args, char *result,
 
 		} else
 			if ((bsp = JbinAlloc(g, args, initid->max_length, NULL)))
-				strncpy(bsp->Msg, g->Message, BMX);
+				*strncpy(bsp->Msg, g->Message, BMX-1)= 0;
 
 		// Keep result of constant function
 		g->Xchk = (initid->const_item) ? bsp : NULL;
@@ -5051,7 +5051,7 @@ char *jbin_object(UDF_INIT *initid, UDF_ARGS *args, char *result,
 
 		} else
 			if ((bsp = JbinAlloc(g, args, initid->max_length, NULL)))
-				strncpy(bsp->Msg, g->Message, BMX);
+				*strncpy(bsp->Msg, g->Message, BMX-1)= 0;
 
 		// Keep result of constant function
 		g->Xchk = (initid->const_item) ? bsp : NULL;
@@ -5107,7 +5107,7 @@ char *jbin_object_nonull(UDF_INIT *initid, UDF_ARGS *args, char *result,
 
 		} else
 			if ((bsp = JbinAlloc(g, args, initid->max_length, NULL)))
-				strncpy(bsp->Msg, g->Message, BMX);
+				*strncpy(bsp->Msg, g->Message, BMX-1)= 0;
 
 		// Keep result of constant function
 		g->Xchk = (initid->const_item) ? bsp : NULL;
@@ -5166,7 +5166,7 @@ char *jbin_object_key(UDF_INIT *initid, UDF_ARGS *args, char *result,
 
 		} else
 			if ((bsp = JbinAlloc(g, args, initid->max_length, NULL)))
-				strncpy(bsp->Msg, g->Message, BMX);
+				*strncpy(bsp->Msg, g->Message, BMX-1)= 0;
 
 		// Keep result of constant function
 		g->Xchk = (initid->const_item) ? bsp : NULL;
diff --git a/storage/connect/tabxml.cpp b/storage/connect/tabxml.cpp
index dcebe18dd36..f0dd2e4ca88 100644
--- a/storage/connect/tabxml.cpp
+++ b/storage/connect/tabxml.cpp
@@ -253,8 +253,7 @@ PQRYRES XMLColumns(PGLOBAL g, char *db, char *tab, PTOS topt, bool info)
      more:
       if (vp->atp) {
 				size_t z = sizeof(colname) - 1;
-        strncpy(colname, vp->atp->GetName(g), z);
-				colname[z] = 0;
+                                *strncpy(colname, vp->atp->GetName(g), z)= 0;
 				strncat(xcol->Name, colname, XLEN(xcol->Name));
 
         switch (vp->atp->GetText(g, buf, sizeof(buf))) {
@@ -275,7 +274,7 @@ PQRYRES XMLColumns(PGLOBAL g, char *db, char *tab, PTOS topt, bool info)
         if (tdp->Usedom && node->GetType() != 1)
           continue;
 
-        strncpy(colname, node->GetName(g), sizeof(colname));
+        *strncpy(colname, node->GetName(g), sizeof(colname)-1)= 0;
 				strncat(xcol->Name, colname, XLEN(xcol->Name));
 
         if (j)
diff --git a/storage/innobase/handler/i_s.cc b/storage/innobase/handler/i_s.cc
index fc719557e4f..14b4decb7f9 100644
--- a/storage/innobase/handler/i_s.cc
+++ b/storage/innobase/handler/i_s.cc
@@ -5127,7 +5127,7 @@ i_s_sys_tables_fill_table_stats(
 
 	while (rec) {
 		const char*	err_msg;
-		dict_table_t*	table_rec;
+		dict_table_t*	table_rec= 0;
 
 		mtr.commit();
 		/* Fetch the dict_table_t structure corresponding to
diff --git a/storage/mroonga/vendor/groonga/lib/ts/ts_expr_parser.c b/storage/mroonga/vendor/groonga/lib/ts/ts_expr_parser.c
index 10e6d2fcca7..7786c04659d 100644
--- a/storage/mroonga/vendor/groonga/lib/ts/ts_expr_parser.c
+++ b/storage/mroonga/vendor/groonga/lib/ts/ts_expr_parser.c
@@ -360,7 +360,7 @@ grn_ts_expr_parser_tokenize_start(grn_ctx *ctx, grn_ts_expr_parser *parser,
                                   grn_ts_str str, grn_ts_expr_token **token)
 {
   grn_ts_str token_str = { str.ptr, 0 };
-  grn_ts_expr_start_token *new_token;
+  grn_ts_expr_start_token *new_token= 0;
   grn_rc rc = grn_ts_expr_start_token_open(ctx, token_str, &new_token);
   if (rc != GRN_SUCCESS) {
     return rc;
@@ -375,7 +375,7 @@ grn_ts_expr_parser_tokenize_end(grn_ctx *ctx, grn_ts_expr_parser *parser,
                                 grn_ts_str str, grn_ts_expr_token **token)
 {
   grn_ts_str token_str = { str.ptr, 0 };
-  grn_ts_expr_end_token *new_token;
+  grn_ts_expr_end_token *new_token= 0;
   grn_rc rc = grn_ts_expr_end_token_open(ctx, token_str, &new_token);
   if (rc != GRN_SUCCESS) {
     return rc;
@@ -393,7 +393,7 @@ grn_ts_expr_parser_tokenize_number(grn_ctx *ctx, grn_ts_expr_parser *parser,
   grn_rc rc;
   grn_ts_int int_value;
   grn_ts_str token_str;
-  grn_ts_expr_const_token *new_token;
+  grn_ts_expr_const_token *new_token= 0;
 
   int_value = strtol(str.ptr, &end, 0);
   if ((end != str.ptr) && (*end != '.') && (*end != 'e')) {
@@ -442,7 +442,7 @@ grn_ts_expr_parser_tokenize_text(grn_ctx *ctx, grn_ts_expr_parser *parser,
   size_t i, n_escapes = 0;
   grn_rc rc;
   grn_ts_str token_str;
-  grn_ts_expr_const_token *new_token;
+  grn_ts_expr_const_token *new_token= 0;
   for (i = 1; i < str.size; i++) {
     if (str.ptr[i] == '\\') {
       i++;
@@ -504,7 +504,7 @@ grn_ts_expr_parser_tokenize_name(grn_ctx *ctx, grn_ts_expr_parser *parser,
   token_str.size = i;
 
   if (grn_ts_str_is_bool(token_str)) {
-    grn_ts_expr_const_token *new_token;
+    grn_ts_expr_const_token *new_token= 0;
     grn_rc rc = grn_ts_expr_const_token_open(ctx, token_str, &new_token);
     if (rc != GRN_SUCCESS) {
       return rc;
@@ -527,7 +527,7 @@ grn_ts_expr_parser_tokenize_bridge(grn_ctx *ctx, grn_ts_expr_parser *parser,
                                    grn_ts_str str, grn_ts_expr_token **token)
 {
   grn_ts_str token_str = { str.ptr, 1 };
-  grn_ts_expr_bridge_token *new_token;
+  grn_ts_expr_bridge_token *new_token= 0;
   grn_rc rc = grn_ts_expr_bridge_token_open(ctx, token_str, &new_token);
   if (rc != GRN_SUCCESS) {
     return rc;
@@ -543,7 +543,7 @@ grn_ts_expr_parser_tokenize_bracket(grn_ctx *ctx, grn_ts_expr_parser *parser,
                                     grn_ts_expr_token **token)
 {
   grn_ts_str token_str = { str.ptr, 1 };
-  grn_ts_expr_bracket_token *new_token;
+  grn_ts_expr_bracket_token *new_token= 0;
   grn_rc rc = grn_ts_expr_bracket_token_open(ctx, token_str, &new_token);
   if (rc != GRN_SUCCESS) {
     return rc;
@@ -567,7 +567,7 @@ grn_ts_expr_parser_tokenize_sign(grn_ctx *ctx, grn_ts_expr_parser *parser,
   grn_ts_op_type op_type;
   grn_ts_str token_str = { str.ptr, 1 };
   grn_ts_expr_token *prev_token = parser->tokens[parser->n_tokens - 1];
-  grn_ts_expr_op_token *new_token;
+  grn_ts_expr_op_token *new_token= 0;
   switch (prev_token->type) {
     case GRN_TS_EXPR_START_TOKEN:
     case GRN_TS_EXPR_OP_TOKEN: {
@@ -626,7 +626,7 @@ grn_ts_expr_parser_tokenize_op(grn_ctx *ctx, grn_ts_expr_parser *parser,
   grn_rc rc = GRN_SUCCESS;
   grn_ts_str token_str = str;
   grn_ts_op_type op_type;
-  grn_ts_expr_op_token *new_token;
+  grn_ts_expr_op_token *new_token= 0;
   switch (str.ptr[0]) {
     case '+': case '-': {
       return grn_ts_expr_parser_tokenize_sign(ctx, parser, str, token);
diff --git a/storage/mroonga/vendor/groonga/vendor/plugins/groonga-normalizer-mysql/normalizers/mysql.c b/storage/mroonga/vendor/groonga/vendor/plugins/groonga-normalizer-mysql/normalizers/mysql.c
index 989d6267940..6096fc22cbc 100644
--- a/storage/mroonga/vendor/groonga/vendor/plugins/groonga-normalizer-mysql/normalizers/mysql.c
+++ b/storage/mroonga/vendor/groonga/vendor/plugins/groonga-normalizer-mysql/normalizers/mysql.c
@@ -439,7 +439,7 @@ normalize(grn_ctx *ctx, grn_obj *string,
   normalized[normalized_length_in_bytes] = '\0';
 
   if (rest_length > 0) {
-    char buffer[SNIPPET_BUFFER_SIZE];
+    char buffer[SNIPPET_BUFFER_SIZE+1];
     GRN_PLUGIN_LOG(ctx, GRN_LOG_DEBUG,
                    "[normalizer][%s] failed to normalize at %u byte: %s",
                    normalizer_type_label,