On 01/15/2018 07:26 AM, Daniel P. Berrange wrote:
> Signed-off-by: Daniel P. Berrange <berra...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> scripts/signrom.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/signrom.py b/scripts/signrom.py
> index d1dabe0240..0497a1c32e 100644
> --- a/scripts/signrom.py
> +++ b/scripts/signrom.py
> @@ -18,7 +18,7 @@ fin = open(sys.argv[1], 'rb')
> fout = open(sys.argv[2], 'wb')
>
> magic = fin.read(2)
> -if magic != '\x55\xaa':
> +if magic != b'\x55\xaa':
> sys.exit("%s: option ROM does not begin with magic 55 aa" % sys.argv[1])
>
> size_byte = ord(fin.read(1))
> @@ -33,7 +33,7 @@ elif len(data) < size:
> # Add padding if necessary, rounding the whole input to a multiple of
> # 512 bytes according to the third byte of the input.
> # size-1 because a final byte is added below to store the checksum.
> - data = data.ljust(size-1, '\0')
> + data = data.ljust(size-1, b'\0')
> else:
> if ord(data[-1:]) != 0:
> sys.stderr.write('WARNING: ROM includes nonzero checksum\n')
>
