Commit 2d9178d9 authored by Laurent Vivier's avatar Laurent Vivier Committed by Eric Blake
Browse files

error: Strip trailing '\n' from error string arguments (again again) 



Re-run Coccinelle script scripts/coccinelle/err-bad-newline.cocci,
and found new error_report() occurrences with '\n'.

Signed-off-by: default avatarLaurent Vivier <lvivier@redhat.com>
Message-Id: <20180323143202.28879-3-lvivier@redhat.com>
Reviewed-by: default avatarEric Blake <eblake@redhat.com>
Signed-off-by: default avatarEric Blake <eblake@redhat.com>
parent fdf235ba

target/i386/hvf/hvf.c

+12 −12
Original line number Diff line number Diff line
@@ -86,25 +86,25 @@ static void assert_hvf_ok(hv_return_t ret) 


    switch (ret) {

    case HV_ERROR:

        error_report("Error: HV_ERROR\n");

        error_report("Error: HV_ERROR");

        break;

    case HV_BUSY:

        error_report("Error: HV_BUSY\n");

        error_report("Error: HV_BUSY");

        break;

    case HV_BAD_ARGUMENT:

        error_report("Error: HV_BAD_ARGUMENT\n");

        error_report("Error: HV_BAD_ARGUMENT");

        break;

    case HV_NO_RESOURCES:

        error_report("Error: HV_NO_RESOURCES\n");

        error_report("Error: HV_NO_RESOURCES");

        break;

    case HV_NO_DEVICE:

        error_report("Error: HV_NO_DEVICE\n");

        error_report("Error: HV_NO_DEVICE");

        break;

    case HV_UNSUPPORTED:

        error_report("Error: HV_UNSUPPORTED\n");

        error_report("Error: HV_UNSUPPORTED");

        break;

    default:

        error_report("Unknown Error\n");

        error_report("Unknown Error");

    }



    abort();
@@ -191,7 +191,7 @@ void hvf_set_phys_mem(MemoryRegionSection *section, bool add) 
    if (mem) {

        mem->size = 0;

        if (do_hvf_set_memory(mem)) {

            error_report("Failed to reset overlapping slot\n");

            error_report("Failed to reset overlapping slot");

            abort();

        }

    }
@@ -211,7 +211,7 @@ void hvf_set_phys_mem(MemoryRegionSection *section, bool add) 
    }



    if (x == hvf_state->num_slots) {

        error_report("No free slots\n");

        error_report("No free slots");

        abort();

    }
@@ -221,7 +221,7 @@ void hvf_set_phys_mem(MemoryRegionSection *section, bool add) 
    mem->region = area;



    if (do_hvf_set_memory(mem)) {

        error_report("Error registering new memory slot\n");

        error_report("Error registering new memory slot");

        abort();

    }

}
@@ -884,7 +884,7 @@ int hvf_vcpu_exec(CPUState *cpu) 
                break;

            }

            default:

                error_report("Unrecognized CR %d\n", cr);

                error_report("Unrecognized CR %d", cr);

                abort();

            }

            RIP(env) += ins_len;
@@ -930,7 +930,7 @@ int hvf_vcpu_exec(CPUState *cpu) 
            env->error_code = 0;

            break;

        default:

            error_report("%llx: unhandled exit %llx\n", rip, exit_reason);

            error_report("%llx: unhandled exit %llx", rip, exit_reason);

        }

    } while (ret == 0);