summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2020-01-23 10:36:24 +0100
committerThomas Haller <thaller@redhat.com>2020-02-10 19:11:50 +0100
commit53f6858a27a0d9b426bab9b7abedaa82d152fb68 (patch)
tree5a9da786743c8c7ffec309223e96d8ef6934b5c6
parent08f86ce04d3e59afabdb9c0fb1d8ba471c5e8f0f (diff)
downloadNetworkManager-53f6858a27a0d9b426bab9b7abedaa82d152fb68.tar.gz
all: add nm_utils_error_is_cancelled() and nm_utils_error_is_cancelled_or_disposing()
Most callers would pass FALSE to nm_utils_error_is_cancelled(). That's not very useful. Split the two functions and have nm_utils_error_is_cancelled() and nm_utils_error_is_cancelled_is_disposing().
-rw-r--r--clients/cloud-setup/main.c10
-rw-r--r--clients/cloud-setup/nm-cloud-setup-utils.c2
-rw-r--r--clients/cloud-setup/nm-http-client.c4
-rw-r--r--clients/cloud-setup/nmcs-provider-ec2.c8
-rw-r--r--clients/common/nm-polkit-listener.c4
-rw-r--r--libnm/nm-client.c16
-rw-r--r--libnm/nm-secret-agent-old.c6
-rw-r--r--shared/nm-glib-aux/nm-shared-utils.c9
-rw-r--r--shared/nm-glib-aux/nm-shared-utils.h11
-rw-r--r--src/devices/bluetooth/nm-bluez-manager.c10
-rw-r--r--src/devices/bluetooth/nm-bluez5-dun.c2
-rw-r--r--src/devices/bluetooth/nm-device-bt.c2
-rw-r--r--src/devices/nm-device-bridge.c2
-rw-r--r--src/devices/nm-device-ethernet.c2
-rw-r--r--src/devices/nm-device-macsec.c2
-rw-r--r--src/devices/nm-device-wireguard.c2
-rw-r--r--src/devices/nm-device.c6
-rw-r--r--src/devices/ovs/nm-ovsdb.c2
-rw-r--r--src/devices/wifi/nm-device-iwd.c14
-rw-r--r--src/devices/wifi/nm-device-wifi.c2
-rw-r--r--src/dns/nm-dns-dnsmasq.c6
-rw-r--r--src/nm-firewall-manager.c2
-rw-r--r--src/settings/nm-secret-agent.c2
-rw-r--r--src/vpn/nm-vpn-connection.c2
24 files changed, 66 insertions, 62 deletions
diff --git a/clients/cloud-setup/main.c b/clients/cloud-setup/main.c
index d46c88e424..4e1a9a4e4e 100644
--- a/clients/cloud-setup/main.c
+++ b/clients/cloud-setup/main.c
@@ -31,7 +31,7 @@ _provider_detect_cb (GObject *source,
nm_assert (success != (!!error));
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
dd = user_data;
@@ -219,7 +219,7 @@ _get_config_cb (GObject *source,
config_dict = nmcs_provider_get_config_finish (NMCS_PROVIDER (source), result, &error);
if (!config_dict) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
_LOGI ("failure to get meta data: %s", error->message);
} else
_LOGD ("meta data received");
@@ -426,7 +426,7 @@ try_again:
&applied_version_id,
&error);
if (!applied_connection) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
_LOGD ("config device %s: device has no applied connection (%s). Skip", hwaddr, error->message);
return any_changes;
}
@@ -477,7 +477,7 @@ try_again:
goto try_again;
}
- if (!nm_utils_error_is_cancelled (error, FALSE)) {
+ if (!nm_utils_error_is_cancelled (error)) {
_LOGD ("config device %s: failure to reapply connection \"%s\" (%s): %s",
hwaddr,
nm_connection_get_id (applied_connection),
@@ -576,7 +576,7 @@ main (int argc, const char *const*argv)
nmcs_wait_for_objects_register (nm_client_get_context_busy_watcher (nmc));
if (error) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
_LOGI ("failure to talk to NetworkManager: %s", error->message);
goto done;
}
diff --git a/clients/cloud-setup/nm-cloud-setup-utils.c b/clients/cloud-setup/nm-cloud-setup-utils.c
index 872ab44d3a..fb802a0812 100644
--- a/clients/cloud-setup/nm-cloud-setup-utils.c
+++ b/clients/cloud-setup/nm-cloud-setup-utils.c
@@ -294,7 +294,7 @@ _poll_done_cb (GObject *source,
poll_task_data->probe_user_data,
&error);
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (nm_utils_error_is_cancelled (error)) {
/* we already handle this differently. Nothing to do. */
return;
}
diff --git a/clients/cloud-setup/nm-http-client.c b/clients/cloud-setup/nm-http-client.c
index 35f41fe775..94f3fe2b8a 100644
--- a/clients/cloud-setup/nm-http-client.c
+++ b/clients/cloud-setup/nm-http-client.c
@@ -163,7 +163,7 @@ _ehandle_complete (EHandleData *edata,
&edata->cancellable_id);
if (error_take) {
- if (nm_utils_error_is_cancelled (error_take, FALSE))
+ if (nm_utils_error_is_cancelled (error_take))
_LOG2T (edata, "cancelled");
else
_LOG2D (edata, "failed with %s", error_take->message);
@@ -422,7 +422,7 @@ _poll_get_probe_finish_fcn (GObject *source,
&local_error);
if (!success) {
- if (nm_utils_error_is_cancelled (local_error, FALSE)) {
+ if (nm_utils_error_is_cancelled (local_error)) {
g_propagate_error (error, g_steal_pointer (&local_error));
return TRUE;
}
diff --git a/clients/cloud-setup/nmcs-provider-ec2.c b/clients/cloud-setup/nmcs-provider-ec2.c
index b6ac5a4c95..82ed094970 100644
--- a/clients/cloud-setup/nmcs-provider-ec2.c
+++ b/clients/cloud-setup/nmcs-provider-ec2.c
@@ -98,7 +98,7 @@ _detect_get_meta_data_done_cb (GObject *source,
NULL,
&get_error);
- if (nm_utils_error_is_cancelled (get_error, FALSE)) {
+ if (nm_utils_error_is_cancelled (get_error)) {
g_task_return_error (task, g_steal_pointer (&get_error));
return;
}
@@ -168,7 +168,7 @@ _get_config_task_return (GetConfigIfaceData *iface_data,
nm_g_slice_free (iface_data);
if (error_take) {
- if (nm_utils_error_is_cancelled (error_take, FALSE))
+ if (nm_utils_error_is_cancelled (error_take))
_LOGD ("get-config: cancelled");
else
_LOGD ("get-config: failed: %s", error_take->message);
@@ -204,7 +204,7 @@ _get_config_fetch_done_cb (NMHttpClient *http_client,
NULL,
&response_data,
&error);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
get_config_data = iface_data->get_config_data;
@@ -327,7 +327,7 @@ _get_config_metadata_ready_cb (GObject *source,
.n_pending = 0,
};
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (nm_utils_error_is_cancelled (error)) {
_get_config_task_return (iface_data, g_steal_pointer (&error));
return;
}
diff --git a/clients/common/nm-polkit-listener.c b/clients/common/nm-polkit-listener.c
index 58684015c5..9ecc92a688 100644
--- a/clients/common/nm-polkit-listener.c
+++ b/clients/common/nm-polkit-listener.c
@@ -233,7 +233,7 @@ agent_register_cb (GObject *source_object,
res,
&error);
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (nm_utils_error_is_cancelled (error)) {
return;
}
@@ -328,7 +328,7 @@ retrieve_session_id_cb (GObject *source_object,
res,
&error);
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (nm_utils_error_is_cancelled (error)) {
return;
}
diff --git a/libnm/nm-client.c b/libnm/nm-client.c
index a93a81c353..86160b0662 100644
--- a/libnm/nm-client.c
+++ b/libnm/nm-client.c
@@ -3142,7 +3142,7 @@ _dbus_get_managed_objects_cb (GObject *source,
nm_assert ((!!ret) != (!!error));
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
priv = NM_CLIENT_GET_PRIVATE (self);
@@ -3193,7 +3193,7 @@ _nm_client_get_settings_call_cb (GObject *source, GAsyncResult *result, gpointer
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
remote_connection = user_data;
@@ -3432,7 +3432,7 @@ _dbus_check_permissions_start_cb (GObject *source, GAsyncResult *result, gpointe
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
self = user_data;
@@ -4865,7 +4865,7 @@ activate_connection_cb (GObject *object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (object), result, &error);
if (!ret) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
g_dbus_error_strip_remote_error (error);
g_task_return_error (task, error);
return;
@@ -5003,7 +5003,7 @@ _add_and_activate_connection_done (GObject *object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (object), result, &error);
if (!ret) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
g_dbus_error_strip_remote_error (error);
g_task_return_error (task, error);
return;
@@ -5526,7 +5526,7 @@ _add_connection_cb (GObject *source,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if (!ret) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
g_dbus_error_strip_remote_error (error);
g_task_return_error (task, error);
return;
@@ -6268,7 +6268,7 @@ checkpoint_create_cb (GObject *object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (object), result, &error);
if (!ret) {
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
g_dbus_error_strip_remote_error (error);
g_task_return_error (task, error);
return;
@@ -6942,7 +6942,7 @@ name_owner_get_cb (GObject *source,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
priv = NM_CLIENT_GET_PRIVATE (self);
diff --git a/libnm/nm-secret-agent-old.c b/libnm/nm-secret-agent-old.c
index eb653327d0..f102ecc752 100644
--- a/libnm/nm-secret-agent-old.c
+++ b/libnm/nm-secret-agent-old.c
@@ -1148,7 +1148,7 @@ _register_call_cb (GObject *source,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
nm_assert (!priv->registering_retry_source);
@@ -1229,7 +1229,7 @@ _get_connection_unix_user_cb (GObject *source,
guint32 sender_uid = 0;
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
self = user_data;
@@ -1324,7 +1324,7 @@ _name_owner_get_cb (const char *name_owner,
gpointer user_data)
{
if ( name_owner
- || !nm_utils_error_is_cancelled (error, FALSE))
+ || !nm_utils_error_is_cancelled (error))
_name_owner_changed (user_data, name_owner, FALSE);
}
diff --git a/shared/nm-glib-aux/nm-shared-utils.c b/shared/nm-glib-aux/nm-shared-utils.c
index 44079dcf3a..74577c9c53 100644
--- a/shared/nm-glib-aux/nm-shared-utils.c
+++ b/shared/nm-glib-aux/nm-shared-utils.c
@@ -1643,16 +1643,13 @@ nm_utils_error_set_cancelled (GError **error,
}
gboolean
-nm_utils_error_is_cancelled (GError *error,
- gboolean consider_is_disposing)
+nm_utils_error_is_cancelled_or_disposing (GError *error)
{
if (error) {
if (error->domain == G_IO_ERROR)
return NM_IN_SET (error->code, G_IO_ERROR_CANCELLED);
- if (consider_is_disposing) {
- if (error->domain == NM_UTILS_ERROR)
- return NM_IN_SET (error->code, NM_UTILS_ERROR_CANCELLED_DISPOSING);
- }
+ if (error->domain == NM_UTILS_ERROR)
+ return NM_IN_SET (error->code, NM_UTILS_ERROR_CANCELLED_DISPOSING);
}
return FALSE;
}
diff --git a/shared/nm-glib-aux/nm-shared-utils.h b/shared/nm-glib-aux/nm-shared-utils.h
index ebb526152a..4c53397d7e 100644
--- a/shared/nm-glib-aux/nm-shared-utils.h
+++ b/shared/nm-glib-aux/nm-shared-utils.h
@@ -883,8 +883,15 @@ nm_utils_error_new_cancelled (gboolean is_disposing,
return error;
}
-gboolean nm_utils_error_is_cancelled (GError *error,
- gboolean consider_is_disposing);
+gboolean nm_utils_error_is_cancelled_or_disposing (GError *error);
+
+static inline gboolean
+nm_utils_error_is_cancelled (GError *error)
+{
+ return error
+ && error->code == G_IO_ERROR_CANCELLED
+ && error->domain == G_IO_ERROR;
+}
gboolean nm_utils_error_is_notfound (GError *error);
diff --git a/src/devices/bluetooth/nm-bluez-manager.c b/src/devices/bluetooth/nm-bluez-manager.c
index 84b8c28e88..c24f1306f8 100644
--- a/src/devices/bluetooth/nm-bluez-manager.c
+++ b/src/devices/bluetooth/nm-bluez-manager.c
@@ -1089,7 +1089,7 @@ _network_server_register_cb (GObject *source_object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source_object), res, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
bzobj = user_data;
@@ -2172,7 +2172,7 @@ _dbus_get_managed_objects_cb (GVariant *result,
GVariant *ifaces;
if ( !result
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
self = user_data;
@@ -2475,7 +2475,7 @@ _connect_dun_step2_cb (NMBluez5DunContext *context,
{
BzDBusObj *bzobj;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
bzobj = user_data;
@@ -2517,7 +2517,7 @@ _connect_dun_step1_cb (GObject *source_object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source_object), res, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
bzobj = user_data;
@@ -2564,7 +2564,7 @@ _connect_nap_cb (GObject *source_object,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source_object), res, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
if (ret)
diff --git a/src/devices/bluetooth/nm-bluez5-dun.c b/src/devices/bluetooth/nm-bluez5-dun.c
index 0d0354e21b..c253d4e28d 100644
--- a/src/devices/bluetooth/nm-bluez5-dun.c
+++ b/src/devices/bluetooth/nm-bluez5-dun.c
@@ -805,7 +805,7 @@ _context_invoke_callback (NMBluez5DunContext *context,
if (!error)
_LOGD (context, "connected via \"%s\"", context->rfcomm_tty_path);
- else if (nm_utils_error_is_cancelled (error, FALSE))
+ else if (nm_utils_error_is_cancelled (error))
_LOGD (context, "cancelled");
else
_LOGD (context, "failed to connect: %s", error->message);
diff --git a/src/devices/bluetooth/nm-device-bt.c b/src/devices/bluetooth/nm-device-bt.c
index 497810c36f..f59007b3c7 100644
--- a/src/devices/bluetooth/nm-device-bt.c
+++ b/src/devices/bluetooth/nm-device-bt.c
@@ -830,7 +830,7 @@ connect_bz_cb (NMBluezManager *bz_mgr,
NMDeviceBtPrivate *priv;
char sbuf[100];
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
self = user_data;
diff --git a/src/devices/nm-device-bridge.c b/src/devices/nm-device-bridge.c
index 164fa0a1e8..53221080c0 100644
--- a/src/devices/nm-device-bridge.c
+++ b/src/devices/nm-device-bridge.c
@@ -518,7 +518,7 @@ _bt_register_bridge_cb (GError *error,
{
NMDeviceBridge *self;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
self = user_data;
diff --git a/src/devices/nm-device-ethernet.c b/src/devices/nm-device-ethernet.c
index c7c42eaa27..db37b07328 100644
--- a/src/devices/nm-device-ethernet.c
+++ b/src/devices/nm-device-ethernet.c
@@ -623,7 +623,7 @@ supplicant_iface_assoc_cb (NMSupplicantInterface *iface,
{
NMDeviceEthernet *self = NM_DEVICE_ETHERNET (user_data);
- if (error && !nm_utils_error_is_cancelled (error, TRUE)) {
+ if (error && !nm_utils_error_is_cancelled_or_disposing (error)) {
supplicant_interface_release (self);
nm_device_queue_state (NM_DEVICE (self),
NM_DEVICE_STATE_FAILED,
diff --git a/src/devices/nm-device-macsec.c b/src/devices/nm-device-macsec.c
index c9592a4971..a7a9b1faa7 100644
--- a/src/devices/nm-device-macsec.c
+++ b/src/devices/nm-device-macsec.c
@@ -270,7 +270,7 @@ supplicant_iface_assoc_cb (NMSupplicantInterface *iface,
{
NMDeviceMacsec *self = NM_DEVICE_MACSEC (user_data);
- if (error && !nm_utils_error_is_cancelled (error, TRUE)) {
+ if (error && !nm_utils_error_is_cancelled_or_disposing (error)) {
supplicant_interface_release (self);
nm_device_queue_state (NM_DEVICE (self),
NM_DEVICE_STATE_FAILED,
diff --git a/src/devices/nm-device-wireguard.c b/src/devices/nm-device-wireguard.c
index a5fce0a604..063ffc3959 100644
--- a/src/devices/nm-device-wireguard.c
+++ b/src/devices/nm-device-wireguard.c
@@ -684,7 +684,7 @@ _peers_resolve_cb (GObject *source_object,
list = g_resolver_lookup_by_name_finish (G_RESOLVER (source_object), res, &resolv_error);
- if (nm_utils_error_is_cancelled (resolv_error, FALSE))
+ if (nm_utils_error_is_cancelled (resolv_error))
return;
peer_data = user_data;
diff --git a/src/devices/nm-device.c b/src/devices/nm-device.c
index 0f410807a6..e26a1602ac 100644
--- a/src/devices/nm-device.c
+++ b/src/devices/nm-device.c
@@ -6505,7 +6505,7 @@ sriov_params_cb (GError *error, gpointer data)
nm_utils_user_data_unpack (data, &self, &plat_vfs);
- if (nm_utils_error_is_cancelled (error, TRUE))
+ if (nm_utils_error_is_cancelled_or_disposing (error))
return;
priv = NM_DEVICE_GET_PRIVATE (self);
@@ -10713,7 +10713,7 @@ fw_change_zone_cb (NMFirewallManager *firewall_manager,
priv->fw_call = NULL;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
switch (priv->fw_state) {
@@ -15466,7 +15466,7 @@ sriov_deactivate_cb (GError *error, gpointer user_data)
NMDevice *self;
gpointer reason;
- if (nm_utils_error_is_cancelled (error, TRUE))
+ if (nm_utils_error_is_cancelled_or_disposing (error))
return;
nm_utils_user_data_unpack (user_data, &self, &reason);
diff --git a/src/devices/ovs/nm-ovsdb.c b/src/devices/ovs/nm-ovsdb.c
index ec4f5c746e..d2c7014186 100644
--- a/src/devices/ovs/nm-ovsdb.c
+++ b/src/devices/ovs/nm-ovsdb.c
@@ -1394,7 +1394,7 @@ static void
_monitor_bridges_cb (NMOvsdb *self, json_t *result, GError *error, gpointer user_data)
{
if (error) {
- if (!nm_utils_error_is_cancelled (error, TRUE)) {
+ if (!nm_utils_error_is_cancelled_or_disposing (error)) {
_LOGI ("%s", error->message);
ovsdb_disconnect (self, FALSE);
}
diff --git a/src/devices/wifi/nm-device-iwd.c b/src/devices/wifi/nm-device-iwd.c
index 356f590faa..2954b517d6 100644
--- a/src/devices/wifi/nm-device-iwd.c
+++ b/src/devices/wifi/nm-device-iwd.c
@@ -993,7 +993,7 @@ scan_cb (GObject *source, GAsyncResult *res, gpointer user_data)
gs_free_error GError *error = NULL;
variant = g_dbus_proxy_call_finish (G_DBUS_PROXY (source), res, &error);
- if (!variant && nm_utils_error_is_cancelled (error, FALSE))
+ if (!variant && nm_utils_error_is_cancelled (error))
return;
priv = NM_DEVICE_IWD_GET_PRIVATE (self);
@@ -1270,7 +1270,7 @@ wifi_secrets_cb (NMActRequest *req,
priv->wifi_secrets_id = NULL;
- if (nm_utils_error_is_cancelled (error, FALSE)) {
+ if (nm_utils_error_is_cancelled (error)) {
g_dbus_method_invocation_return_error_literal (invocation, NM_DEVICE_ERROR,
NM_DEVICE_ERROR_INVALID_CONNECTION,
"NM secrets request cancelled");
@@ -1366,7 +1366,7 @@ network_connect_cb (GObject *source, GAsyncResult *res, gpointer user_data)
"Activation: (wifi) Network.Connect failed: %s",
error->message);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
if (!NM_IN_SET (nm_device_get_state (device), NM_DEVICE_STATE_CONFIG, NM_DEVICE_STATE_NEED_AUTH))
@@ -1438,7 +1438,7 @@ act_failed_cb (GObject *source, GAsyncResult *res, gpointer user_data)
gs_free_error GError *error = NULL;
variant = g_dbus_proxy_call_finish (G_DBUS_PROXY (source), res, &error);
- if (!variant && nm_utils_error_is_cancelled (error, FALSE))
+ if (!variant && nm_utils_error_is_cancelled (error))
return;
/* Change state to FAILED unless already done by state_changed
@@ -1467,7 +1467,7 @@ act_start_cb (GObject *source, GAsyncResult *res, gpointer user_data)
"Activation: (wifi) Network.Connect failed: %s",
error->message);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
if (!NM_IN_SET (nm_device_get_state (device), NM_DEVICE_STATE_CONFIG))
@@ -1587,7 +1587,7 @@ act_set_mode_cb (GObject *source, GAsyncResult *res, gpointer user_data)
"Activation: (wifi) Setting Device.Mode failed: %s",
error->message);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
if ( !NM_IN_SET (nm_device_get_state (device), NM_DEVICE_STATE_CONFIG)
@@ -1647,7 +1647,7 @@ act_psk_cb (NMActRequest *req,
NMDeviceIwdPrivate *priv;
NMDevice *device;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
priv = NM_DEVICE_IWD_GET_PRIVATE (self);
diff --git a/src/devices/wifi/nm-device-wifi.c b/src/devices/wifi/nm-device-wifi.c
index f334858cb4..daa38a3fd4 100644
--- a/src/devices/wifi/nm-device-wifi.c
+++ b/src/devices/wifi/nm-device-wifi.c
@@ -2171,7 +2171,7 @@ supplicant_iface_assoc_cb (NMSupplicantInterface *iface,
NMDeviceWifi *self = NM_DEVICE_WIFI (user_data);
NMDevice *device = NM_DEVICE (self);
- if ( error && !nm_utils_error_is_cancelled (error, TRUE)
+ if ( error && !nm_utils_error_is_cancelled_or_disposing (error)
&& nm_device_is_activating (device)) {
cleanup_association_attempt (self, TRUE);
nm_device_queue_state (device, NM_DEVICE_STATE_FAILED, NM_DEVICE_STATE_REASON_SUPPLICANT_FAILED);
diff --git a/src/dns/nm-dns-dnsmasq.c b/src/dns/nm-dns-dnsmasq.c
index 81a55834fd..1c430afdad 100644
--- a/src/dns/nm-dns-dnsmasq.c
+++ b/src/dns/nm-dns-dnsmasq.c
@@ -304,7 +304,7 @@ _gl_pid_spawn_notify (GlPidSpawnAsyncData *sdata,
if (error) {
nm_assert (pid == 0);
nm_assert (!p_exit_code);
- if (!nm_utils_error_is_cancelled (error, FALSE))
+ if (!nm_utils_error_is_cancelled (error))
_LOGD ("spawn: dnsmasq failed: %s", error->message);
} else if (p_exit_code) {
/* the only caller already logged about this condition extensively. */
@@ -838,7 +838,7 @@ dnsmasq_update_done (GObject *source_object, GAsyncResult *res, gpointer user_da
response = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source_object), res, &error);
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
self = user_data;
@@ -987,7 +987,7 @@ spawn_notify (GCancellable *cancellable,
NMDnsDnsmasq *self;
NMDnsDnsmasqPrivate *priv;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
self = notify_user_data;
diff --git a/src/nm-firewall-manager.c b/src/nm-firewall-manager.c
index cf6494bdb5..304b15f99a 100644
--- a/src/nm-firewall-manager.c
+++ b/src/nm-firewall-manager.c
@@ -265,7 +265,7 @@ _handle_dbus_cb (GObject *source,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
call_id = user_data;
diff --git a/src/settings/nm-secret-agent.c b/src/settings/nm-secret-agent.c
index e5c727b49f..6159943a19 100644
--- a/src/settings/nm-secret-agent.c
+++ b/src/settings/nm-secret-agent.c
@@ -370,7 +370,7 @@ _dbus_call_cb (GObject *source,
ret = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, &error);
if ( !ret
- && nm_utils_error_is_cancelled (error, FALSE))
+ && nm_utils_error_is_cancelled (error))
return;
call_id = user_data;
diff --git a/src/vpn/nm-vpn-connection.c b/src/vpn/nm-vpn-connection.c
index 97ff5b84be..21368b01dc 100644
--- a/src/vpn/nm-vpn-connection.c
+++ b/src/vpn/nm-vpn-connection.c
@@ -1194,7 +1194,7 @@ fw_change_zone_cb (NMFirewallManager *firewall_manager,
priv->fw_call = NULL;
- if (nm_utils_error_is_cancelled (error, FALSE))
+ if (nm_utils_error_is_cancelled (error))
return;
if (error) {