Commit a1f91087 authored by Peter Maydell's avatar Peter Maydell Committed by Michael Tokarev
Browse files

hw/i386/kvmvapic: Remove dead code in patch_hypercalls() 



The patch_hypercalls() function sets up a 'patches'
variable and checks it at the end of the function, but
never modifies it in the middle. Remove this dead code,
which seems to have been present since the function was
added in commit e5ad936b in 2012.

(Spotted by Coverity: CID 1005581.)

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
Acked-by: default avatarJan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: default avatarMichael Tokarev <mjt@tls.msk.ru>
parent 9d7b2670

hw/i386/kvmvapic.c

+0 −6
Original line number Diff line number Diff line
@@ -535,7 +535,6 @@ static int patch_hypercalls(VAPICROMState *s) 
    uint8_t alternates[2];

    const uint8_t *pattern;

    const uint8_t *patch;

    int patches = 0;

    off_t pos;

    uint8_t *rom;
@@ -566,11 +565,6 @@ static int patch_hypercalls(VAPICROMState *s) 
    }



    g_free(rom);



    if (patches != 0 && patches != 2) {

        return -1;

    }



    return 0;

}