[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 14/14] load_multiboot(): get_image_size() returns in
From: |
Jes . Sorensen |
Subject: |
[Qemu-devel] [PATCH 14/14] load_multiboot(): get_image_size() returns int |
Date: |
Mon, 30 Aug 2010 17:35:47 +0200 |
From: Jes Sorensen <address@hidden>
Do not store return of get_image_size() in a uint32_t as it makes it
impossible to detect error returns from get_image_size.
Signed-off-by: Jes Sorensen <address@hidden>
---
hw/multiboot.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/multiboot.c b/hw/multiboot.c
index dc980e6..f9097a2 100644
--- a/hw/multiboot.c
+++ b/hw/multiboot.c
@@ -252,7 +252,7 @@ int load_multiboot(void *fw_cfg,
do {
char *next_space;
- uint32_t mb_mod_length;
+ int mb_mod_length;
uint32_t offs = mbs.mb_buf_size;
next_initrd = strchr(initrd_filename, ',');
--
1.7.2.2
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., (continued)
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Anthony Liguori, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Paolo Bonzini, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Anthony Liguori, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Paolo Bonzini, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Jes Sorensen, 2010/08/30
[Qemu-devel] [PATCH 11/14] Remove unused function arguments, Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 02/14] Respect return value from nbd_client(), Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 13/14] Change DPRINTF() to do{}while(0) to avoid compiler warning, Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 14/14] load_multiboot(): get_image_size() returns int,
Jes . Sorensen <=
[Qemu-devel] [PATCH 07/14] Remove unused argument for get_whole_cluster(), Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 03/14] Fix repeated typo: was "end if list" instead of "end of list", Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 04/14] Zero initialize timespec struct explicitly, Jes . Sorensen, 2010/08/30
- [Qemu-devel] Re: [PATCH 04/14] Zero initialize timespec struct explicitly, Anthony Liguori, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 04/14] Zero initialize timespec struct explicitly, malc, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 04/14] Zero initialize timespec struct explicitly, Jes Sorensen, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 04/14] Zero initialize timespec struct explicitly, Anthony Liguori, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 04/14] Zero initialize timespec struct explicitly, malc, 2010/08/30