Possible usage of unitialized variable

https://gitlab.com/Remmina/Remmina/-/blob/master/plugins/rdp/rdp_plugin.c#L1885-1890

					gchar **temp_items;
					gchar **rot_items;
					temp_items = g_strsplit(monitorids_string, ",", -1);
					for (i = 0; i < g_strv_length(temp_items); i++) {
						rdpMonitor *current = &base[atoi(rot_items[0])];
						rot_items = g_strsplit(temp_items[i], ":", -1);

Looks that in first iteration rot_items[0] is used before initialization Shouldn't be lines swapped like this?

					gchar **temp_items;
					gchar **rot_items;
					temp_items = g_strsplit(monitorids_string, ",", -1);
					for (i = 0; i < g_strv_length(temp_items); i++) {
						rot_items = g_strsplit(temp_items[i], ":", -1);
						rdpMonitor *current = &base[atoi(rot_items[0])];
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information