|
 |
016a62 |
From f11136998ed22e121b0a9df26f83e252bd5918fa Mon Sep 17 00:00:00 2001
|
|
|
016a62 |
From: Markus Armbruster <armbru@redhat.com>
|
|
|
016a62 |
Date: Mon, 7 Oct 2019 07:35:06 +0100
|
|
|
016a62 |
Subject: [PATCH 13/22] fw_cfg: Improve error message when can't load splash
|
|
|
016a62 |
file
|
|
|
016a62 |
MIME-Version: 1.0
|
|
|
016a62 |
Content-Type: text/plain; charset=UTF-8
|
|
|
016a62 |
Content-Transfer-Encoding: 8bit
|
|
|
016a62 |
|
|
|
016a62 |
RH-Author: Markus Armbruster <armbru@redhat.com>
|
|
|
016a62 |
Message-id: <20191007073509.5887-2-armbru@redhat.com>
|
|
|
016a62 |
Patchwork-id: 90978
|
|
|
016a62 |
O-Subject: [RHEL-8.2.0 qemu-kvm PATCH v2 1/4] fw_cfg: Improve error message when can't load splash file
|
|
|
016a62 |
Bugzilla: 1607367
|
|
|
016a62 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
016a62 |
RH-Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
016a62 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
016a62 |
|
|
|
016a62 |
From: Li Qiang <liq3ea@gmail.com>
|
|
|
016a62 |
|
|
|
016a62 |
read_splashfile() reports "failed to read splash file" without
|
|
|
016a62 |
further details. Get the details from g_file_get_contents(), and
|
|
|
016a62 |
include them in the error message. Also remove unnecessary 'res'
|
|
|
016a62 |
variable.
|
|
|
016a62 |
|
|
|
016a62 |
Signed-off-by: Li Qiang <liq3ea@gmail.com>
|
|
|
016a62 |
Reviewed-by: Markus Armbruster <armbru@redhat.com>
|
|
|
016a62 |
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
016a62 |
Message-Id: <1541052148-28752-1-git-send-email-liq3ea@gmail.com>
|
|
|
016a62 |
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
016a62 |
(cherry picked from commit bed66336771ecdcb788d394bdd081a78b843e509)
|
|
|
016a62 |
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
|
|
|
016a62 |
---
|
|
|
016a62 |
hw/nvram/fw_cfg.c | 7 +++----
|
|
|
016a62 |
1 file changed, 3 insertions(+), 4 deletions(-)
|
|
|
016a62 |
|
|
|
016a62 |
diff --git a/hw/nvram/fw_cfg.c b/hw/nvram/fw_cfg.c
|
|
|
016a62 |
index 2a0739d..d35ac7b 100644
|
|
|
016a62 |
--- a/hw/nvram/fw_cfg.c
|
|
|
016a62 |
+++ b/hw/nvram/fw_cfg.c
|
|
|
016a62 |
@@ -68,15 +68,14 @@ static char *read_splashfile(char *filename, gsize *file_sizep,
|
|
|
016a62 |
int *file_typep)
|
|
|
016a62 |
{
|
|
|
016a62 |
GError *err = NULL;
|
|
|
016a62 |
- gboolean res;
|
|
|
016a62 |
gchar *content;
|
|
|
016a62 |
int file_type;
|
|
|
016a62 |
unsigned int filehead;
|
|
|
016a62 |
int bmp_bpp;
|
|
|
016a62 |
|
|
|
016a62 |
- res = g_file_get_contents(filename, &content, file_sizep, &err;;
|
|
|
016a62 |
- if (res == FALSE) {
|
|
|
016a62 |
- error_report("failed to read splash file '%s'", filename);
|
|
|
016a62 |
+ if (!g_file_get_contents(filename, &content, file_sizep, &err)) {
|
|
|
016a62 |
+ error_report("failed to read splash file '%s': %s",
|
|
|
016a62 |
+ filename, err->message);
|
|
|
016a62 |
g_error_free(err);
|
|
|
016a62 |
return NULL;
|
|
|
016a62 |
}
|
|
|
016a62 |
--
|
|
|
016a62 |
1.8.3.1
|
|
|
016a62 |
|