From 125e23e16fee4346729b104ed8f17dbabf27e3c1 Mon Sep 17 00:00:00 2001 From: Hans Verkuil Date: Sun, 10 Mar 2013 11:40:00 +0100 Subject: [PATCH] v4l2-compliance: missing check on the frameinterval A wrong discrete frameinterval as returned by ENUM_FRAMEINTERVALS wasn't detected. Signed-off-by: Hans Verkuil --- utils/v4l2-compliance/v4l2-test-formats.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/utils/v4l2-compliance/v4l2-test-formats.cpp b/utils/v4l2-compliance/v4l2-test-formats.cpp index a4f73c8..4e26020 100644 --- a/utils/v4l2-compliance/v4l2-test-formats.cpp +++ b/utils/v4l2-compliance/v4l2-test-formats.cpp @@ -82,6 +82,10 @@ static int testEnumFrameIntervals(struct node *node, __u32 pixfmt, __u32 w, __u3 switch (frmival.type) { case V4L2_FRMIVAL_TYPE_DISCRETE: ret = check_fract(&frmival.discrete); + if (ret) + return fail("invalid frameinterval %d (%d/%d)\n", f, + frmival.discrete.numerator, + frmival.discrete.denominator); if (found_stepwise) return fail("mixing discrete and stepwise is not allowed\n"); break; -- 2.7.4