diff options
-rw-r--r-- | src/cli/cli.c | 3 | ||||
-rw-r--r-- | src/daemon/abrt-action-save-package-data.c | 10 | ||||
-rw-r--r-- | src/gui-wizard-gtk/wizard.c | 15 | ||||
-rw-r--r-- | src/lib/abrt_dbus.c | 3 | ||||
-rw-r--r-- | src/lib/event_config.c | 10 | ||||
-rw-r--r-- | src/lib/event_xml_parser.c | 3 | ||||
-rw-r--r-- | src/lib/glib_support.c | 3 | ||||
-rw-r--r-- | src/plugins/abrt-retrace-client.c | 9 |
8 files changed, 38 insertions, 18 deletions
diff --git a/src/cli/cli.c b/src/cli/cli.c index 0c895a76..0af9ef2b 100644 --- a/src/cli/cli.c +++ b/src/cli/cli.c @@ -116,7 +116,8 @@ static void print_crash(crash_data_t *crash_data) */ static void print_crash_list(vector_of_crash_data_t *crash_list, bool include_reported) { - for (unsigned i = 0; i < crash_list->len; ++i) + unsigned i; + for (i = 0; i < crash_list->len; ++i) { crash_data_t *crash = get_crash_data(crash_list, i); if (!include_reported) diff --git a/src/daemon/abrt-action-save-package-data.c b/src/daemon/abrt-action-save-package-data.c index 34905b60..e58b4254 100644 --- a/src/daemon/abrt-action-save-package-data.c +++ b/src/daemon/abrt-action-save-package-data.c @@ -64,7 +64,8 @@ static char *get_argv1_if_full_path(const char* cmdline) static bool is_path_blacklisted(const char *path) { - for (GList *li = g_settings_setBlackListedPaths; li != NULL; li = g_list_next(li)) + GList *li; + for (li = g_settings_setBlackListedPaths; li != NULL; li = g_list_next(li)) { if (fnmatch((char*)li->data, path, /*flags:*/ 0) == 0) { @@ -184,7 +185,9 @@ static int SavePackageDescriptionToDebugDump(const char *dump_dir_name) package_short_name = get_package_name_from_NVR_or_NULL(package_full_name); VERB2 log("Package:'%s' short:'%s'", package_full_name, package_short_name); - for (GList *li = g_settings_setBlackListedPkgs; li != NULL; li = g_list_next(li)) + GList *li; + + for (li = g_settings_setBlackListedPkgs; li != NULL; li = g_list_next(li)) { if (strcmp((char*)li->data, package_short_name) == 0) { @@ -278,7 +281,8 @@ int main(int argc, char **argv) VERB1 log("Initializing rpm library"); rpm_init(); - for (GList *li = g_settings_setOpenGPGPublicKeys; li != NULL; li = g_list_next(li)) + GList *li; + for (li = g_settings_setOpenGPGPublicKeys; li != NULL; li = g_list_next(li)) { VERB1 log("Loading GPG key '%s'", (char*)li->data); rpm_load_gpgkey((char*)li->data); diff --git a/src/gui-wizard-gtk/wizard.c b/src/gui-wizard-gtk/wizard.c index 33a69110..f23be58f 100644 --- a/src/gui-wizard-gtk/wizard.c +++ b/src/gui-wizard-gtk/wizard.c @@ -551,7 +551,8 @@ void update_gui_state_from_crash_data(void) /* Update reporter checkboxes */ /* Remember names of selected reporters */ GList *old_reporters = gtk_container_get_children(GTK_CONTAINER(g_box_reporters)); - for (GList *li = old_reporters; li; li = li->next) + GList *li; + for (li = old_reporters; li; li = li->next) { if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(li->data)) == TRUE) li->data = xstrdup(gtk_button_get_label(GTK_BUTTON(li->data))); @@ -563,11 +564,13 @@ void update_gui_state_from_crash_data(void) add_event_buttons(g_box_reporters, g_report_events, /*callback:*/ G_CALLBACK(report_tb_was_toggled), /*radio:*/ false, /*prev:*/ NULL); /* Re-select new reporters which were selected before we deleted them */ GList *new_reporters = gtk_container_get_children(GTK_CONTAINER(g_box_reporters)); - for (GList *li_new = new_reporters; li_new; li_new = li_new->next) + GList *li_new; + for (li_new = new_reporters; li_new; li_new = li_new->next) { const char *new_name = gtk_button_get_label(GTK_BUTTON(li_new->data)); + GList *li_old; - for (GList *li_old = old_reporters; li_old; li_old = li_old->next) + for (li_old = old_reporters; li_old; li_old = li_old->next) { if (strcmp(new_name, li_old->data) == 0) { @@ -1022,7 +1025,8 @@ static void next_page(GtkAssistant *assistant, gpointer user_data) GList *reporters = gtk_container_get_children(GTK_CONTAINER(g_box_reporters)); if (reporters) { - for (GList *li = reporters; li; li = li->next) + GList *li; + for (li = reporters; li; li = li->next) { if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(li->data)) == TRUE) /* Button's tooltip contains event_name */ @@ -1210,7 +1214,8 @@ static void add_pages(void) error_msg_and_die("Can't load %s: %s", g_glade_file, error->message); } - for (int i = 0; page_names[i] != NULL; i++) + int i; + for (i = 0; page_names[i] != NULL; i++) { GtkWidget *page = GTK_WIDGET(gtk_builder_get_object(builder, page_names[i])); if (page == NULL) diff --git a/src/lib/abrt_dbus.c b/src/lib/abrt_dbus.c index a4bab15f..51962243 100644 --- a/src/lib/abrt_dbus.c +++ b/src/lib/abrt_dbus.c @@ -232,12 +232,13 @@ void store_crash_data(DBusMessageIter* dbus_iter, crash_data_t *val) void store_vector_of_crash_data(DBusMessageIter* iter, vector_of_crash_data_t *val) { DBusMessageIter sub_iter; + unsigned i; /* "array of maps". map in dbus is an array ("a") of two element structs "({...})": * "s" (string) for key and "as" for value (in this case, array of strings) */ if (!dbus_message_iter_open_container(iter, DBUS_TYPE_ARRAY, "a{sas}", &sub_iter)) die_out_of_memory(); - for (unsigned i = 0; i < val->len; i++) + for (i = 0; i < val->len; i++) { crash_data_t *crash_data = get_crash_data(val, i); store_crash_data(&sub_iter, crash_data); diff --git a/src/lib/event_config.c b/src/lib/event_config.c index 4e14c79a..a84aae0c 100644 --- a/src/lib/event_config.c +++ b/src/lib/event_config.c @@ -45,13 +45,15 @@ void free_event_option(event_option_t *p) void free_event_config(event_config_t *p) { + GList *opt; + if (!p) return; free(p->screen_name); //free(p->title); //free(p->action); free(p->description); - for (GList *opt = p->options; opt; opt = opt->next) + for (opt = p->options; opt; opt = opt->next) free_event_option(opt->data); g_list_free(p->options); free(p); @@ -254,7 +256,8 @@ GList *export_event_config(const char *event_name) event_config_t *config = get_event_config(event_name); if (config) { - for (GList *lopt = config->options; lopt; lopt = lopt->next) + GList *lopt; + for (lopt = config->options; lopt; lopt = lopt->next) { event_option_t *opt = lopt->data; if (!opt->value) @@ -289,8 +292,9 @@ GHashTable *validate_event(const char *event_name) GHashTable *errors = g_hash_table_new_full(g_str_hash, g_str_equal, free, free); + GList *li; - for (GList *li = config->options; li; li = li->next) + for (li = config->options; li; li = li->next) { event_option_t *opt = (event_option_t *)li->data; char *err = validate_event_option(opt); diff --git a/src/lib/event_xml_parser.c b/src/lib/event_xml_parser.c index 74329134..a6aa9a1f 100644 --- a/src/lib/event_xml_parser.c +++ b/src/lib/event_xml_parser.c @@ -111,8 +111,9 @@ static void start_element(GMarkupParseContext *context, } event_option_t *opt = parse_data->cur_option = new_event_option(); + int i; - for (int i = 0; attribute_names[i] != NULL; ++i) + for (i = 0; attribute_names[i] != NULL; ++i) { VERB2 log("attr: %s:%s", attribute_names[i], attribute_values[i]); if (strcmp(attribute_names[i], "name") == 0) diff --git a/src/lib/glib_support.c b/src/lib/glib_support.c index feb4c18b..fc4d364e 100644 --- a/src/lib/glib_support.c +++ b/src/lib/glib_support.c @@ -20,7 +20,8 @@ void list_free_with_free(GList *list) { - for (GList *li = list; li; li = g_list_next(li)) + GList *li; + for (li = list; li; li = g_list_next(li)) free(li->data); g_list_free(list); } diff --git a/src/plugins/abrt-retrace-client.c b/src/plugins/abrt-retrace-client.c index da11a161..005a03f0 100644 --- a/src/plugins/abrt-retrace-client.c +++ b/src/plugins/abrt-retrace-client.c @@ -362,9 +362,10 @@ static int http_get_response_code(const char *message) static void http_print_headers(FILE *file, const char *message) { const char *headers_end = strstr(message, "\r\n\r\n"); + const char *c; if (!headers_end) headers_end = message + strlen(message); - for (const char *c = message; c != headers_end + 2; ++c) + for (c = message; c != headers_end + 2; ++c) { if (*c == '\r') continue; @@ -575,7 +576,8 @@ static void backtrace(const char *task_id, const char *task_password, * '\r' characters are not copied to the backtrace. */ *backtrace = xmalloc(length); char *b = *backtrace; - for (char *c = headers_end + strlen("\r\n\r\n"); *c; ++c) + char *c; + for (c = headers_end + strlen("\r\n\r\n"); *c; ++c) { if (*c == '\r') continue; @@ -625,9 +627,10 @@ static void run_log(const char *task_id, const char *task_password) response_code, http_response); } char *headers_end = strstr(http_response, "\r\n\r\n"); + char *c; if (!headers_end) error_msg_and_die("Invalid response from server: missing HTTP message body."); - for (char *c = headers_end + 4; *c; ++c) + for (c = headers_end + 4; *c; ++c) { if (*c == '\r') continue; |