summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorSebastian Mitterle <smitterl@redhat.com>2022-10-11 17:12:16 +0200
committerGerd Hoffmann <kraxel@redhat.com>2022-10-12 13:38:15 +0200
commit82a628f887409f8a863482627c6d6ea6f208d3b2 (patch)
tree7f331198e5889c48de8f7876de76ba58a0f7c3db
parent8af5f82b90cc860d79171ade228ccea59c7f525e (diff)
downloadfocaccia-qemu-82a628f887409f8a863482627c6d6ea6f208d3b2.tar.gz
focaccia-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>
-rw-r--r--qemu-edid.c4
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;