Hi Luiz, On Mon, Apr 11, 2016, Luiz Augusto von Dentz wrote: > +static void convert_info(struct btd_device *device, GKeyFile *key_file) > +{ > + char **uuids; > + > + /* Load device profile list from legacy property */ > + uuids = g_key_file_get_string_list(key_file, "General", "SDPServices", > + NULL, NULL); > + if (uuids) { > + char **uuid; > + > + for (uuid = uuids; *uuid; uuid++) > + device_add_uuid(device, *uuid); > + > + /* Remove SDPServices so it is not loaded again */ > + g_key_file_remove_key(key_file, "General", "SDPServices", NULL); Note that this doesn't modify the file but just the GKeyFile object. For places where we need to write back out the contents to the file we usually use g_key_file_to_data + g_file_set_contents, but I didn't see any such calls in this call path. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html