diff options
| author | Sebastian Mitterle | 2022-10-11 17:12:16 +0200 |
|---|---|---|
| committer | Gerd Hoffmann | 2022-10-12 13:38:15 +0200 |
| commit | 82a628f887409f8a863482627c6d6ea6f208d3b2 (patch) | |
| tree | 7f331198e5889c48de8f7876de76ba58a0f7c3db /qemu-edid.c | |
| parent | ui/gtk: Fix the implicit mouse ungrabbing logic (diff) | |
| download | qemu-82a628f887409f8a863482627c6d6ea6f208d3b2.tar.gz qemu-82a628f887409f8a863482627c6d6ea6f208d3b2.tar.xz qemu-82a628f887409f8a863482627c6d6ea6f208d3b2.zip | |
qemu-edid: Restrict input parameter -d to avoid division by zero
A zero value for dpi will lead to a division by zero in qemu_edid_dpi_to_mm().
Tested by runnig qemu-edid -dX, X = 0, 100.
Resolves: qemu-project/qemu#1249
Suggested-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Sebastian Mitterle <smitterl@redhat.com>
Message-Id: <20221011151216.64897-1-smitterl@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'qemu-edid.c')
| -rw-r--r-- | qemu-edid.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/qemu-edid.c b/qemu-edid.c index 20c958d9c7..92e1a660a7 100644 --- a/qemu-edid.c +++ b/qemu-edid.c @@ -92,6 +92,10 @@ int main(int argc, char *argv[]) fprintf(stderr, "not a number: %s\n", optarg); exit(1); } + if (dpi == 0) { + fprintf(stderr, "cannot be zero: %s\n", optarg); + exit(1); + } break; case 'v': info.vendor = optarg; |