diff mbox series

qemu-edid: Restrict input parameter -d to avoid division by zero

Message ID 20221011151216.64897-1-smitterl@redhat.com
State New
Headers show
Series qemu-edid: Restrict input parameter -d to avoid division by zero | expand

Commit Message

Sebastian Mitterle Oct. 11, 2022, 3:12 p.m. UTC 
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>
---
 qemu-edid.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Gerd Hoffmann Oct. 12, 2022, 12:33 p.m. UTC | #1 
On Tue, Oct 11, 2022 at 05:12:16PM +0200, Sebastian Mitterle wrote:
> 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>

Added to queue.

thanks,
  Gerd
diff mbox series

Patch

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;