|
From: | Eric Blake |
Subject: | Re: [Qemu-devel] [PATCH] fw_cfg_reboot: ensure reboot_time is nonegative |
Date: | Wed, 31 Oct 2018 10:54:56 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 |
On 10/24/18 6:35 AM, Laszlo Ersek wrote:
On 10/24/18 09:11, Li Qiang wrote:This can avoid setting a negative value to etc/boot-fail-wait. Signed-off-by: Li Qiang <address@hidden> --- hw/nvram/fw_cfg.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/hw/nvram/fw_cfg.c b/hw/nvram/fw_cfg.c index f4a52d8..276dcb1 100644 --- a/hw/nvram/fw_cfg.c +++ b/hw/nvram/fw_cfg.c @@ -199,12 +199,17 @@ static void fw_cfg_reboot(FWCfgState *s) reboot_timeout = strtol(p, &p, 10);
Looks like Markus handled the question about replacing strtol(), so I'll just point out one additional nit:
} } - /* validate the input */ - if (reboot_timeout > 0xffff) { - error_report("reboot timeout is larger than 65535, force it to 65535.");
Pre-existing, but now's as good a time as any to improve it:
- reboot_timeout = 0xffff; + + if (reboot_timeout >= 0) { + /* validate the input */ + if (reboot_timeout > 0xffff) { + error_report("reboot timeout is larger than 65535," + "force it to 65535.");
error_report() callers generally do not end with trailing '.' -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |