From: Julien ROPE <jrope@xxxxxxxxxx> When doing a screenshot, if the user provides a filename without a file extension, an error occurs because the image format could not be determined. This patch adds a .png extension to such filenames, so that there is a default file format for screenshots. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1752514 Signed-off-by: Julien Ropé <jrope@xxxxxxxxxx> --- src/virt-viewer-window.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c index 4c08423..f58ebad 100644 --- a/src/virt-viewer-window.c +++ b/src/virt-viewer-window.c @@ -1069,6 +1069,14 @@ virt_viewer_window_menu_file_screenshot(GtkWidget *menu G_GNUC_UNUSED, GError *error = NULL; filename = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER (dialog)); + if (g_strrstr(filename, ".") == NULL) { + // no extension provided: add the .png default + char *tmp_filename ; + tmp_filename = g_strdup_printf("%s.png", filename) ; + g_free(filename) ; + filename = tmp_filename ; + } + if (!virt_viewer_window_save_screenshot(self, filename, &error)) { virt_viewer_app_simple_message_dialog(self->priv->app, error->message); -- 2.21.0 _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
