about summary refs log tree commit diff
path: root/src/core/settings/plugins/keyfile
diff options
context:
space:
mode:
Diffstat (limited to 'src/core/settings/plugins/keyfile')
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-plugin.c258
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-plugin.h30
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-reader.c40
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-reader.h20
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-storage.c28
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-storage.h18
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-utils.c42
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-utils.h32
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-writer.c98
-rw-r--r--src/core/settings/plugins/keyfile/nms-keyfile-writer.h28
-rw-r--r--src/core/settings/plugins/keyfile/tests/test-keyfile-settings.c534
11 files changed, 564 insertions, 564 deletions
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-plugin.c b/src/core/settings/plugins/keyfile/nms-keyfile-plugin.c
index aa36ee8f..efb9bfce 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-plugin.c
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-plugin.c
@@ -92,7 +92,7 @@ G_DEFINE_TYPE(NMSKeyfilePlugin, nms_keyfile_plugin, NM_TYPE_SETTINGS_PLUGIN)
 /*****************************************************************************/
 
 static const char *
-_extra_flags_to_string(char *   str,
+_extra_flags_to_string(char    *str,
                        gsize    str_len,
                        gboolean is_nm_generated,
                        gboolean is_volatile,
@@ -147,19 +147,19 @@ _get_plugin_dir(NMSKeyfilePluginPrivate *priv)
 }
 
 static gboolean
-_path_detect_storage_type(const char *           full_filename,
-                          const char *const *    dirname_libs,
-                          const char *           dirname_etc,
-                          const char *           dirname_run,
+_path_detect_storage_type(const char            *full_filename,
+                          const char *const     *dirname_libs,
+                          const char            *dirname_etc,
+                          const char            *dirname_run,
                           NMSKeyfileStorageType *out_storage_type,
-                          const char **          out_dirname,
-                          const char **          out_filename,
-                          gboolean *             out_is_nmmeta_file,
-                          gboolean *             out_failed_due_to_invalid_filename)
+                          const char           **out_dirname,
+                          const char           **out_filename,
+                          gboolean              *out_is_nmmeta_file,
+                          gboolean              *out_failed_due_to_invalid_filename)
 {
     NMSKeyfileStorageType storage_type;
-    const char *          filename = NULL;
-    const char *          dirname  = NULL;
+    const char           *filename = NULL;
+    const char           *dirname  = NULL;
     guint                 i;
     gboolean              is_nmmeta_file = FALSE;
 
@@ -208,15 +208,15 @@ _path_detect_storage_type(const char *           full_filename,
 /*****************************************************************************/
 
 static NMConnection *
-_read_from_file(const char * full_filename,
-                const char * plugin_dir,
+_read_from_file(const char  *full_filename,
+                const char  *plugin_dir,
                 struct stat *out_stat,
-                NMTernary *  out_is_nm_generated,
-                NMTernary *  out_is_volatile,
-                NMTernary *  out_is_external,
-                char **      out_shadowed_storage,
-                NMTernary *  out_shadowed_owned,
-                GError **    error)
+                NMTernary   *out_is_nm_generated,
+                NMTernary   *out_is_volatile,
+                NMTernary   *out_is_external,
+                char       **out_shadowed_storage,
+                NMTernary   *out_shadowed_owned,
+                GError     **error)
 {
     NMConnection *connection;
 
@@ -248,7 +248,7 @@ _nm_assert_storage(gpointer plugin /* NMSKeyfilePlugin  */,
 {
 #if NM_MORE_ASSERTS
     NMSettUtilStorageByUuidHead *sbuh;
-    const char *                 uuid;
+    const char                  *uuid;
 
     nm_assert(!plugin || NMS_IS_KEYFILE_PLUGIN(plugin));
     nm_assert(NMS_IS_KEYFILE_STORAGE(storage));
@@ -256,7 +256,7 @@ _nm_assert_storage(gpointer plugin /* NMSKeyfilePlugin  */,
 
     nm_assert(({
         const char *f = nms_keyfile_storage_get_filename(storage);
-        f &&        f[0] == '/';
+        f         &&f[0] == '/';
     }));
 
     uuid = nms_keyfile_storage_get_uuid(storage);
@@ -294,22 +294,22 @@ _nm_assert_storage(gpointer plugin /* NMSKeyfilePlugin  */,
 /*****************************************************************************/
 
 static NMSKeyfileStorage *
-_load_file(NMSKeyfilePlugin *    self,
-           const char *          dirname,
-           const char *          filename,
+_load_file(NMSKeyfilePlugin     *self,
+           const char           *dirname,
+           const char           *filename,
            NMSKeyfileStorageType storage_type,
-           GError **             error)
+           GError              **error)
 {
-    NMSKeyfilePluginPrivate *priv;
+    NMSKeyfilePluginPrivate      *priv;
     gs_unref_object NMConnection *connection = NULL;
     NMTernary                     is_nm_generated_opt;
     NMTernary                     is_volatile_opt;
     NMTernary                     is_external_opt;
     NMTernary                     shadowed_owned_opt;
-    gs_free char *                shadowed_storage = NULL;
-    gs_free_error GError *local                    = NULL;
-    gs_free char *        full_filename            = NULL;
-    struct stat           st;
+    gs_free char                 *shadowed_storage = NULL;
+    gs_free_error GError         *local            = NULL;
+    gs_free char                 *full_filename    = NULL;
+    struct stat                   st;
 
     if (_ignore_filename(storage_type, filename)) {
         gs_free char *nmmeta                    = NULL;
@@ -403,14 +403,14 @@ _load_file(NMSKeyfilePlugin *    self,
 }
 
 static NMSKeyfileStorage *
-_load_file_from_path(NMSKeyfilePlugin *    self,
-                     const char *          full_filename,
+_load_file_from_path(NMSKeyfilePlugin     *self,
+                     const char           *full_filename,
                      NMSKeyfileStorageType storage_type,
-                     GError **             error)
+                     GError              **error)
 {
     gs_free char *f_dirname_free = NULL;
-    const char *  f_filename;
-    const char *  f_dirname;
+    const char   *f_filename;
+    const char   *f_dirname;
 
     nm_assert(full_filename && full_filename[0] == '/');
 
@@ -421,13 +421,13 @@ _load_file_from_path(NMSKeyfilePlugin *    self,
 }
 
 static void
-_load_dir(NMSKeyfilePlugin *    self,
+_load_dir(NMSKeyfilePlugin     *self,
           NMSKeyfileStorageType storage_type,
-          const char *          dirname,
-          NMSettUtilStorages *  storages)
+          const char           *dirname,
+          NMSettUtilStorages   *storages)
 {
-    const char *       filename;
-    GDir *             dir;
+    const char                    *filename;
+    GDir                          *dir;
     gs_unref_hashtable GHashTable *dupl_filenames = NULL;
 
     dir = g_dir_open(dirname, 0, NULL);
@@ -465,21 +465,21 @@ _load_dir(NMSKeyfilePlugin *    self,
 /*****************************************************************************/
 
 static void
-_storages_consolidate(NMSKeyfilePlugin *                     self,
-                      NMSettUtilStorages *                   storages_new,
+_storages_consolidate(NMSKeyfilePlugin                      *self,
+                      NMSettUtilStorages                    *storages_new,
                       gboolean                               replace_all,
-                      GHashTable *                           storages_replaced,
+                      GHashTable                            *storages_replaced,
                       NMSettingsPluginConnectionLoadCallback callback,
                       gpointer                               user_data)
 {
-    NMSKeyfilePluginPrivate *priv                  = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
-    CList                    lst_conn_info_deleted = C_LIST_INIT(lst_conn_info_deleted);
-    gs_unref_ptrarray GPtrArray *storages_modified = NULL;
+    NMSKeyfilePluginPrivate     *priv                  = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    CList                        lst_conn_info_deleted = C_LIST_INIT(lst_conn_info_deleted);
+    gs_unref_ptrarray GPtrArray *storages_modified     = NULL;
     CList                        storages_deleted;
-    NMSKeyfileStorage *          storage_safe;
-    NMSKeyfileStorage *          storage_new;
-    NMSKeyfileStorage *          storage_old;
-    NMSKeyfileStorage *          storage;
+    NMSKeyfileStorage           *storage_safe;
+    NMSKeyfileStorage           *storage_new;
+    NMSKeyfileStorage           *storage_old;
+    NMSKeyfileStorage           *storage;
     guint                        i;
 
     storages_modified = g_ptr_array_new_with_free_func(g_object_unref);
@@ -576,12 +576,12 @@ _storages_consolidate(NMSKeyfilePlugin *                     self,
 }
 
 static void
-reload_connections(NMSettingsPlugin *                     plugin,
+reload_connections(NMSettingsPlugin                      *plugin,
                    NMSettingsPluginConnectionLoadCallback callback,
                    gpointer                               user_data)
 {
-    NMSKeyfilePlugin *                                  self = NMS_KEYFILE_PLUGIN(plugin);
-    NMSKeyfilePluginPrivate *                           priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    NMSKeyfilePlugin                                   *self = NMS_KEYFILE_PLUGIN(plugin);
+    NMSKeyfilePluginPrivate                            *priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
     nm_auto_clear_sett_util_storages NMSettUtilStorages storages_new =
         NM_SETT_UTIL_STORAGES_INIT(storages_new, nms_keyfile_storage_destroy);
     int i;
@@ -596,20 +596,20 @@ reload_connections(NMSettingsPlugin *                     plugin,
 }
 
 static void
-load_connections(NMSettingsPlugin *                     plugin,
-                 NMSettingsPluginConnectionLoadEntry *  entries,
+load_connections(NMSettingsPlugin                      *plugin,
+                 NMSettingsPluginConnectionLoadEntry   *entries,
                  gsize                                  n_entries,
                  NMSettingsPluginConnectionLoadCallback callback,
                  gpointer                               user_data)
 {
-    NMSKeyfilePlugin *                                  self = NMS_KEYFILE_PLUGIN(plugin);
-    NMSKeyfilePluginPrivate *                           priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    NMSKeyfilePlugin                                   *self = NMS_KEYFILE_PLUGIN(plugin);
+    NMSKeyfilePluginPrivate                            *priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
     nm_auto_clear_sett_util_storages NMSettUtilStorages storages_new =
         NM_SETT_UTIL_STORAGES_INIT(storages_new, nms_keyfile_storage_destroy);
     gs_unref_hashtable GHashTable *dupl_filenames    = NULL;
     gs_unref_hashtable GHashTable *storages_replaced = NULL;
     gs_unref_hashtable GHashTable *loaded_uuids      = NULL;
-    const char *                   loaded_uuid;
+    const char                    *loaded_uuid;
     GHashTableIter                 h_iter;
     gsize                          i;
 
@@ -625,15 +625,15 @@ load_connections(NMSettingsPlugin *                     plugin,
     for (i = 0; i < n_entries; i++) {
         NMSettingsPluginConnectionLoadEntry *const entry = &entries[i];
         NMSKeyfileStorageType                      storage_type;
-        gs_free_error GError *               local = NULL;
-        const char *                         f_filename;
-        const char *                         f_dirname;
-        const char *                         full_filename;
-        gs_free char *                       full_filename_keep = NULL;
-        gboolean                             is_nmmeta_file;
-        NMSettingsPluginConnectionLoadEntry *dupl_content_entry;
-        gboolean                             failed_due_to_invalid_filename;
-        gs_unref_object NMSKeyfileStorage *storage = NULL;
+        gs_free_error GError                      *local = NULL;
+        const char                                *f_filename;
+        const char                                *f_dirname;
+        const char                                *full_filename;
+        gs_free char                              *full_filename_keep = NULL;
+        gboolean                                   is_nmmeta_file;
+        NMSettingsPluginConnectionLoadEntry       *dupl_content_entry;
+        gboolean                                   failed_due_to_invalid_filename;
+        gs_unref_object NMSKeyfileStorage         *storage = NULL;
 
         if (entry->handled)
             continue;
@@ -700,7 +700,7 @@ load_connections(NMSettingsPlugin *                     plugin,
     /* now we visit all UUIDs that are about to change... */
     g_hash_table_iter_init(&h_iter, loaded_uuids);
     while (g_hash_table_iter_next(&h_iter, (gpointer *) &loaded_uuid, NULL)) {
-        NMSKeyfileStorage *          storage;
+        NMSKeyfileStorage           *storage;
         NMSettUtilStorageByUuidHead *sbuh;
 
         sbuh = nm_sett_util_storages_lookup_by_uuid(&priv->storages, loaded_uuid);
@@ -710,9 +710,9 @@ load_connections(NMSettingsPlugin *                     plugin,
         c_list_for_each_entry (storage,
                                &sbuh->_storage_by_uuid_lst_head,
                                parent._storage_by_uuid_lst) {
-            const char *    full_filename = nms_keyfile_storage_get_filename(storage);
+            const char *full_filename = nms_keyfile_storage_get_filename(storage);
             gs_unref_object NMSKeyfileStorage *storage_new = NULL;
-            gs_free_error GError *local                    = NULL;
+            gs_free_error GError              *local       = NULL;
 
             if (g_hash_table_contains(dupl_filenames, full_filename)) {
                 /* already re-loaded. */
@@ -744,25 +744,25 @@ load_connections(NMSettingsPlugin *                     plugin,
 }
 
 gboolean
-nms_keyfile_plugin_add_connection(NMSKeyfilePlugin *  self,
-                                  NMConnection *      connection,
+nms_keyfile_plugin_add_connection(NMSKeyfilePlugin   *self,
+                                  NMConnection       *connection,
                                   gboolean            in_memory,
                                   gboolean            is_nm_generated,
                                   gboolean            is_volatile,
                                   gboolean            is_external,
-                                  const char *        shadowed_storage,
+                                  const char         *shadowed_storage,
                                   gboolean            shadowed_owned,
                                   NMSettingsStorage **out_storage,
-                                  NMConnection **     out_connection,
-                                  GError **           error)
+                                  NMConnection      **out_connection,
+                                  GError            **error)
 {
-    NMSKeyfilePluginPrivate *priv               = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
-    gs_unref_object NMConnection *reread        = NULL;
-    gs_free char *                full_filename = NULL;
-    NMSKeyfileStorageType         storage_type;
+    NMSKeyfilePluginPrivate           *priv          = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    gs_unref_object NMConnection      *reread        = NULL;
+    gs_free char                      *full_filename = NULL;
+    NMSKeyfileStorageType              storage_type;
     gs_unref_object NMSKeyfileStorage *storage = NULL;
-    GError *                           local   = NULL;
-    const char *                       uuid;
+    GError                            *local   = NULL;
+    const char                        *uuid;
     gboolean                           reread_same;
     struct timespec                    mtime;
     char                               strbuf[100];
@@ -849,11 +849,11 @@ nms_keyfile_plugin_add_connection(NMSKeyfilePlugin *  self,
 }
 
 static gboolean
-add_connection(NMSettingsPlugin *  plugin,
-               NMConnection *      connection,
+add_connection(NMSettingsPlugin   *plugin,
+               NMConnection       *connection,
                NMSettingsStorage **out_storage,
-               NMConnection **     out_connection,
-               GError **           error)
+               NMConnection      **out_connection,
+               GError            **error)
 {
     return nms_keyfile_plugin_add_connection(NMS_KEYFILE_PLUGIN(plugin),
                                              connection,
@@ -869,30 +869,30 @@ add_connection(NMSettingsPlugin *  plugin,
 }
 
 gboolean
-nms_keyfile_plugin_update_connection(NMSKeyfilePlugin *  self,
-                                     NMSettingsStorage * storage_x,
-                                     NMConnection *      connection,
+nms_keyfile_plugin_update_connection(NMSKeyfilePlugin   *self,
+                                     NMSettingsStorage  *storage_x,
+                                     NMConnection       *connection,
                                      gboolean            is_nm_generated,
                                      gboolean            is_volatile,
                                      gboolean            is_external,
-                                     const char *        shadowed_storage,
+                                     const char         *shadowed_storage,
                                      gboolean            shadowed_owned,
                                      gboolean            force_rename,
                                      NMSettingsStorage **out_storage,
-                                     NMConnection **     out_connection,
-                                     GError **           error)
+                                     NMConnection      **out_connection,
+                                     GError            **error)
 {
-    NMSKeyfilePluginPrivate *priv                  = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
-    NMSKeyfileStorage *      storage               = NMS_KEYFILE_STORAGE(storage_x);
+    NMSKeyfilePluginPrivate      *priv             = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    NMSKeyfileStorage            *storage          = NMS_KEYFILE_STORAGE(storage_x);
     gs_unref_object NMConnection *connection_clone = NULL;
     gs_unref_object NMConnection *reread           = NULL;
-    gs_free char *                full_filename    = NULL;
-    gs_free_error GError *local                    = NULL;
-    struct timespec       mtime;
-    const char *          previous_filename;
-    gboolean              reread_same;
-    const char *          uuid;
-    char                  strbuf[100];
+    gs_free char                 *full_filename    = NULL;
+    gs_free_error GError         *local            = NULL;
+    struct timespec               mtime;
+    const char                   *previous_filename;
+    gboolean                      reread_same;
+    const char                   *uuid;
+    char                          strbuf[100];
 
     _nm_assert_storage(self, storage, TRUE);
     nm_assert(NM_IS_CONNECTION(connection));
@@ -971,12 +971,12 @@ nms_keyfile_plugin_update_connection(NMSKeyfilePlugin *  self,
 }
 
 static gboolean
-update_connection(NMSettingsPlugin *  plugin,
-                  NMSettingsStorage * storage,
-                  NMConnection *      connection,
+update_connection(NMSettingsPlugin   *plugin,
+                  NMSettingsStorage  *storage,
+                  NMConnection       *connection,
                   NMSettingsStorage **out_storage,
-                  NMConnection **     out_connection,
-                  GError **           error)
+                  NMConnection      **out_connection,
+                  GError            **error)
 {
     return nms_keyfile_plugin_update_connection(NMS_KEYFILE_PLUGIN(plugin),
                                                 storage,
@@ -995,13 +995,13 @@ update_connection(NMSettingsPlugin *  plugin,
 static gboolean
 delete_connection(NMSettingsPlugin *plugin, NMSettingsStorage *storage_x, GError **error)
 {
-    NMSKeyfilePlugin *       self              = NMS_KEYFILE_PLUGIN(plugin);
-    NMSKeyfilePluginPrivate *priv              = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
+    NMSKeyfilePlugin                  *self    = NMS_KEYFILE_PLUGIN(plugin);
+    NMSKeyfilePluginPrivate           *priv    = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
     gs_unref_object NMSKeyfileStorage *storage = g_object_ref(NMS_KEYFILE_STORAGE(storage_x));
-    const char *                       remove_from_disk_errmsg = NULL;
-    const char *                       operation_message;
-    const char *                       previous_filename;
-    const char *                       uuid;
+    const char                        *remove_from_disk_errmsg = NULL;
+    const char                        *operation_message;
+    const char                        *previous_filename;
+    const char                        *uuid;
     gboolean                           success = TRUE;
 
     _nm_assert_storage(self, storage, TRUE);
@@ -1087,24 +1087,24 @@ delete_connection(NMSettingsPlugin *plugin, NMSettingsStorage *storage_x, GError
  * Returns: %TRUE on success.
  */
 gboolean
-nms_keyfile_plugin_set_nmmeta_tombstone(NMSKeyfilePlugin *  self,
+nms_keyfile_plugin_set_nmmeta_tombstone(NMSKeyfilePlugin   *self,
                                         gboolean            simulate,
-                                        const char *        uuid,
+                                        const char         *uuid,
                                         gboolean            in_memory,
                                         gboolean            set,
-                                        const char *        shadowed_storage,
+                                        const char         *shadowed_storage,
                                         NMSettingsStorage **out_storage,
-                                        gboolean *          out_hard_failure)
+                                        gboolean           *out_hard_failure)
 {
-    NMSKeyfilePluginPrivate *priv;
-    gboolean                 hard_failure = FALSE;
-    NMSKeyfileStorage *      storage;
+    NMSKeyfilePluginPrivate           *priv;
+    gboolean                           hard_failure = FALSE;
+    NMSKeyfileStorage                 *storage;
     gs_unref_object NMSKeyfileStorage *storage_result = NULL;
     gboolean                           nmmeta_errno;
-    gs_free char *                     nmmeta_filename = NULL;
+    gs_free char                      *nmmeta_filename = NULL;
     NMSKeyfileStorageType              storage_type;
-    const char *                       loaded_path;
-    const char *                       dirname;
+    const char                        *loaded_path;
+    const char                        *dirname;
 
     nm_assert(NMS_IS_KEYFILE_PLUGIN(self));
     nm_assert(nm_uuid_is_normalized(uuid));
@@ -1197,11 +1197,11 @@ out:
 /*****************************************************************************/
 
 static void
-config_changed_cb(NMConfig *          config,
-                  NMConfigData *      config_data,
+config_changed_cb(NMConfig           *config,
+                  NMConfigData       *config_data,
                   NMConfigChangeFlags changes,
-                  NMConfigData *      old_data,
-                  NMSKeyfilePlugin *  self)
+                  NMConfigData       *old_data,
+                  NMSKeyfilePlugin   *self)
 {
     gs_free char *old_value = NULL;
     gs_free char *new_value = NULL;
@@ -1223,7 +1223,7 @@ static GSList *
 get_unmanaged_specs(NMSettingsPlugin *config)
 {
     NMSKeyfilePluginPrivate *priv  = NMS_KEYFILE_PLUGIN_GET_PRIVATE(config);
-    gs_free char *           value = NULL;
+    gs_free char            *value = NULL;
 
     value = nm_config_data_get_value(nm_config_get_data(priv->config),
                                      NM_CONFIG_KEYFILE_GROUP_KEYFILE,
@@ -1280,7 +1280,7 @@ nms_keyfile_plugin_init(NMSKeyfilePlugin *plugin)
 static void
 constructed(GObject *object)
 {
-    NMSKeyfilePlugin *       self = NMS_KEYFILE_PLUGIN(object);
+    NMSKeyfilePlugin        *self = NMS_KEYFILE_PLUGIN(object);
     NMSKeyfilePluginPrivate *priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
 
     G_OBJECT_CLASS(nms_keyfile_plugin_parent_class)->constructed(object);
@@ -1312,7 +1312,7 @@ nms_keyfile_plugin_new(void)
 static void
 dispose(GObject *object)
 {
-    NMSKeyfilePlugin *       self = NMS_KEYFILE_PLUGIN(object);
+    NMSKeyfilePlugin        *self = NMS_KEYFILE_PLUGIN(object);
     NMSKeyfilePluginPrivate *priv = NMS_KEYFILE_PLUGIN_GET_PRIVATE(self);
 
     if (priv->config)
@@ -1332,7 +1332,7 @@ dispose(GObject *object)
 static void
 nms_keyfile_plugin_class_init(NMSKeyfilePluginClass *klass)
 {
-    GObjectClass *         object_class = G_OBJECT_CLASS(klass);
+    GObjectClass          *object_class = G_OBJECT_CLASS(klass);
     NMSettingsPluginClass *plugin_class = NM_SETTINGS_PLUGIN_CLASS(klass);
 
     object_class->constructed = constructed;
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-plugin.h b/src/core/settings/plugins/keyfile/nms-keyfile-plugin.h
index 5d9f2da3..19e4122d 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-plugin.h
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-plugin.h
@@ -30,38 +30,38 @@ GType nms_keyfile_plugin_get_type(void);
 
 NMSKeyfilePlugin *nms_keyfile_plugin_new(void);
 
-gboolean nms_keyfile_plugin_add_connection(NMSKeyfilePlugin *  self,
-                                           NMConnection *      connection,
+gboolean nms_keyfile_plugin_add_connection(NMSKeyfilePlugin   *self,
+                                           NMConnection       *connection,
                                            gboolean            in_memory,
                                            gboolean            is_nm_generated,
                                            gboolean            is_volatile,
                                            gboolean            is_external,
-                                           const char *        shadowed_storage,
+                                           const char         *shadowed_storage,
                                            gboolean            shadowed_owned,
                                            NMSettingsStorage **out_storage,
-                                           NMConnection **     out_connection,
-                                           GError **           error);
+                                           NMConnection      **out_connection,
+                                           GError            **error);
 
-gboolean nms_keyfile_plugin_update_connection(NMSKeyfilePlugin *  self,
-                                              NMSettingsStorage * storage,
-                                              NMConnection *      connection,
+gboolean nms_keyfile_plugin_update_connection(NMSKeyfilePlugin   *self,
+                                              NMSettingsStorage  *storage,
+                                              NMConnection       *connection,
                                               gboolean            is_nm_generated,
                                               gboolean            is_volatile,
                                               gboolean            is_external,
-                                              const char *        shadowed_storage,
+                                              const char         *shadowed_storage,
                                               gboolean            shadowed_owned,
                                               gboolean            force_rename,
                                               NMSettingsStorage **out_storage,
-                                              NMConnection **     out_connection,
-                                              GError **           error);
+                                              NMConnection      **out_connection,
+                                              GError            **error);
 
-gboolean nms_keyfile_plugin_set_nmmeta_tombstone(NMSKeyfilePlugin *  self,
+gboolean nms_keyfile_plugin_set_nmmeta_tombstone(NMSKeyfilePlugin   *self,
                                                  gboolean            simulate,
-                                                 const char *        uuid,
+                                                 const char         *uuid,
                                                  gboolean            in_memory,
                                                  gboolean            set,
-                                                 const char *        shadowed_storage,
+                                                 const char         *shadowed_storage,
                                                  NMSettingsStorage **out_storage,
-                                                 gboolean *          out_hard_failure);
+                                                 gboolean           *out_hard_failure);
 
 #endif /* __NMS_KEYFILE_PLUGIN_H__ */
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-reader.c b/src/core/settings/plugins/keyfile/nms-keyfile-reader.c
index 8b9d982f..81e3d633 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-reader.c
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-reader.c
@@ -23,10 +23,10 @@ _fmt_warn(const NMKeyfileHandlerData *handler_data, char **out_message)
     const char *message = _nm_keyfile_handler_data_warn_get_message(handler_data);
 
     if (group) {
-        NMSetting * setting       = handler_data->cur_setting;
+        NMSetting  *setting       = handler_data->cur_setting;
         const char *property_name = handler_data->cur_property;
         const char *setting_name  = setting ? nm_setting_get_name(setting) : NULL;
-        char *      res;
+        char       *res;
 
         if (setting_name) {
             if (property_name && !strcmp(group, setting_name))
@@ -51,18 +51,18 @@ typedef struct {
 } ReadInfo;
 
 static gboolean
-_handler_read(GKeyFile *            keyfile,
-              NMConnection *        connection,
+_handler_read(GKeyFile             *keyfile,
+              NMConnection         *connection,
               NMKeyfileHandlerType  handler_type,
               NMKeyfileHandlerData *handler_data,
-              void *                user_data)
+              void                 *user_data)
 {
     const ReadInfo *read_info = user_data;
 
     if (handler_type == NM_KEYFILE_HANDLER_TYPE_WARN) {
         const NMKeyfileHandlerDataWarn *warn_data = &handler_data->warn;
         NMLogLevel                      level;
-        char *                          message_free = NULL;
+        char                           *message_free = NULL;
 
         if (!read_info->verbose)
             return TRUE;
@@ -90,21 +90,21 @@ _handler_read(GKeyFile *            keyfile,
 }
 
 NMConnection *
-nms_keyfile_reader_from_keyfile(GKeyFile *  key_file,
+nms_keyfile_reader_from_keyfile(GKeyFile   *key_file,
                                 const char *filename,
                                 const char *base_dir,
                                 const char *profile_dir,
                                 gboolean    verbose,
-                                GError **   error)
+                                GError    **error)
 {
     NMConnection *connection;
     ReadInfo      read_info = {
-        .verbose = verbose,
+             .verbose = verbose,
     };
     gs_free char *base_dir_free         = NULL;
     gs_free char *profile_filename_free = NULL;
     gs_free char *filename_id           = NULL;
-    const char *  profile_filename      = NULL;
+    const char   *profile_filename      = NULL;
 
     nm_assert(filename && filename[0]);
     nm_assert(!base_dir || base_dir[0] == '/');
@@ -154,19 +154,19 @@ nms_keyfile_reader_from_keyfile(GKeyFile *  key_file,
 }
 
 NMConnection *
-nms_keyfile_reader_from_file(const char * full_filename,
-                             const char * profile_dir,
+nms_keyfile_reader_from_file(const char  *full_filename,
+                             const char  *profile_dir,
                              struct stat *out_stat,
-                             NMTernary *  out_is_nm_generated,
-                             NMTernary *  out_is_volatile,
-                             NMTernary *  out_is_external,
-                             char **      out_shadowed_storage,
-                             NMTernary *  out_shadowed_owned,
-                             GError **    error)
+                             NMTernary   *out_is_nm_generated,
+                             NMTernary   *out_is_volatile,
+                             NMTernary   *out_is_external,
+                             char       **out_shadowed_storage,
+                             NMTernary   *out_shadowed_owned,
+                             GError     **error)
 {
     nm_auto_unref_keyfile GKeyFile *key_file     = NULL;
-    NMConnection *                  connection   = NULL;
-    GError *                        verify_error = NULL;
+    NMConnection                   *connection   = NULL;
+    GError                         *verify_error = NULL;
 
     nm_assert(full_filename && full_filename[0] == '/');
     nm_assert(!profile_dir || profile_dir[0] == '/');
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-reader.h b/src/core/settings/plugins/keyfile/nms-keyfile-reader.h
index 247457d8..3722fb8a 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-reader.h
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-reader.h
@@ -9,23 +9,23 @@
 
 #include "nm-connection.h"
 
-NMConnection *nms_keyfile_reader_from_keyfile(GKeyFile *  key_file,
+NMConnection *nms_keyfile_reader_from_keyfile(GKeyFile   *key_file,
                                               const char *filename,
                                               const char *base_dir,
                                               const char *profile_dir,
                                               gboolean    verbose,
-                                              GError **   error);
+                                              GError    **error);
 
 struct stat;
 
-NMConnection *nms_keyfile_reader_from_file(const char * full_filename,
-                                           const char * profile_dir,
+NMConnection *nms_keyfile_reader_from_file(const char  *full_filename,
+                                           const char  *profile_dir,
                                            struct stat *out_stat,
-                                           NMTernary *  out_is_nm_generated,
-                                           NMTernary *  out_is_volatile,
-                                           NMTernary *  out_is_external,
-                                           char **      out_shadowed_storage,
-                                           NMTernary *  out_shadowed_owned,
-                                           GError **    error);
+                                           NMTernary   *out_is_nm_generated,
+                                           NMTernary   *out_is_volatile,
+                                           NMTernary   *out_is_external,
+                                           char       **out_shadowed_storage,
+                                           NMTernary   *out_shadowed_owned,
+                                           GError     **error);
 
 #endif /* __NMS_KEYFILE_READER_H__ */
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-storage.c b/src/core/settings/plugins/keyfile/nms-keyfile-storage.c
index ec0634c4..ddbb3c85 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-storage.c
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-storage.c
@@ -41,7 +41,7 @@ nms_keyfile_storage_copy_content(NMSKeyfileStorage *dst, const NMSKeyfileStorage
         dst->u.meta_data.shadowed_storage = g_strdup(dst->u.meta_data.shadowed_storage);
     } else {
         gs_unref_object NMConnection *connection_to_free       = NULL;
-        gs_free char *                shadowed_storage_to_free = NULL;
+        gs_free char                 *shadowed_storage_to_free = NULL;
 
         connection_to_free       = g_steal_pointer(&dst->u.conn_data.connection);
         shadowed_storage_to_free = g_steal_pointer(&dst->u.conn_data.shadowed_storage);
@@ -99,9 +99,9 @@ nms_keyfile_storage_init(NMSKeyfileStorage *self)
 {}
 
 static NMSKeyfileStorage *
-_storage_new(NMSKeyfilePlugin *    plugin,
-             const char *          uuid,
-             const char *          filename,
+_storage_new(NMSKeyfilePlugin     *plugin,
+             const char           *uuid,
+             const char           *filename,
              gboolean              is_meta_data,
              NMSKeyfileStorageType storage_type)
 
@@ -128,11 +128,11 @@ _storage_new(NMSKeyfilePlugin *    plugin,
 }
 
 NMSKeyfileStorage *
-nms_keyfile_storage_new_tombstone(NMSKeyfilePlugin *    plugin,
-                                  const char *          uuid,
-                                  const char *          filename,
+nms_keyfile_storage_new_tombstone(NMSKeyfilePlugin     *plugin,
+                                  const char           *uuid,
+                                  const char           *filename,
                                   NMSKeyfileStorageType storage_type,
-                                  const char *          shadowed_storage)
+                                  const char           *shadowed_storage)
 {
     NMSKeyfileStorage *self;
 
@@ -149,14 +149,14 @@ nms_keyfile_storage_new_tombstone(NMSKeyfilePlugin *    plugin,
 }
 
 NMSKeyfileStorage *
-nms_keyfile_storage_new_connection(NMSKeyfilePlugin *     plugin,
-                                   NMConnection *         connection_take /* pass reference */,
-                                   const char *           filename,
+nms_keyfile_storage_new_connection(NMSKeyfilePlugin      *plugin,
+                                   NMConnection          *connection_take /* pass reference */,
+                                   const char            *filename,
                                    NMSKeyfileStorageType  storage_type,
                                    NMTernary              is_nm_generated_opt,
                                    NMTernary              is_volatile_opt,
                                    NMTernary              is_external_opt,
-                                   const char *           shadowed_storage,
+                                   const char            *shadowed_storage,
                                    NMTernary              shadowed_owned_opt,
                                    const struct timespec *stat_mtime)
 {
@@ -228,7 +228,7 @@ nms_keyfile_storage_destroy(NMSKeyfileStorage *self)
 static void
 nms_keyfile_storage_class_init(NMSKeyfileStorageClass *klass)
 {
-    GObjectClass *          object_class  = G_OBJECT_CLASS(klass);
+    GObjectClass           *object_class  = G_OBJECT_CLASS(klass);
     NMSettingsStorageClass *storage_class = NM_SETTINGS_STORAGE_CLASS(klass);
 
     object_class->dispose = dispose;
@@ -241,7 +241,7 @@ nms_keyfile_storage_class_init(NMSKeyfileStorageClass *klass)
 #include "settings/nm-settings-connection.h"
 
 void
-nm_settings_storage_load_sett_flags(NMSettingsStorage *           self,
+nm_settings_storage_load_sett_flags(NMSettingsStorage            *self,
                                     NMSettingsConnectionIntFlags *sett_flags,
                                     NMSettingsConnectionIntFlags *sett_mask)
 {
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-storage.h b/src/core/settings/plugins/keyfile/nms-keyfile-storage.h
index 008070df..8343cb2b 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-storage.h
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-storage.h
@@ -106,22 +106,22 @@ GType nms_keyfile_storage_get_type(void);
 struct _NMSKeyfilePlugin;
 
 NMSKeyfileStorage *nms_keyfile_storage_new_tombstone(struct _NMSKeyfilePlugin *self,
-                                                     const char *              uuid,
-                                                     const char *              filename,
+                                                     const char               *uuid,
+                                                     const char               *filename,
                                                      NMSKeyfileStorageType     storage_type,
-                                                     const char *              shadowed_storage);
+                                                     const char               *shadowed_storage);
 
 NMSKeyfileStorage *
 nms_keyfile_storage_new_connection(struct _NMSKeyfilePlugin *self,
-                                   NMConnection *            connection_take /* pass reference */,
-                                   const char *              filename,
+                                   NMConnection             *connection_take /* pass reference */,
+                                   const char               *filename,
                                    NMSKeyfileStorageType     storage_type,
                                    NMTernary                 is_nm_generated_opt,
                                    NMTernary                 is_volatile_opt,
                                    NMTernary                 is_external_opt,
-                                   const char *              shadowed_storage,
+                                   const char               *shadowed_storage,
                                    NMTernary                 shadowed_owned_opt,
-                                   const struct timespec *   stat_mtime);
+                                   const struct timespec    *stat_mtime);
 
 void nms_keyfile_storage_destroy(NMSKeyfileStorage *storage);
 
@@ -204,7 +204,7 @@ nm_settings_storage_is_meta_data_alive(const NMSettingsStorage *storage)
 
 static inline const char *
 nm_settings_storage_get_shadowed_storage(const NMSettingsStorage *storage,
-                                         gboolean *               out_shadowed_owned)
+                                         gboolean                *out_shadowed_owned)
 {
     if (NMS_IS_KEYFILE_STORAGE(storage)) {
         const NMSKeyfileStorage *self = (const NMSKeyfileStorage *) storage;
@@ -248,7 +248,7 @@ nm_settings_storage_get_filename_for_shadowed_storage(const NMSettingsStorage *s
 
 enum _NMSettingsConnectionIntFlags;
 
-void nm_settings_storage_load_sett_flags(NMSettingsStorage *                 self,
+void nm_settings_storage_load_sett_flags(NMSettingsStorage                  *self,
                                          enum _NMSettingsConnectionIntFlags *sett_flags,
                                          enum _NMSettingsConnectionIntFlags *sett_mask);
 
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-utils.c b/src/core/settings/plugins/keyfile/nms-keyfile-utils.c
index e2c68bbe..7c0e329e 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-utils.c
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-utils.c
@@ -95,21 +95,21 @@ nms_keyfile_nmmeta_filename(const char *dirname, const char *uuid, gboolean temp
 }
 
 gboolean
-nms_keyfile_nmmeta_read(const char * dirname,
-                        const char * filename,
-                        char **      out_full_filename,
-                        char **      out_uuid,
-                        char **      out_loaded_path,
-                        char **      out_shadowed_storage,
+nms_keyfile_nmmeta_read(const char  *dirname,
+                        const char  *filename,
+                        char       **out_full_filename,
+                        char       **out_uuid,
+                        char       **out_loaded_path,
+                        char       **out_shadowed_storage,
                         struct stat *out_st)
 {
-    const char *  uuid;
+    const char   *uuid;
     guint         uuid_len;
     gs_free char *full_filename    = NULL;
     gs_free char *loaded_path      = NULL;
     gs_free char *shadowed_storage = NULL;
     struct stat   st_stack;
-    struct stat * st = out_st ?: &st_stack;
+    struct stat  *st = out_st ?: &st_stack;
 
     nm_assert(dirname && dirname[0] == '/');
     nm_assert(filename && filename[0] && !strchr(filename, '/'));
@@ -128,7 +128,7 @@ nms_keyfile_nmmeta_read(const char * dirname,
 
     if (S_ISREG(st->st_mode)) {
         nm_auto_unref_keyfile GKeyFile *kf     = NULL;
-        gs_free char *                  v_uuid = NULL;
+        gs_free char                   *v_uuid = NULL;
 
         kf = g_key_file_new();
 
@@ -176,11 +176,11 @@ nms_keyfile_nmmeta_read(const char * dirname,
 
 gboolean
 nms_keyfile_nmmeta_read_from_file(const char *full_filename,
-                                  char **     out_dirname,
-                                  char **     out_filename,
-                                  char **     out_uuid,
-                                  char **     out_loaded_path,
-                                  char **     out_shadowed_storage)
+                                  char      **out_dirname,
+                                  char      **out_filename,
+                                  char      **out_uuid,
+                                  char      **out_loaded_path,
+                                  char      **out_shadowed_storage)
 {
     gs_free char *dirname  = NULL;
     gs_free char *filename = NULL;
@@ -210,7 +210,7 @@ nms_keyfile_nmmeta_write(const char *dirname,
                          const char *loaded_path,
                          gboolean    loaded_path_allow_relative,
                          const char *shadowed_storage,
-                         char **     out_full_filename)
+                         char      **out_full_filename)
 {
     gs_free char *full_filename_tmp = NULL;
     gs_free char *full_filename     = NULL;
@@ -255,7 +255,7 @@ nms_keyfile_nmmeta_write(const char *dirname,
 
     if (shadowed_storage) {
         nm_auto_unref_keyfile GKeyFile *kf       = NULL;
-        gs_free char *                  contents = NULL;
+        gs_free char                   *contents = NULL;
         gsize                           length;
 
         kf = g_key_file_new();
@@ -313,7 +313,7 @@ nms_keyfile_nmmeta_write(const char *dirname,
 gboolean
 nms_keyfile_utils_check_file_permissions_stat(NMSKeyfileFiletype filetype,
                                               const struct stat *st,
-                                              GError **          error)
+                                              GError           **error)
 {
     g_return_val_if_fail(st, FALSE);
 
@@ -337,7 +337,7 @@ nms_keyfile_utils_check_file_permissions_stat(NMSKeyfileFiletype filetype,
         g_return_val_if_reached(FALSE);
 
     if (!NM_FLAGS_HAS(nm_utils_get_testing(), NM_UTILS_TEST_NO_KEYFILE_OWNER_CHECK)) {
-        if (st->st_uid != 0) {
+        if (!NM_IN_SET(st->st_uid, 0, nm_utils_get_nm_uid())) {
             g_set_error(error,
                         NM_SETTINGS_ERROR,
                         NM_SETTINGS_ERROR_INVALID_CONNECTION,
@@ -361,9 +361,9 @@ nms_keyfile_utils_check_file_permissions_stat(NMSKeyfileFiletype filetype,
 
 gboolean
 nms_keyfile_utils_check_file_permissions(NMSKeyfileFiletype filetype,
-                                         const char *       filename,
-                                         struct stat *      out_st,
-                                         GError **          error)
+                                         const char        *filename,
+                                         struct stat       *out_st,
+                                         GError           **error)
 {
     struct stat st;
     int         errsv;
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-utils.h b/src/core/settings/plugins/keyfile/nms-keyfile-utils.h
index 419a17ed..0fd83bdf 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-utils.h
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-utils.h
@@ -34,38 +34,38 @@ const char *nms_keyfile_nmmeta_check_filename(const char *filename, guint *out_u
 
 char *nms_keyfile_nmmeta_filename(const char *dirname, const char *uuid, gboolean temporary);
 
-gboolean nms_keyfile_nmmeta_read(const char * dirname,
-                                 const char * filename,
-                                 char **      out_full_filename,
-                                 char **      out_uuid,
-                                 char **      out_loaded_path,
-                                 char **      out_shadowed_storage,
+gboolean nms_keyfile_nmmeta_read(const char  *dirname,
+                                 const char  *filename,
+                                 char       **out_full_filename,
+                                 char       **out_uuid,
+                                 char       **out_loaded_path,
+                                 char       **out_shadowed_storage,
                                  struct stat *out_st);
 
 gboolean nms_keyfile_nmmeta_read_from_file(const char *full_filename,
-                                           char **     out_dirname,
-                                           char **     out_filename,
-                                           char **     out_uuid,
-                                           char **     out_loaded_path,
-                                           char **     out_shadowed_storage);
+                                           char      **out_dirname,
+                                           char      **out_filename,
+                                           char      **out_uuid,
+                                           char      **out_loaded_path,
+                                           char      **out_shadowed_storage);
 
 int nms_keyfile_nmmeta_write(const char *dirname,
                              const char *uuid,
                              const char *loaded_path,
                              gboolean    loaded_path_allow_relative,
                              const char *shadowed_storage,
-                             char **     out_full_filename);
+                             char      **out_full_filename);
 
 /*****************************************************************************/
 
 struct stat;
 gboolean nms_keyfile_utils_check_file_permissions_stat(NMSKeyfileFiletype filetype,
                                                        const struct stat *st,
-                                                       GError **          error);
+                                                       GError           **error);
 
 gboolean nms_keyfile_utils_check_file_permissions(NMSKeyfileFiletype filetype,
-                                                  const char *       filename,
-                                                  struct stat *      out_st,
-                                                  GError **          error);
+                                                  const char        *filename,
+                                                  struct stat       *out_st,
+                                                  GError           **error);
 
 #endif /* __NMS_KEYFILE_UTILS_H__ */
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-writer.c b/src/core/settings/plugins/keyfile/nms-keyfile-writer.c
index 70afdc23..ad6f277c 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-writer.c
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-writer.c
@@ -26,21 +26,21 @@ typedef struct {
 } WriteInfo;
 
 static void
-cert_writer(NMConnection *                    connection,
-            GKeyFile *                        file,
-            NMSetting8021x *                  setting,
+cert_writer(NMConnection                     *connection,
+            GKeyFile                         *file,
+            NMSetting8021x                   *setting,
             const NMSetting8021xSchemeVtable *vtable,
-            WriteInfo *                       info,
-            GError **                         error)
+            WriteInfo                        *info,
+            GError                          **error)
 {
-    const char *           setting_name = nm_setting_get_name(NM_SETTING(setting));
+    const char            *setting_name = nm_setting_get_name(NM_SETTING(setting));
     NMSetting8021xCKScheme scheme;
     NMSetting8021xCKFormat format;
-    const char *           path = NULL, *ext = "pem";
+    const char            *path = NULL, *ext = "pem";
 
     scheme = vtable->scheme_func(setting);
     if (scheme == NM_SETTING_802_1X_CK_SCHEME_PATH) {
-        char *      tmp           = NULL;
+        char       *tmp           = NULL;
         const char *accepted_path = NULL;
 
         path = vtable->path_func(setting);
@@ -92,12 +92,12 @@ cert_writer(NMConnection *                    connection,
                                         vtable->setting_key,
                                         vtable->uri_func(setting));
     } else if (scheme == NM_SETTING_802_1X_CK_SCHEME_BLOB) {
-        GBytes *      blob;
+        GBytes       *blob;
         const guint8 *blob_data;
         gsize         blob_len;
         gboolean      success;
-        GError *      local = NULL;
-        char *        new_path;
+        GError       *local = NULL;
+        char         *new_path;
 
         blob = vtable->blob_func(setting);
         g_assert(blob);
@@ -163,11 +163,11 @@ cert_writer(NMConnection *                    connection,
 }
 
 static gboolean
-_handler_write(NMConnection *        connection,
-               GKeyFile *            keyfile,
+_handler_write(NMConnection         *connection,
+               GKeyFile             *keyfile,
                NMKeyfileHandlerType  type,
                NMKeyfileHandlerData *type_data,
-               void *                user_data)
+               void                 *user_data)
 {
     if (type == NM_KEYFILE_HANDLER_TYPE_WRITE_CERT) {
         cert_writer(connection,
@@ -182,39 +182,39 @@ _handler_write(NMConnection *        connection,
 }
 
 static gboolean
-_internal_write_connection(NMConnection *                  connection,
+_internal_write_connection(NMConnection                   *connection,
                            gboolean                        is_nm_generated,
                            gboolean                        is_volatile,
                            gboolean                        is_external,
-                           const char *                    shadowed_storage,
+                           const char                     *shadowed_storage,
                            gboolean                        shadowed_owned,
-                           const char *                    keyfile_dir,
-                           const char *                    profile_dir,
+                           const char                     *keyfile_dir,
+                           const char                     *profile_dir,
                            gboolean                        with_extension,
                            uid_t                           owner_uid,
                            pid_t                           owner_grp,
-                           const char *                    existing_path,
+                           const char                     *existing_path,
                            gboolean                        existing_path_read_only,
                            gboolean                        force_rename,
                            NMSKeyfileWriterAllowFilenameCb allow_filename_cb,
                            gpointer                        allow_filename_user_data,
-                           char **                         out_path,
-                           NMConnection **                 out_reread,
-                           gboolean *                      out_reread_same,
-                           GError **                       error)
+                           char                          **out_path,
+                           NMConnection                  **out_reread,
+                           gboolean                       *out_reread_same,
+                           GError                        **error)
 {
     nm_auto_unref_keyfile GKeyFile *kf_file        = NULL;
-    gs_free char *                  kf_content_buf = NULL;
+    gs_free char                   *kf_content_buf = NULL;
     gsize                           kf_content_len;
-    gs_free char *                  path = NULL;
-    const char *                    id;
-    WriteInfo                       info = {0};
-    gs_free_error GError *local_err      = NULL;
-    int                   errsv;
-    gboolean              rename;
-    int                   i_path;
-    gs_unref_object NMConnection *reread      = NULL;
-    gboolean                      reread_same = FALSE;
+    gs_free char                   *path = NULL;
+    const char                     *id;
+    WriteInfo                       info      = {0};
+    gs_free_error GError           *local_err = NULL;
+    int                             errsv;
+    gboolean                        rename;
+    int                             i_path;
+    gs_unref_object NMConnection   *reread      = NULL;
+    gboolean                        reread_same = FALSE;
 
     g_return_val_if_fail(!out_path || !*out_path, FALSE);
     g_return_val_if_fail(keyfile_dir && keyfile_dir[0] == '/', FALSE);
@@ -417,23 +417,23 @@ _internal_write_connection(NMConnection *                  connection,
 }
 
 gboolean
-nms_keyfile_writer_connection(NMConnection *                  connection,
+nms_keyfile_writer_connection(NMConnection                   *connection,
                               gboolean                        is_nm_generated,
                               gboolean                        is_volatile,
                               gboolean                        is_external,
-                              const char *                    shadowed_storage,
+                              const char                     *shadowed_storage,
                               gboolean                        shadowed_owned,
-                              const char *                    keyfile_dir,
-                              const char *                    profile_dir,
-                              const char *                    existing_path,
+                              const char                     *keyfile_dir,
+                              const char                     *profile_dir,
+                              const char                     *existing_path,
                               gboolean                        existing_path_read_only,
                               gboolean                        force_rename,
                               NMSKeyfileWriterAllowFilenameCb allow_filename_cb,
                               gpointer                        allow_filename_user_data,
-                              char **                         out_path,
-                              NMConnection **                 out_reread,
-                              gboolean *                      out_reread_same,
-                              GError **                       error)
+                              char                          **out_path,
+                              NMConnection                  **out_reread,
+                              gboolean                       *out_reread_same,
+                              GError                        **error)
 {
     return _internal_write_connection(connection,
                                       is_nm_generated,
@@ -444,8 +444,8 @@ nms_keyfile_writer_connection(NMConnection *                  connection,
                                       keyfile_dir,
                                       profile_dir,
                                       TRUE,
-                                      0,
-                                      0,
+                                      nm_utils_get_nm_uid(),
+                                      nm_utils_get_nm_gid(),
                                       existing_path,
                                       existing_path_read_only,
                                       force_rename,
@@ -458,14 +458,14 @@ nms_keyfile_writer_connection(NMConnection *                  connection,
 }
 
 gboolean
-nms_keyfile_writer_test_connection(NMConnection * connection,
-                                   const char *   keyfile_dir,
+nms_keyfile_writer_test_connection(NMConnection  *connection,
+                                   const char    *keyfile_dir,
                                    uid_t          owner_uid,
                                    pid_t          owner_grp,
-                                   char **        out_path,
+                                   char         **out_path,
                                    NMConnection **out_reread,
-                                   gboolean *     out_reread_same,
-                                   GError **      error)
+                                   gboolean      *out_reread_same,
+                                   GError       **error)
 {
     return _internal_write_connection(connection,
                                       FALSE,
diff --git a/src/core/settings/plugins/keyfile/nms-keyfile-writer.h b/src/core/settings/plugins/keyfile/nms-keyfile-writer.h
index dffd3b04..62aaa19d 100644
--- a/src/core/settings/plugins/keyfile/nms-keyfile-writer.h
+++ b/src/core/settings/plugins/keyfile/nms-keyfile-writer.h
@@ -12,31 +12,31 @@
 typedef gboolean (*NMSKeyfileWriterAllowFilenameCb)(const char *check_filename,
                                                     gpointer    allow_filename_user_data);
 
-gboolean nms_keyfile_writer_connection(NMConnection *                  connection,
+gboolean nms_keyfile_writer_connection(NMConnection                   *connection,
                                        gboolean                        is_nm_generated,
                                        gboolean                        is_volatile,
                                        gboolean                        is_external,
-                                       const char *                    shadowed_storage,
+                                       const char                     *shadowed_storage,
                                        gboolean                        shadowed_owned,
-                                       const char *                    keyfile_dir,
-                                       const char *                    profile_dir,
-                                       const char *                    existing_path,
+                                       const char                     *keyfile_dir,
+                                       const char                     *profile_dir,
+                                       const char                     *existing_path,
                                        gboolean                        existing_path_read_only,
                                        gboolean                        force_rename,
                                        NMSKeyfileWriterAllowFilenameCb allow_filename_cb,
                                        gpointer                        allow_filename_user_data,
-                                       char **                         out_path,
-                                       NMConnection **                 out_reread,
-                                       gboolean *                      out_reread_same,
-                                       GError **                       error);
+                                       char                          **out_path,
+                                       NMConnection                  **out_reread,
+                                       gboolean                       *out_reread_same,
+                                       GError                        **error);
 
-gboolean nms_keyfile_writer_test_connection(NMConnection * connection,
-                                            const char *   keyfile_dir,
+gboolean nms_keyfile_writer_test_connection(NMConnection  *connection,
+                                            const char    *keyfile_dir,
                                             uid_t          owner_uid,
                                             pid_t          owner_grp,
-                                            char **        out_path,
+                                            char         **out_path,
                                             NMConnection **out_reread,
-                                            gboolean *     out_reread_same,
-                                            GError **      error);
+                                            gboolean      *out_reread_same,
+                                            GError       **error);
 
 #endif /* __NMS_KEYFILE_WRITER_H__ */
diff --git a/src/core/settings/plugins/keyfile/tests/test-keyfile-settings.c b/src/core/settings/plugins/keyfile/tests/test-keyfile-settings.c
index 4ceed49d..b4c30df7 100644
--- a/src/core/settings/plugins/keyfile/tests/test-keyfile-settings.c
+++ b/src/core/settings/plugins/keyfile/tests/test-keyfile-settings.c
@@ -42,9 +42,9 @@ check_ip_address(NMSettingIPConfig *config, int idx, const char *address, int pl
 static void
 check_ip_route(NMSettingIPConfig *config,
                int                idx,
-               const char *       destination,
+               const char        *destination,
                int                plen,
-               const char *       next_hop,
+               const char        *next_hop,
                gint64             metric)
 {
     NMIPRoute *route = nm_setting_ip_config_get_route(config, idx);
@@ -59,7 +59,7 @@ check_ip_route(NMSettingIPConfig *config,
 #define keyfile_read_connection_from_file(full_filename)                                  \
     ({                                                                                    \
         gs_free_error GError *_error = NULL;                                              \
-        NMConnection *        _connection;                                                \
+        NMConnection         *_connection;                                                \
                                                                                           \
         g_assert(full_filename &&full_filename[0] == '/');                                \
                                                                                           \
@@ -84,7 +84,7 @@ assert_reread(NMConnection *connection, gboolean normalize_connection, const cha
 {
     gs_unref_object NMConnection *reread           = NULL;
     gs_unref_object NMConnection *connection_clone = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     g_assert(NM_IS_CONNECTION(connection));
 
@@ -104,7 +104,7 @@ assert_reread(NMConnection *connection, gboolean normalize_connection, const cha
 static void
 assert_reread_and_unlink(NMConnection *connection,
                          gboolean      normalize_connection,
-                         const char *  testfile)
+                         const char   *testfile)
 {
     assert_reread(connection, normalize_connection, testfile);
     unlink(testfile);
@@ -118,16 +118,16 @@ assert_reread_same(NMConnection *connection, NMConnection *reread)
 }
 
 static void
-write_test_connection_reread(NMConnection * connection,
-                             char **        testfile,
+write_test_connection_reread(NMConnection  *connection,
+                             char         **testfile,
                              NMConnection **out_reread,
-                             gboolean *     out_reread_same)
+                             gboolean      *out_reread_same)
 {
-    uid_t           owner_uid;
-    gid_t           owner_grp;
-    gboolean        success;
-    GError *        error   = NULL;
-    GError **       p_error = (nmtst_get_rand_uint32() % 2) ? &error : NULL;
+    uid_t                         owner_uid;
+    gid_t                         owner_grp;
+    gboolean                      success;
+    GError                       *error   = NULL;
+    GError                      **p_error = (nmtst_get_rand_uint32() % 2) ? &error : NULL;
     gs_unref_object NMConnection *connection_normalized = NULL;
 
     g_assert(NM_IS_CONNECTION(connection));
@@ -177,7 +177,7 @@ static GKeyFile *
 keyfile_load_from_file(const char *testfile)
 {
     GKeyFile *keyfile;
-    GError *  error = NULL;
+    GError   *error = NULL;
     gboolean  success;
 
     g_assert(testfile && *testfile);
@@ -193,12 +193,12 @@ keyfile_load_from_file(const char *testfile)
 static void
 _setting_copy_property_gbytes(NMConnection *src,
                               NMConnection *dst,
-                              const char *  setting_name,
-                              const char *  property_name)
+                              const char   *setting_name,
+                              const char   *property_name)
 {
     gs_unref_bytes GBytes *blob = NULL;
-    NMSetting *            s_src;
-    NMSetting *            s_dst;
+    NMSetting             *s_src;
+    NMSetting             *s_dst;
 
     g_assert(NM_IS_CONNECTION(src));
     g_assert(NM_IS_CONNECTION(dst));
@@ -220,12 +220,12 @@ static void
 test_read_valid_wired_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPRoute *                   route;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPRoute                    *route;
+    const char                   *mac;
     char expected_mac_address[ETH_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0x55};
 
     NMTST_EXPECT_NM_INFO("*ipv4.addresses:*semicolon at the end*addresses1*");
@@ -370,7 +370,7 @@ test_read_valid_wired_connection(void)
 static void
 add_one_ip_address(NMSettingIPConfig *s_ip, const char *addr, guint32 prefix)
 {
-    NMIPAddress * ip_addr;
+    NMIPAddress          *ip_addr;
     gs_free_error GError *error = NULL;
 
     ip_addr = nm_ip_address_new(NM_IS_SETTING_IP4_CONFIG(s_ip) ? AF_INET : AF_INET6,
@@ -384,12 +384,12 @@ add_one_ip_address(NMSettingIPConfig *s_ip, const char *addr, guint32 prefix)
 
 static void
 add_one_ip_route(NMSettingIPConfig *s_ip,
-                 const char *       dest,
-                 const char *       nh,
+                 const char        *dest,
+                 const char        *nh,
                  guint32            prefix,
                  gint64             metric)
 {
-    NMIPRoute *   route;
+    NMIPRoute            *route;
     gs_free_error GError *error = NULL;
 
     g_assert(prefix > 0);
@@ -409,39 +409,39 @@ test_write_wired_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPRoute *                   rt;
-    const char *                  mac         = "99:88:77:66:55:44";
-    const char *                  dns1        = "4.2.2.1";
-    const char *                  dns2        = "4.2.2.2";
-    const char *                  address1    = "192.168.0.5";
-    const char *                  address2    = "1.2.3.4";
-    const char *                  gw          = "192.168.0.1";
-    const char *                  route1      = "10.10.10.2";
-    const char *                  route1_nh   = "10.10.10.1";
-    const char *                  route2      = "1.1.1.1";
-    const char *                  route2_nh   = "1.2.1.1";
-    const char *                  route3      = "2.2.2.2";
-    const char *                  route3_nh   = NULL;
-    const char *                  route4      = "3.3.3.3";
-    const char *                  route4_nh   = NULL;
-    const char *                  dns6_1      = "1::cafe";
-    const char *                  dns6_2      = "2::cafe";
-    const char *                  address6_1  = "abcd::beef";
-    const char *                  address6_2  = "dcba::beef";
-    const char *                  route6_1    = "1:2:3:4:5:6:7:8";
-    const char *                  route6_1_nh = "8:7:6:5:4:3:2:1";
-    const char *                  route6_2    = "2001::1000";
-    const char *                  route6_2_nh = "2001::1111";
-    const char *                  route6_3    = "4:5:6:7:8:9:0:1";
-    const char *                  route6_3_nh = "::";
-    const char *                  route6_4    = "5:6:7:8:9:0:1:2";
-    const char *                  route6_4_nh = "::";
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPRoute                    *rt;
+    const char                   *mac         = "99:88:77:66:55:44";
+    const char                   *dns1        = "4.2.2.1";
+    const char                   *dns2        = "4.2.2.2";
+    const char                   *address1    = "192.168.0.5";
+    const char                   *address2    = "1.2.3.4";
+    const char                   *gw          = "192.168.0.1";
+    const char                   *route1      = "10.10.10.2";
+    const char                   *route1_nh   = "10.10.10.1";
+    const char                   *route2      = "1.1.1.1";
+    const char                   *route2_nh   = "1.2.1.1";
+    const char                   *route3      = "2.2.2.2";
+    const char                   *route3_nh   = NULL;
+    const char                   *route4      = "3.3.3.3";
+    const char                   *route4_nh   = NULL;
+    const char                   *dns6_1      = "1::cafe";
+    const char                   *dns6_2      = "2::cafe";
+    const char                   *address6_1  = "abcd::beef";
+    const char                   *address6_2  = "dcba::beef";
+    const char                   *route6_1    = "1:2:3:4:5:6:7:8";
+    const char                   *route6_1_nh = "8:7:6:5:4:3:2:1";
+    const char                   *route6_2    = "2001::1000";
+    const char                   *route6_2_nh = "2001::1111";
+    const char                   *route6_3    = "4:5:6:7:8:9:0:1";
+    const char                   *route6_3_nh = "::";
+    const char                   *route6_4    = "5:6:7:8:9:0:1:2";
+    const char                   *route6_4_nh = "::";
     guint64                       timestamp   = 0x12345678L;
-    GError *                      error       = NULL;
+    GError                       *error       = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -534,10 +534,10 @@ static void
 test_read_ip6_wired_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Wired_Connection_IP6");
 
@@ -573,13 +573,13 @@ test_write_ip6_wired_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char *                  dns     = "1::cafe";
-    const char *                  address = "abcd::beef";
-    const char *                  gw      = "dcba::beef";
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *dns     = "1::cafe";
+    const char                   *address = "abcd::beef";
+    const char                   *gw      = "dcba::beef";
 
     connection = nm_simple_connection_new();
 
@@ -639,9 +639,9 @@ static void
 test_read_wired_mac_case(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    const char                   *mac;
     char expected_mac_address[ETH_ALEN] = {0x00, 0x11, 0xaa, 0xbb, 0xcc, 0x55};
 
     NMTST_EXPECT_NM_INFO("*ipv4.addresses*semicolon at the end*addresses1*");
@@ -669,8 +669,8 @@ static void
 test_read_mac_old_format(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    const char *                  mac;
+    NMSettingWired               *s_wired;
+    const char                   *mac;
     char                          expected_mac[ETH_ALEN] = {0x00, 0x11, 0xaa, 0xbb, 0xcc, 0x55};
     char expected_cloned_mac[ETH_ALEN]                   = {0x00, 0x16, 0xaa, 0xbb, 0xcc, 0xfe};
 
@@ -692,8 +692,8 @@ static void
 test_read_mac_ib_old_format(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingInfiniband *         s_ib;
-    const char *                  mac;
+    NMSettingInfiniband          *s_ib;
+    const char                   *mac;
     guint8 expected_mac[INFINIBAND_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0x55, 0x66,
                                             0x77, 0x88, 0x99, 0x01, 0x12, 0x23, 0x34,
                                             0x45, 0x56, 0x67, 0x78, 0x89, 0x90};
@@ -713,10 +713,10 @@ static void
 test_read_valid_wireless_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wireless;
-    NMSettingIPConfig *           s_ip4;
-    const char *                  bssid;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wireless;
+    NMSettingIPConfig            *s_ip4;
+    const char                   *bssid;
     const guint8                  expected_bssid[ETH_ALEN] = {0x00, 0x1a, 0x33, 0x44, 0x99, 0x82};
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Wireless_Connection");
@@ -746,12 +746,12 @@ test_write_wireless_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wireless;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char *                  bssid = "aa:b9:a1:74:55:44";
-    GBytes *                      ssid;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wireless;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *bssid = "aa:b9:a1:74:55:44";
+    GBytes                       *ssid;
     unsigned char                 tmpssid[] = {0x31, 0x33, 0x33, 0x37};
     guint64                       timestamp = 0x12344433L;
 
@@ -814,11 +814,11 @@ static void
 test_read_string_ssid(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWireless *           s_wireless;
-    GBytes *                      ssid;
-    const guint8 *                ssid_data;
+    NMSettingWireless            *s_wireless;
+    GBytes                       *ssid;
+    const guint8                 *ssid_data;
     gsize                         ssid_len;
-    const char *                  expected_ssid = "blah blah ssid 1234";
+    const char                   *expected_ssid = "blah blah ssid 1234";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_String_SSID");
 
@@ -836,15 +836,15 @@ static void
 test_write_string_ssid(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wireless;
-    NMSettingIPConfig *           s_ip4;
-    char *                        tmp;
-    gs_free char *                testfile = NULL;
-    GBytes *                      ssid;
-    unsigned char                 tmpssid[] = {65, 49, 50, 51, 32, 46, 92, 46, 36, 37, 126, 93};
-    nm_auto_unref_keyfile GKeyFile *keyfile = NULL;
+    gs_unref_object NMConnection   *connection = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWireless              *s_wireless;
+    NMSettingIPConfig              *s_ip4;
+    char                           *tmp;
+    gs_free char                   *testfile = NULL;
+    GBytes                         *ssid;
+    unsigned char                   tmpssid[] = {65, 49, 50, 51, 32, 46, 92, 46, 36, 37, 126, 93};
+    nm_auto_unref_keyfile GKeyFile *keyfile   = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -894,11 +894,11 @@ static void
 test_read_intlist_ssid(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const guint8 *                ssid_data;
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const guint8                 *ssid_data;
     gsize                         ssid_len;
-    const char *                  expected_ssid = "blah1234";
+    const char                   *expected_ssid = "blah1234";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Intlist_SSID");
 
@@ -915,16 +915,16 @@ static void
 test_write_intlist_ssid(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    gs_free char *                testfile = NULL;
-    GBytes *                      ssid;
-    unsigned char                 tmpssid[] = {65, 49, 50, 51, 0, 50, 50};
-    gs_free_error GError *error             = NULL;
-    nm_auto_unref_keyfile GKeyFile *keyfile = NULL;
-    int *                           intlist;
+    gs_unref_object NMConnection   *connection = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWireless              *s_wifi;
+    NMSettingIPConfig              *s_ip4;
+    gs_free char                   *testfile = NULL;
+    GBytes                         *ssid;
+    unsigned char                   tmpssid[] = {65, 49, 50, 51, 0, 50, 50};
+    gs_free_error GError           *error     = NULL;
+    nm_auto_unref_keyfile GKeyFile *keyfile   = NULL;
+    int                            *intlist;
     gsize                           len = 0, i;
 
     connection = nm_simple_connection_new();
@@ -981,9 +981,9 @@ static void
 test_read_intlike_ssid(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "101";
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "101";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Intlike_SSID");
 
@@ -999,9 +999,9 @@ static void
 test_read_intlike_ssid_2(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "11;12;13;";
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "11;12;13;";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Intlike_SSID_2");
 
@@ -1017,16 +1017,16 @@ static void
 test_write_intlike_ssid(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    gs_free char *                testfile = NULL;
-    GBytes *                      ssid;
-    unsigned char                 tmpssid[] = {49, 48, 49};
-    gs_free_error GError *error             = NULL;
-    nm_auto_unref_keyfile GKeyFile *keyfile = NULL;
-    char *                          tmp;
+    gs_unref_object NMConnection   *connection = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWireless              *s_wifi;
+    NMSettingIPConfig              *s_ip4;
+    gs_free char                   *testfile = NULL;
+    GBytes                         *ssid;
+    unsigned char                   tmpssid[] = {49, 48, 49};
+    gs_free_error GError           *error     = NULL;
+    nm_auto_unref_keyfile GKeyFile *keyfile   = NULL;
+    char                           *tmp;
 
     connection = nm_simple_connection_new();
 
@@ -1078,16 +1078,16 @@ static void
 test_write_intlike_ssid_2(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    gs_free char *                testfile = NULL;
-    GBytes *                      ssid;
-    unsigned char                 tmpssid[] = {49, 49, 59, 49, 50, 59, 49, 51, 59};
-    gs_free_error GError *error             = NULL;
-    nm_auto_unref_keyfile GKeyFile *keyfile = NULL;
-    char *                          tmp;
+    gs_unref_object NMConnection   *connection = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWireless              *s_wifi;
+    NMSettingIPConfig              *s_ip4;
+    gs_free char                   *testfile = NULL;
+    GBytes                         *ssid;
+    unsigned char                   tmpssid[] = {49, 49, 59, 49, 50, 59, 49, 51, 59};
+    gs_free_error GError           *error     = NULL;
+    nm_auto_unref_keyfile GKeyFile *keyfile   = NULL;
+    char                           *tmp;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -1140,11 +1140,11 @@ static void
 test_read_bt_dun_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBluetooth *          s_bluetooth;
-    NMSettingSerial *             s_serial;
-    NMSettingGsm *                s_gsm;
-    const char *                  bdaddr;
+    NMSettingConnection          *s_con;
+    NMSettingBluetooth           *s_bluetooth;
+    NMSettingSerial              *s_serial;
+    NMSettingGsm                 *s_gsm;
+    const char                   *bdaddr;
     const guint8                  expected_bdaddr[ETH_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0x55};
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/ATT_Data_Connect_BT");
@@ -1181,11 +1181,11 @@ test_write_bt_dun_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBluetooth *          s_bt;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingGsm *                s_gsm;
-    const char *                  bdaddr    = "aa:b9:a1:74:55:44";
+    NMSettingConnection          *s_con;
+    NMSettingBluetooth           *s_bt;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingGsm                 *s_gsm;
+    const char                   *bdaddr    = "aa:b9:a1:74:55:44";
     guint64                       timestamp = 0x12344433L;
 
     connection = nm_simple_connection_new();
@@ -1247,9 +1247,9 @@ static void
 test_read_gsm_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingSerial *             s_serial;
-    NMSettingGsm *                s_gsm;
+    NMSettingConnection          *s_con;
+    NMSettingSerial              *s_serial;
+    NMSettingGsm                 *s_gsm;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/ATT_Data_Connect_Plain");
 
@@ -1286,9 +1286,9 @@ test_write_gsm_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingGsm *                s_gsm;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingGsm                 *s_gsm;
     guint64                       timestamp = 0x12344433L;
 
     connection = nm_simple_connection_new();
@@ -1350,10 +1350,10 @@ static void
 test_read_wired_8021x_tls_blob_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    NMSetting8021x *              s_8021x;
-    const char *                  tmp;
-    GBytes *                      blob;
+    NMSettingWired               *s_wired;
+    NMSetting8021x               *s_8021x;
+    const char                   *tmp;
+    GBytes                       *blob;
 
     NMTST_EXPECT_NM_WARN("keyfile: 802-1x.client-cert: certificate or key file "
                          "'/CASA/dcbw/Desktop/certinfra/client.pem' does not exist*");
@@ -1403,10 +1403,10 @@ static void
 test_read_wired_8021x_tls_bad_path_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    NMSetting8021x *              s_8021x;
-    const char *                  tmp;
-    char *                        tmp2;
+    NMSettingWired               *s_wired;
+    NMSetting8021x               *s_8021x;
+    const char                   *tmp;
+    char                         *tmp2;
 
     NMTST_EXPECT_NM_WARN("*does not exist*");
     connection =
@@ -1451,9 +1451,9 @@ static void
 test_read_wired_8021x_tls_old_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    NMSetting8021x *              s_8021x;
-    const char *                  tmp;
+    NMSettingWired               *s_wired;
+    NMSetting8021x               *s_8021x;
+    const char                   *tmp;
 
     NMTST_EXPECT_NM_WARN("keyfile: 802-1x.ca-cert: certificate or key file "
                          "'/CASA/dcbw/Desktop/certinfra/CA/eaptest_ca_cert.pem' does not exist*");
@@ -1493,10 +1493,10 @@ static void
 test_read_wired_8021x_tls_new_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    NMSetting8021x *              s_8021x;
-    const char *                  tmp;
-    char *                        tmp2;
+    NMSettingWired               *s_wired;
+    NMSetting8021x               *s_8021x;
+    const char                   *tmp;
+    char                         *tmp2;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Wired_TLS_New");
 
@@ -1540,12 +1540,12 @@ static NMConnection *
 create_wired_tls_connection(NMSetting8021xCKScheme scheme)
 {
     NMTST_UUID_INIT(uuid);
-    NMConnection *       connection;
-    NMSettingConnection *s_con;
-    NMSettingIPConfig *  s_ip4;
-    NMSetting *          s_wired;
-    NMSetting8021x *     s_8021x;
-    gboolean             success;
+    NMConnection         *connection;
+    NMSettingConnection  *s_con;
+    NMSettingIPConfig    *s_ip4;
+    NMSetting            *s_wired;
+    NMSetting8021x       *s_8021x;
+    gboolean              success;
     gs_free_error GError *error = NULL;
 
     connection = nm_simple_connection_new();
@@ -1617,11 +1617,11 @@ get_path(const char *file, gboolean relative)
 static void
 test_write_wired_8021x_tls_connection_path(void)
 {
-    gs_unref_object NMConnection *connection = NULL;
-    gs_unref_object NMConnection *reread     = NULL;
-    char *                        tmp, *tmp2;
-    gboolean                      success;
-    gs_free char *                testfile      = NULL;
+    gs_unref_object NMConnection   *connection = NULL;
+    gs_unref_object NMConnection   *reread     = NULL;
+    char                           *tmp, *tmp2;
+    gboolean                        success;
+    gs_free char                   *testfile    = NULL;
     nm_auto_unref_keyfile GKeyFile *keyfile     = NULL;
     gboolean                        relative    = FALSE;
     gboolean                        reread_same = FALSE;
@@ -1705,15 +1705,15 @@ test_write_wired_8021x_tls_connection_blob(void)
 {
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSetting8021x *              s_8021x;
-    gs_free char *                testfile = NULL;
-    char *                        new_ca_cert;
-    char *                        new_client_cert;
-    char *                        new_priv_key;
-    const char *                  uuid;
+    NMSettingConnection          *s_con;
+    NMSetting8021x               *s_8021x;
+    gs_free char                 *testfile = NULL;
+    char                         *new_ca_cert;
+    char                         *new_client_cert;
+    char                         *new_priv_key;
+    const char                   *uuid;
     gboolean                      reread_same = FALSE;
-    GBytes *                      password_raw;
+    GBytes                       *password_raw;
 
 #define PASSWORD_RAW "password-raw\0test"
 
@@ -1801,9 +1801,9 @@ static void
 test_read_infiniband_connection(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingInfiniband *         s_ib;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingInfiniband          *s_ib;
+    const char                   *mac;
     guint8      expected_mac[INFINIBAND_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0x55, 0x66,
                                             0x77, 0x88, 0x99, 0x01, 0x12, 0x23, 0x34,
                                             0x45, 0x56, 0x67, 0x78, 0x89, 0x90};
@@ -1830,10 +1830,10 @@ test_write_infiniband_connection(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingInfiniband *         s_ib;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingInfiniband          *s_ib;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     const char *mac = "99:88:77:66:55:44:ab:bc:cd:de:ef:f0:0a:1b:2c:3d:4e:5f:6f:ba";
 
     connection = nm_simple_connection_new();
@@ -1888,11 +1888,11 @@ static void
 test_read_bridge_main(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingBridge *             s_bridge;
-    const char *                  expected_id   = "Test Bridge Main";
-    const char *                  expected_uuid = "8f061643-fe41-4d4c-a8d9-097d26e2ad3a";
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingBridge              *s_bridge;
+    const char                   *expected_id   = "Test Bridge Main";
+    const char                   *expected_uuid = "8f061643-fe41-4d4c-a8d9-097d26e2ad3a";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Bridge_Main");
 
@@ -1922,10 +1922,10 @@ test_write_bridge_main(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBridge *             s_bridge;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingBridge              *s_bridge;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -1981,13 +1981,13 @@ static void
 test_read_bridge_component(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBridgePort *         s_port;
-    NMSettingWired *              s_wired;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingBridgePort          *s_port;
+    NMSettingWired               *s_wired;
+    const char                   *mac;
     guint8                        expected_mac[ETH_ALEN] = {0x00, 0x22, 0x15, 0x59, 0x62, 0x97};
-    const char *                  expected_id            = "Test Bridge Component";
-    const char *                  expected_uuid          = "d7b4f96c-c45e-4298-bef8-f48574f8c1c0";
+    const char                   *expected_id            = "Test Bridge Component";
+    const char                   *expected_uuid          = "d7b4f96c-c45e-4298-bef8-f48574f8c1c0";
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Bridge_Component");
 
@@ -2016,10 +2016,10 @@ test_write_bridge_component(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBridgePort *         s_port;
-    NMSettingWired *              s_wired;
-    const char *                  mac = "99:88:77:66:55:44";
+    NMSettingConnection          *s_con;
+    NMSettingBridgePort          *s_port;
+    NMSettingWired               *s_wired;
+    const char                   *mac = "99:88:77:66:55:44";
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -2070,8 +2070,8 @@ static void
 test_read_new_wired_group_name(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWired *              s_wired;
-    const char *                  mac;
+    NMSettingWired               *s_wired;
+    const char                   *mac;
     guint8                        expected_mac[ETH_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0x55};
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_New_Wired_Group_Name");
@@ -2089,14 +2089,14 @@ static void
 test_write_new_wired_group_name(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    nm_auto_unref_keyfile GKeyFile *kf       = NULL;
-    NMSettingConnection *           s_con;
-    NMSettingWired *                s_wired;
-    gs_free char *                  testfile = NULL;
-    gs_free_error GError *error              = NULL;
-    char *                s;
-    int                   mtu;
+    gs_unref_object NMConnection   *connection = NULL;
+    nm_auto_unref_keyfile GKeyFile *kf         = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWired                 *s_wired;
+    gs_free char                   *testfile = NULL;
+    gs_free_error GError           *error    = NULL;
+    char                           *s;
+    int                             mtu;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -2147,10 +2147,10 @@ static void
 test_read_new_wireless_group_names(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "foobar";
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "foobar";
 
     connection =
         keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_New_Wireless_Group_Names");
@@ -2174,17 +2174,17 @@ static void
 test_write_new_wireless_group_names(void)
 {
     NMTST_UUID_INIT(uuid);
-    gs_unref_object NMConnection *connection = NULL;
-    nm_auto_unref_keyfile GKeyFile *kf       = NULL;
-    NMSettingConnection *           s_con;
-    NMSettingWireless *             s_wifi;
-    NMSettingWirelessSecurity *     s_wsec;
-    GBytes *                        ssid;
+    gs_unref_object NMConnection   *connection = NULL;
+    nm_auto_unref_keyfile GKeyFile *kf         = NULL;
+    NMSettingConnection            *s_con;
+    NMSettingWireless              *s_wifi;
+    NMSettingWirelessSecurity      *s_wsec;
+    GBytes                         *ssid;
     unsigned char                   tmpssid[]    = {0x31, 0x33, 0x33, 0x37};
-    const char *                    expected_psk = "asdfasdfasdfa12315";
-    gs_free char *                  testfile     = NULL;
-    gs_free_error GError *error                  = NULL;
-    char *                s;
+    const char                     *expected_psk = "asdfasdfasdfa12315";
+    gs_free char                   *testfile     = NULL;
+    gs_free_error GError           *error        = NULL;
+    char                           *s;
 
     connection = nm_simple_connection_new();
 
@@ -2257,7 +2257,7 @@ static void
 test_read_missing_vlan_setting(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Missing_Vlan_Setting");
 
@@ -2271,7 +2271,7 @@ static void
 test_read_missing_vlan_flags(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Missing_Vlan_Flags");
 
@@ -2287,8 +2287,8 @@ static void
 test_read_missing_id_uuid(void)
 {
     gs_unref_object NMConnection *connection    = NULL;
-    gs_free char *                expected_uuid = NULL;
-    const char *                  FILENAME      = TEST_KEYFILES_DIR "/Test_Missing_ID_UUID";
+    gs_free char                 *expected_uuid = NULL;
+    const char                   *FILENAME      = TEST_KEYFILES_DIR "/Test_Missing_ID_UUID";
 
     expected_uuid = nm_uuid_generate_from_strings("keyfile", FILENAME, NULL);
 
@@ -2303,7 +2303,7 @@ test_read_minimal(void)
 {
     gs_unref_object NMConnection *connection    = NULL;
     gs_unref_object NMConnection *con_archetype = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     con_archetype = nmtst_create_minimal_connection("Test_minimal_x",
                                                     "a15bd68f-c32b-40b8-8d27-49e472a85919",
@@ -2337,7 +2337,7 @@ test_read_minimal_slave(void)
 {
     gs_unref_object NMConnection *connection    = NULL;
     gs_unref_object NMConnection *con_archetype = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     con_archetype = nmtst_create_minimal_connection("Test_minimal_slave_x",
                                                     "a56b4ca5-7075-43d4-82c7-5d0cb15f7654",
@@ -2396,7 +2396,7 @@ static void
 test_read_enum_property(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingIPConfig            *s_ip6;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Enum_Property");
 
@@ -2412,9 +2412,9 @@ test_write_enum_property(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -2455,7 +2455,7 @@ static void
 test_read_flags_property(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingGsm *                s_gsm;
+    NMSettingGsm                 *s_gsm;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_Flags_Property");
 
@@ -2471,8 +2471,8 @@ test_write_flags_property(void)
 {
     NMTST_UUID_INIT(uuid);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSetting *                   s_gsm;
+    NMSettingConnection          *s_con;
+    NMSetting                    *s_gsm;
 
     connection = nm_simple_connection_new();
 
@@ -2513,10 +2513,10 @@ static void
 test_read_tc_config(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingTCConfig *           s_tc;
-    NMTCQdisc *                   qdisc1, *qdisc2;
-    NMTCAction *                  action1, *action2;
-    NMTCTfilter *                 tfilter1, *tfilter2;
+    NMSettingTCConfig            *s_tc;
+    NMTCQdisc                    *qdisc1, *qdisc2;
+    NMTCAction                   *action1, *action2;
+    NMTCTfilter                  *tfilter1, *tfilter2;
 
     connection = keyfile_read_connection_from_file(TEST_KEYFILES_DIR "/Test_TC_Config");
 
@@ -2567,11 +2567,11 @@ static void
 test_write_tc_config(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSetting *                   s_tc;
-    NMTCQdisc *                   qdisc1, *qdisc2;
-    NMTCTfilter *                 tfilter1, *tfilter2;
-    NMTCAction *                  action;
-    GError *                      error = NULL;
+    NMSetting                    *s_tc;
+    NMTCQdisc                    *qdisc1, *qdisc2;
+    NMTCTfilter                  *tfilter1, *tfilter2;
+    NMTCAction                   *action;
+    GError                       *error = NULL;
 
     connection =
         nmtst_create_minimal_connection("Test TC", NULL, NM_SETTING_WIRED_SETTING_NAME, NULL);
@@ -2751,12 +2751,12 @@ _assert_keyfile_nmmeta(const char *dirname,
 static void
 test_nmmeta(void)
 {
-    const char *  uuid          = "3c03fd17-ddc3-4100-a954-88b6fafff959";
+    const char   *uuid          = "3c03fd17-ddc3-4100-a954-88b6fafff959";
     gs_free char *filename      = g_strdup_printf("%s%s", uuid, NM_KEYFILE_PATH_SUFFIX_NMMETA);
     gs_free char *full_filename = g_strdup_printf("%s/%s", TEST_SCRATCH_DIR, filename);
-    const char *  loaded_path0  = NM_KEYFILE_PATH_NMMETA_SYMLINK_NULL;
-    const char *  loaded_path1  = "/some/where/but/not/scratch/dir";
-    const char *  filename2     = "foo1";
+    const char   *loaded_path0  = NM_KEYFILE_PATH_NMMETA_SYMLINK_NULL;
+    const char   *loaded_path1  = "/some/where/but/not/scratch/dir";
+    const char   *filename2     = "foo1";
     gs_free char *loaded_path2  = g_strdup_printf("%s/%s", TEST_SCRATCH_DIR, filename2);
 
     _assert_keyfile_nmmeta(TEST_SCRATCH_DIR, uuid, NULL, FALSE, full_filename, NULL, NULL, NULL);