cli: reject team.config from files with '\0'

The team-config must be valid utf-8. First of all, JSON
is also defined for other unicode encodings, but libjansson
can only handle utf-8. So, just require that.

A file with a '\0' truncates part of the file and is thus
invalid.

1 files changed, 6 insertions, 0 deletions

diff --git a/clients/cli/common.c b/clients/cli/common.c
index 22e750475b..efe41cc6e4 100644
--- a/clients/cli/common.c
+++ b/clients/cli/common.c
@@ -871,6 +871,12 @@ nmc_team_check_config (const char *config, char **out_config, GError **error)
 				return FALSE;
 			}
 		} else {
+			if (c_len != strlen (contents)) {
+				g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
+				             _("team config file '%s' contains non-valid utf-8"),
+				             config);
+				return FALSE;
+			}
 			filename = config;
 			config = config_clone = g_steal_pointer (&contents);
 		}