We are currently experiencing downtime impacting viewing & cloning the Mesa repo, and some GitLab pages returning 503. Please see #freedesktop on IRC for more updates.

Commit 5d3736ac authored by Thomas Haller's avatar Thomas Haller

cli: drop nmc_strsplit_set()

In most cases, it copies the entire strv needlessly.
We can do better.

Also, the max_tokens argument is handled wrongly (albeit
not used anywhere anymore).
parent 88699435
...@@ -1555,18 +1555,18 @@ get_invisible_active_connections (NmCli *nmc) ...@@ -1555,18 +1555,18 @@ get_invisible_active_connections (NmCli *nmc)
static GArray * static GArray *
parse_preferred_connection_order (const char *order, GError **error) parse_preferred_connection_order (const char *order, GError **error)
{ {
char **strv, **iter; gs_free const char **strv = NULL;
const char *const*iter;
const char *str; const char *str;
GArray *order_arr; GArray *order_arr;
NmcSortOrder val; NmcSortOrder val;
gboolean inverse, unique; gboolean inverse, unique;
int i; int i;
strv = nmc_strsplit_set (order, ":", -1); strv = nm_utils_strsplit_set (order, ":");
if (!strv || !*strv) { if (!strv) {
g_set_error (error, NMCLI_ERROR, 0, g_set_error (error, NMCLI_ERROR, 0,
_("incorrect string '%s' of '--order' option"), order); _("incorrect string '%s' of '--order' option"), order);
g_strfreev (strv);
return NULL; return NULL;
} }
...@@ -1608,7 +1608,6 @@ parse_preferred_connection_order (const char *order, GError **error) ...@@ -1608,7 +1608,6 @@ parse_preferred_connection_order (const char *order, GError **error)
g_array_append_val (order_arr, val); g_array_append_val (order_arr, val);
} }
g_strfreev (strv);
return order_arr; return order_arr;
} }
...@@ -2261,48 +2260,50 @@ activate_connection_cb (GObject *client, GAsyncResult *result, gpointer user_dat ...@@ -2261,48 +2260,50 @@ activate_connection_cb (GObject *client, GAsyncResult *result, gpointer user_dat
static GHashTable * static GHashTable *
parse_passwords (const char *passwd_file, GError **error) parse_passwords (const char *passwd_file, GError **error)
{ {
GHashTable *pwds_hash; gs_unref_hashtable GHashTable *pwds_hash = NULL;
char *contents = NULL; gs_free char *contents = NULL;
gsize len = 0; gsize len = 0;
GError *local_err = NULL; GError *local_err = NULL;
char **lines, **iter; gs_free const char **strv = NULL;
const char *const*iter;
char *pwd_spec, *pwd, *prop; char *pwd_spec, *pwd, *prop;
const char *setting; const char *setting;
pwds_hash = g_hash_table_new_full (nm_str_hash, g_str_equal, g_free, g_free); pwds_hash = g_hash_table_new_full (nm_str_hash, g_str_equal, g_free, g_free);
if (!passwd_file) if (!passwd_file)
return pwds_hash; return g_steal_pointer (&pwds_hash);
/* Read the passwords file */ /* Read the passwords file */
if (!g_file_get_contents (passwd_file, &contents, &len, &local_err)) { if (!g_file_get_contents (passwd_file, &contents, &len, &local_err)) {
g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT, g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
_("failed to read passwd-file '%s': %s"), _("failed to read passwd-file '%s': %s"),
passwd_file, local_err->message); passwd_file, local_err->message);
g_error_free (local_err); g_error_free (local_err);
g_hash_table_destroy (pwds_hash);
return NULL; return NULL;
} }
lines = nmc_strsplit_set (contents, "\r\n", -1); strv = nm_utils_strsplit_set (contents, "\r\n");
for (iter = lines; *iter; iter++) { for (iter = strv; *iter; iter++) {
pwd = strchr (*iter, ':'); gs_free char *iter_s = g_strdup (*iter);
pwd = strchr (iter_s, ':');
if (!pwd) { if (!pwd) {
g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT, g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
_("missing colon in 'password' entry '%s'"), *iter); _("missing colon in 'password' entry '%s'"), *iter);
goto failure; return NULL;
} }
*(pwd++) = '\0'; *(pwd++) = '\0';
prop = strchr (*iter, '.'); prop = strchr (iter_s, '.');
if (!prop) { if (!prop) {
g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT, g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
_("missing dot in 'password' entry '%s'"), *iter); _("missing dot in 'password' entry '%s'"), *iter);
goto failure; return NULL;
} }
*(prop++) = '\0'; *(prop++) = '\0';
setting = *iter; setting = iter_s;
while (g_ascii_isspace (*setting)) while (g_ascii_isspace (*setting))
setting++; setting++;
/* Accept wifi-sec or wifi instead of cumbersome '802-11-wireless-security' */ /* Accept wifi-sec or wifi instead of cumbersome '802-11-wireless-security' */
...@@ -2311,21 +2312,13 @@ parse_passwords (const char *passwd_file, GError **error) ...@@ -2311,21 +2312,13 @@ parse_passwords (const char *passwd_file, GError **error)
if (nm_setting_lookup_type (setting) == G_TYPE_INVALID) { if (nm_setting_lookup_type (setting) == G_TYPE_INVALID) {
g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT, g_set_error (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
_("invalid setting name in 'password' entry '%s'"), setting); _("invalid setting name in 'password' entry '%s'"), setting);
goto failure; return NULL;
} }
pwd_spec = g_strdup_printf ("%s.%s", setting, prop); pwd_spec = g_strdup_printf ("%s.%s", setting, prop);
g_hash_table_insert (pwds_hash, pwd_spec, g_strdup (pwd)); g_hash_table_insert (pwds_hash, pwd_spec, g_strdup (pwd));
} }
g_strfreev (lines); return g_steal_pointer (&pwds_hash);
g_free (contents);
return pwds_hash;
failure:
g_strfreev (lines);
g_free (contents);
g_hash_table_destroy (pwds_hash);
return NULL;
} }
......
...@@ -313,16 +313,18 @@ _set_fcn_precheck_connection_secondaries (const char *value, ...@@ -313,16 +313,18 @@ _set_fcn_precheck_connection_secondaries (const char *value,
{ {
const GPtrArray *connections; const GPtrArray *connections;
NMConnection *con; NMConnection *con;
gs_free const char **strv0 = NULL;
gs_strfreev char **strv = NULL; gs_strfreev char **strv = NULL;
char **iter; char **iter;
gboolean modified; gboolean modified = FALSE;
strv = nmc_strsplit_set (value, " \t,", 0); strv0 = nm_utils_strsplit_set (value, " \t,");
if (!strv) if (!strv0)
return TRUE; return TRUE;
connections = nm_client_get_connections (nm_cli.client); connections = nm_client_get_connections (nm_cli.client);
strv = g_strdupv ((char **) strv0);
for (iter = strv; *iter; iter++) { for (iter = strv; *iter; iter++) {
if (nm_utils_is_uuid (*iter)) { if (nm_utils_is_uuid (*iter)) {
con = nmc_find_connection (connections, "uuid", *iter, NULL, FALSE); con = nmc_find_connection (connections, "uuid", *iter, NULL, FALSE);
......
...@@ -603,9 +603,14 @@ int ...@@ -603,9 +603,14 @@ int
nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote, nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote,
char ***argv, int *argc) char ***argv, int *argc)
{ {
gs_free const char **arr0 = NULL;
char **arr; char **arr;
arr = nmc_strsplit_set (line ? line : "", delim ? delim : " \t", 0); arr0 = nm_utils_strsplit_set (line ?: "", delim ?: " \t");
if (!arr0)
arr = g_new0 (char *, 1);
else
arr = g_strdupv ((char **) arr0);
if (unquote) { if (unquote) {
int i = 0; int i = 0;
...@@ -613,7 +618,7 @@ nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote, ...@@ -613,7 +618,7 @@ nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote,
size_t l; size_t l;
const char *quotes = "\"'"; const char *quotes = "\"'";
while (arr && arr[i]) { while (arr[i]) {
s = arr[i]; s = arr[i];
l = strlen (s); l = strlen (s);
if (l >= 2) { if (l >= 2) {
...@@ -628,7 +633,6 @@ nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote, ...@@ -628,7 +633,6 @@ nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquote,
*argv = arr; *argv = arr;
*argc = g_strv_length (arr); *argc = g_strv_length (arr);
return 0; return 0;
} }
......
...@@ -53,7 +53,6 @@ int nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquo ...@@ -53,7 +53,6 @@ int nmc_string_to_arg_array (const char *line, const char *delim, gboolean unquo
char ***argv, int *argc); char ***argv, int *argc);
const char *nmc_string_is_valid (const char *input, const char **allowed, GError **error); const char *nmc_string_is_valid (const char *input, const char **allowed, GError **error);
char * nmc_util_strv_for_display (const char *const*strv, gboolean brackets); char * nmc_util_strv_for_display (const char *const*strv, gboolean brackets);
char **nmc_strsplit_set (const char *str, const char *delimiter, int max_tokens);
int nmc_string_screen_width (const char *start, const char *end); int nmc_string_screen_width (const char *start, const char *end);
void set_val_str (NmcOutputField fields_array[], guint32 index, char *value); void set_val_str (NmcOutputField fields_array[], guint32 index, char *value);
void set_val_strc (NmcOutputField fields_array[], guint32 index, const char *value); void set_val_strc (NmcOutputField fields_array[], guint32 index, const char *value);
......
...@@ -173,18 +173,6 @@ finish: ...@@ -173,18 +173,6 @@ finish:
return ret; return ret;
} }
/*
* Wrapper function for g_strsplit_set() that removes empty strings
* from the vector as they are not useful in most cases.
*/
char **
nmc_strsplit_set (const char *str, const char *delimiter, int max_tokens)
{
/* remove empty strings */
return _nm_utils_strv_cleanup (g_strsplit_set (str, delimiter, max_tokens),
FALSE, TRUE, FALSE);
}
gboolean gboolean
matches (const char *cmd, const char *pattern) matches (const char *cmd, const char *pattern)
{ {
......
...@@ -32,8 +32,6 @@ typedef enum { ...@@ -32,8 +32,6 @@ typedef enum {
const char *nmc_string_is_valid (const char *input, const char **allowed, GError **error); const char *nmc_string_is_valid (const char *input, const char **allowed, GError **error);
char **nmc_strsplit_set (const char *str, const char *delimiter, int max_tokens);
gboolean nmc_string_to_uint (const char *str, gboolean nmc_string_to_uint (const char *str,
gboolean range_check, gboolean range_check,
unsigned long int min, unsigned long int min,
......
...@@ -117,7 +117,6 @@ _parse_ip_route (int family, ...@@ -117,7 +117,6 @@ _parse_ip_route (int family,
GError **error) GError **error)
{ {
const int MAX_PREFIX = (family == AF_INET) ? 32 : 128; const int MAX_PREFIX = (family == AF_INET) ? 32 : 128;
char *plen = NULL;
const char *next_hop = NULL; const char *next_hop = NULL;
const char *canon_dest; const char *canon_dest;
int prefix; int prefix;
...@@ -125,9 +124,11 @@ _parse_ip_route (int family, ...@@ -125,9 +124,11 @@ _parse_ip_route (int family,
GError *local = NULL; GError *local = NULL;
gint64 metric = -1; gint64 metric = -1;
guint i; guint i;
gs_strfreev char **routev = NULL; gs_free const char **routev = NULL;
gs_free char *str_clean = NULL; gs_free char *str_clean = NULL;
char *dest; gs_free char *dest_clone = NULL;
const char *dest;
const char *plen;
gs_unref_hashtable GHashTable *attrs = NULL; gs_unref_hashtable GHashTable *attrs = NULL;
GHashTable *tmp_attrs; GHashTable *tmp_attrs;
#define ROUTE_SYNTAX _("The valid syntax is: 'ip[/prefix] [next-hop] [metric] [attribute=val]... [,ip[/prefix] ...]'") #define ROUTE_SYNTAX _("The valid syntax is: 'ip[/prefix] [next-hop] [metric] [attribute=val]... [,ip[/prefix] ...]'")
...@@ -137,8 +138,8 @@ _parse_ip_route (int family, ...@@ -137,8 +138,8 @@ _parse_ip_route (int family,
nm_assert (!error || !*error); nm_assert (!error || !*error);
str_clean = g_strstrip (g_strdup (str)); str_clean = g_strstrip (g_strdup (str));
routev = nmc_strsplit_set (str_clean, " \t", 0); routev = nm_utils_strsplit_set (str_clean, " \t");
if (!routev || !routev[0]) { if (!routev) {
g_set_error (error, 1, 0, g_set_error (error, 1, 0,
"'%s' is not valid. %s", "'%s' is not valid. %s",
str, ROUTE_SYNTAX); str, ROUTE_SYNTAX);
...@@ -147,8 +148,13 @@ _parse_ip_route (int family, ...@@ -147,8 +148,13 @@ _parse_ip_route (int family,
dest = routev[0]; dest = routev[0];
plen = strchr (dest, '/'); /* prefix delimiter */ plen = strchr (dest, '/'); /* prefix delimiter */
if (plen) if (plen) {
*plen++ = '\0'; dest_clone = g_strdup (dest);
plen = &dest_clone[plen - dest];
dest = dest_clone;
*((char *) plen) = '\0';
plen++;
}
prefix = MAX_PREFIX; prefix = MAX_PREFIX;
if (plen) { if (plen) {
if ((prefix = _nm_utils_ascii_str_to_int64 (plen, 10, 1, MAX_PREFIX, -1)) == -1) { if ((prefix = _nm_utils_ascii_str_to_int64 (plen, 10, 1, MAX_PREFIX, -1)) == -1) {
...@@ -295,7 +301,7 @@ static NMTeamLinkWatcher * ...@@ -295,7 +301,7 @@ static NMTeamLinkWatcher *
_parse_team_link_watcher (const char *str, _parse_team_link_watcher (const char *str,
GError **error) GError **error)
{ {
gs_strfreev char **watcherv = NULL; gs_free const char **watcherv = NULL;
gs_free char *str_clean = NULL; gs_free char *str_clean = NULL;
guint i; guint i;
gs_free const char *name = NULL; gs_free const char *name = NULL;
...@@ -308,17 +314,17 @@ _parse_team_link_watcher (const char *str, ...@@ -308,17 +314,17 @@ _parse_team_link_watcher (const char *str,
nm_assert (!error || !*error); nm_assert (!error || !*error);
str_clean = g_strstrip (g_strdup (str)); str_clean = g_strstrip (g_strdup (str));
watcherv = nmc_strsplit_set (str_clean, " \t", 0); watcherv = nm_utils_strsplit_set (str_clean, " \t");
if (!watcherv || !watcherv[0]) { if (!watcherv) {
g_set_error (error, 1, 0, "'%s' is not valid", str); g_set_error (error, 1, 0, "'%s' is not valid", str);
return NULL; return NULL;
} }
for (i = 0; watcherv[i]; i++) { for (i = 0; watcherv[i]; i++) {
gs_strfreev char **pair = NULL; gs_free const char **pair = NULL;
pair = nmc_strsplit_set (watcherv[i], "=", 0); pair = nm_utils_strsplit_set (watcherv[i], "=");
if (!pair[0]) { if (!pair) {
g_set_error (error, 1, 0, "'%s' is not valid: %s", watcherv[i], g_set_error (error, 1, 0, "'%s' is not valid: %s", watcherv[i],
"properties should be specified as 'key=value'"); "properties should be specified as 'key=value'");
return NULL; return NULL;
...@@ -1170,22 +1176,21 @@ _set_fcn_gobject_mac (ARGS_SET_FCN) ...@@ -1170,22 +1176,21 @@ _set_fcn_gobject_mac (ARGS_SET_FCN)
static gboolean static gboolean
_set_fcn_gobject_secret_flags (ARGS_SET_FCN) _set_fcn_gobject_secret_flags (ARGS_SET_FCN)
{ {
char **strv = NULL, **iter; gs_free const char **strv = NULL;
const char **iter;
unsigned long flags = 0, val_int; unsigned long flags = 0, val_int;
g_return_val_if_fail (error == NULL || *error == NULL, FALSE); nm_assert (!error || !*error);
strv = nmc_strsplit_set (value, " \t,", 0); strv = nm_utils_strsplit_set (value, " \t,");
for (iter = strv; iter && *iter; iter++) { for (iter = strv; iter && *iter; iter++) {
if (!nmc_string_to_uint (*iter, TRUE, 0, ALL_SECRET_FLAGS, &val_int)) { if (!nmc_string_to_uint (*iter, TRUE, 0, ALL_SECRET_FLAGS, &val_int)) {
g_set_error (error, 1, 0, _("'%s' is not a valid flag number; use <0-%d>"), g_set_error (error, 1, 0, _("'%s' is not a valid flag number; use <0-%d>"),
*iter, ALL_SECRET_FLAGS); *iter, ALL_SECRET_FLAGS);
g_strfreev (strv);
return FALSE; return FALSE;
} }
flags += val_int; flags += val_int;
} }
g_strfreev (strv);
/* Validate the flags number */ /* Validate the flags number */
if (flags > ALL_SECRET_FLAGS) { if (flags > ALL_SECRET_FLAGS) {
...@@ -1572,18 +1577,19 @@ vpn_data_item (const char *key, const char *value, gpointer user_data) ...@@ -1572,18 +1577,19 @@ vpn_data_item (const char *key, const char *value, gpointer user_data)
const char **valid_strv, \ const char **valid_strv, \
GError **error) \ GError **error) \
{ \ { \
char **strv = NULL, **iter; \ gs_free const char **strv = NULL; \
gsize i; \
const char *item; \ const char *item; \
g_return_val_if_fail (error == NULL || *error == NULL, FALSE); \ nm_assert (!error || !*error); \
strv = nmc_strsplit_set (value, " \t,", 0); \ strv = nm_utils_strsplit_set (value, " \t,"); \
for (iter = strv; iter && *iter; iter++) { \ if (strv) { \
if (!(item = nmc_string_is_valid (g_strstrip (*iter), valid_strv, error))) { \ for (i = 0; strv[i]; i++) { \
g_strfreev (strv); \ if (!(item = nmc_string_is_valid (strv[i], valid_strv, error))) { \
return FALSE; \ return FALSE; \
} \
set_func (s_macro (setting), item); \
} \ } \
set_func (s_macro (setting), item); \
} \ } \
g_strfreev (strv); \
return TRUE; \ return TRUE; \
} }
...@@ -1591,43 +1597,42 @@ vpn_data_item (const char *key, const char *value, gpointer user_data) ...@@ -1591,43 +1597,42 @@ vpn_data_item (const char *key, const char *value, gpointer user_data)
static gboolean \ static gboolean \
def_func (ARGS_SET_FCN) \ def_func (ARGS_SET_FCN) \
{ \ { \
char **strv = NULL, **iter; \ gs_free const char **strv = NULL; \
const char **iter; \
const char **(*valid_func1_p) (s_type *) = valid_func1; \ const char **(*valid_func1_p) (s_type *) = valid_func1; \
const char * (*valid_func2_p) (const char *, const char *, GError **) = valid_func2; \ const char * (*valid_func2_p) (const char *, const char *, GError **) = valid_func2; \
const char *opt_name, *opt_val; \ const char *opt_name, *opt_val; \
\ \
g_return_val_if_fail (error == NULL || *error == NULL, FALSE); \ nm_assert (!error || !*error); \
\ \
strv = nmc_strsplit_set (value, ",", 0); \ strv = nm_utils_strsplit_set (value, ","); \
for (iter = strv; iter && *iter; iter++) { \ for (iter = strv; iter && *iter; iter++) { \
char *left = g_strstrip (*iter); \ gs_free char *left_clone = g_strstrip (g_strdup (*iter)); \
char *left = left_clone; \
char *right = strchr (left, '='); \ char *right = strchr (left, '='); \
if (!right) { \ if (!right) { \
g_set_error (error, 1, 0, _("'%s' is not valid; use <option>=<value>"), *iter); \ g_set_error (error, 1, 0, _("'%s' is not valid; use <option>=<value>"), *iter); \
g_strfreev (strv); \
return FALSE; \ return FALSE; \
} \ } \
*right++ = '\0'; \ *right++ = '\0'; \
g_strchomp (left); \
\ \
if (valid_func1_p) { \ if (valid_func1_p) { \
const char **valid_options = valid_func1_p (s_macro (setting)); \ const char **valid_options = valid_func1_p (s_macro (setting)); \
if (!(opt_name = nmc_string_is_valid (g_strstrip (left), valid_options, error))) { \ if (!(opt_name = nmc_string_is_valid (left, valid_options, error))) { \
g_strfreev (strv); \
return FALSE; \ return FALSE; \
} \ } \
} else \ } else \
opt_name = g_strstrip (left);\ opt_name = left;\
\ \
opt_val = g_strstrip (right); \ opt_val = g_strchug (right); \
if (valid_func2_p) { \ if (valid_func2_p) { \
if (!(opt_val = valid_func2_p ((const char *) left, (const char *) opt_val, error))) { \ if (!(opt_val = valid_func2_p ((const char *) left, (const char *) opt_val, error))) { \
g_strfreev (strv); \
return FALSE; \ return FALSE; \
}\ }\
}\ }\
add_func (s_macro (setting), opt_name, opt_val); \ add_func (s_macro (setting), opt_name, opt_val); \
} \ } \
g_strfreev (strv); \
return TRUE; \ return TRUE; \
} }
...@@ -1682,47 +1687,46 @@ vpn_data_item (const char *key, const char *value, gpointer user_data) ...@@ -1682,47 +1687,46 @@ vpn_data_item (const char *key, const char *value, gpointer user_data)
def_func (ARGS_SET_FCN) \ def_func (ARGS_SET_FCN) \
{ \ { \
guint8 buf[32]; \ guint8 buf[32]; \
char **list = NULL, **iter; \ gs_free const char **strv = NULL; \
GSList *macaddr_blacklist = NULL; \ const char *const*iter; \
\ \
g_return_val_if_fail (error == NULL || *error == NULL, FALSE); \ nm_assert (!error || !*error); \
\ \
list = nmc_strsplit_set (value, " \t,", 0); \ strv = nm_utils_strsplit_set (value, " \t,"); \
for (iter = list; iter && *iter; iter++) { \ for (iter = strv; strv && *iter; iter++) { \
if (!nm_utils_hwaddr_aton (*iter, buf, ETH_ALEN)) { \ if (!nm_utils_hwaddr_aton (*iter, buf, ETH_ALEN)) { \
g_set_error (error, 1, 0, _("'%s' is not a valid MAC"), *iter); \ g_set_error (error, 1, 0, _("'%s' is not a valid MAC"), *iter); \
g_strfreev (list); \
g_slist_free (macaddr_blacklist); \
return FALSE; \ return FALSE; \
} \ } \
} \ } \
\ \
for (iter = list; iter && *iter; iter++) \ for (iter = strv; strv && *iter; iter++) \
add_func (s_macro (setting), *iter); \ add_func (s_macro (setting), *iter); \
\ \
g_strfreev (list); \
return TRUE; \ return TRUE; \
} }
static gboolean static gboolean
verify_string_list (char **strv, verify_string_list (const char *const*strv,
const char *prop, const char *prop,
gboolean (*validate_func) (const char *), gboolean (*validate_func) (const char *),
GError **error) GError **error)
{ {
char **iter; const char *const*iter;
g_return_val_if_fail (error == NULL || *error == NULL, FALSE); nm_assert (!error || !*error);
for (iter = strv; iter && *iter; iter++) {