diff options
author | Lubomir Rintel <lkundrak@v3.sk> | 2015-08-10 12:33:01 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2015-09-02 09:47:28 +0200 |
commit | 25fad68ad7ca5b84216c45a5e2027095be3bbaa8 (patch) | |
tree | b7a65bdaadbfd5da3d87c9c1d32002ad38858888 | |
parent | 0b0e15389b903c3d2e2830940937fd6cc376fec1 (diff) | |
download | NetworkManager-25fad68ad7ca5b84216c45a5e2027095be3bbaa8.tar.gz |
audit-manager: don't log connection whose addition fails
In AddAndActivate the uuid and id make no sense until authorization succeeds
and the connection is saved.
-rw-r--r-- | src/nm-audit-manager.c | 1 | ||||
-rw-r--r-- | src/nm-manager.c | 6 |
2 files changed, 3 insertions, 4 deletions
diff --git a/src/nm-audit-manager.c b/src/nm-audit-manager.c index eb9bed171b..929800d96a 100644 --- a/src/nm-audit-manager.c +++ b/src/nm-audit-manager.c @@ -220,7 +220,6 @@ _nm_audit_manager_log_connection_op (NMAuditManager *self, const char *file, gui AuditField uuid_field = { }, name_field = { }; g_return_if_fail (op); - g_return_if_fail (connection || !strcmp (op, NM_AUDIT_OP_CONN_ADD)); fields = g_ptr_array_new (); diff --git a/src/nm-manager.c b/src/nm-manager.c index df25a086f4..2428eb1fa9 100644 --- a/src/nm-manager.c +++ b/src/nm-manager.c @@ -3250,7 +3250,7 @@ activation_add_done (NMSettings *self, nm_settings_connection_delete (new_connection, NULL, NULL); g_dbus_method_invocation_return_gerror (context, error); nm_audit_log_connection_op (NM_AUDIT_OP_CONN_ADD_ACTIVATE, - nm_active_connection_get_connection (info->active), + NULL, FALSE, nm_active_connection_get_subject (info->active), error->message); @@ -3292,7 +3292,7 @@ _add_and_activate_auth_done (NMActiveConnection *active, NM_MANAGER_ERROR_PERMISSION_DENIED, error_desc); nm_audit_log_connection_op (NM_AUDIT_OP_CONN_ADD_ACTIVATE, - nm_active_connection_get_connection (active), + NULL, FALSE, nm_active_connection_get_subject (active), error->message); @@ -3398,7 +3398,7 @@ impl_manager_add_and_activate_connection (NMManager *self, return; error: - nm_audit_log_connection_op (NM_AUDIT_OP_CONN_ADD_ACTIVATE, connection, FALSE, subject, error->message); + nm_audit_log_connection_op (NM_AUDIT_OP_CONN_ADD_ACTIVATE, NULL, FALSE, subject, error->message); g_clear_object (&connection); g_slist_free (all_connections); g_clear_object (&subject); |