test: fix two coverity warnings
authorPeter Hutterer <peter.hutterer@who-t.net>
Sun, 7 Jul 2019 23:47:31 +0000 (09:47 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Sun, 7 Jul 2019 23:47:31 +0000 (09:47 +1000)
Alleged division by zero and use of an uninitialized variable. Both cannot
happen the way we call the tests, so let's just abort to make coverity happy.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
test/test-pointer.c
test/test-tablet.c

index 25cef8534164b0af69dccf52957dd66e86f53c87..6561489a502f7fc573018f3b0e6da4265d20d7e6 100644 (file)
@@ -68,6 +68,7 @@ test_relative_event(struct litest_device *dev, double dx, double dy)
        prop = udev_device_get_property_value(ud, "MOUSE_DPI");
        if (prop) {
                dpi = parse_mouse_dpi_property(prop);
+               ck_assert_int_ne(dpi, 0);
 
                dx *= 1000.0/dpi;
                dy *= 1000.0/dpi;
index 5f6526133ae039ea5b3dbefa8d5bb0df37d3a27b..793190393c2e6501d75e49fb317156707a1dad65 100644 (file)
@@ -111,6 +111,8 @@ START_TEST(button_seat_count)
        case BTN_0:
                dev2 = litest_add_device(li, LITEST_DELL_CANVAS_TOTEM);
                break;
+       default:
+               ck_abort();
        }
 
        litest_tablet_proximity_in(dev, 10, 10, axes);