From 73221a809d80799c09b2c97fd821e362acce75c3 Mon Sep 17 00:00:00 2001 From: Beniamino Galvani Date: Thu, 19 May 2016 15:01:35 +0200 Subject: [PATCH] cli: remove wrong assertion in check_valid_name() As explained in the function comment, a NULL value for @val is acceptable. Otherwise: $ nmcli connection edit (process:2276): nmcli-CRITICAL **: check_valid_name: assertion 'val' failed Fixes: fb8fe1d8cb452d203c8cf0941c2a82e5903a0c8a --- clients/cli/connections.c | 1 - 1 file changed, 1 deletion(-) diff --git a/clients/cli/connections.c b/clients/cli/connections.c index 2db2fcb696..d345026aca 100644 --- a/clients/cli/connections.c +++ b/clients/cli/connections.c @@ -2931,7 +2931,6 @@ check_valid_name (const char *val, const NameItem *array, const NameItem *array_ GError *tmp_err = NULL; int i; - g_return_val_if_fail (val, NULL); g_return_val_if_fail (array, NULL); /* Create a temporary array that can be used in nmc_string_is_valid() */