spapr: Add missing checks for NULL pointers and report failures
diff mbox

Message ID 55074FB1.5070607@suse.de
State New
Headers show

Commit Message

Alexander Graf March 16, 2015, 9:48 p.m. UTC
On 14.03.15 11:29, Stefan Weil wrote:
> This fixes potential runtime crashes and two warnings from Coverity.
> 
> The new error message does not add  a prefix "qemu:" because that is
> already done in function hw_error. It also starts with an uppercase
> letter because that seems to be the mostly used form.
> 
> Signed-off-by: Stefan Weil <sw@weilnetz.de>

I've squashed the patch below into your commit as well.


Alex

Comments

Stefan Weil March 16, 2015, 10:13 p.m. UTC | #1
Am 16.03.2015 um 22:48 schrieb Alexander Graf:
>
> On 14.03.15 11:29, Stefan Weil wrote:
>> This fixes potential runtime crashes and two warnings from Coverity.
>>
>> The new error message does not add  a prefix "qemu:" because that is
>> already done in function hw_error. It also starts with an uppercase
>> letter because that seems to be the mostly used form.
>>
>> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> I've squashed the patch below into your commit as well.
>
>
> Alex
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 9c87a70..7febff7 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -1507,7 +1507,7 @@ static void ppc_spapr_init(MachineState *machine)
>
>       filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, "spapr-rtas.bin");
>       if (!filename) {
> -        hw_error("Could not find LPAR rtas '%s'\n", spapr-rtas.bin);
> +        hw_error("Could not find LPAR rtas '%s'\n", "spapr-rtas.bin");
>           exit(1);
>       }
>       spapr->rtas_size = get_image_size(filename);

I really should not make last minute changes after the build test.

Here I previously had included the filename in the format string,
but then I decided to make the format string identical to the
2nd one and moved the filename out of the string without adding "".

Sorry for the trouble and thank you for the fix.

Stefan

Patch
diff mbox

diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index 9c87a70..7febff7 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -1507,7 +1507,7 @@  static void ppc_spapr_init(MachineState *machine)

     filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, "spapr-rtas.bin");
     if (!filename) {
-        hw_error("Could not find LPAR rtas '%s'\n", spapr-rtas.bin);
+        hw_error("Could not find LPAR rtas '%s'\n", "spapr-rtas.bin");
         exit(1);
     }
     spapr->rtas_size = get_image_size(filename);