diff options
author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-11-09 13:03:22 +0100 |
---|---|---|
committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-11-09 13:03:22 +0100 |
commit | ce2d59cf21eed7012a848e72b1db18c6a5c11490 (patch) | |
tree | 8c61c19c058358fd71fafd0981f11b4ad46268f9 | |
parent | e14b30c51b2426ffcd933878963e39a02beab44f (diff) | |
download | v4l-utils-ce2d59cf21eed7012a848e72b1db18c6a5c11490.tar.gz |
v4l2-compliance: add missing return
fail() by itself does not return, the correct usage is 'return fail();'
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
-rw-r--r-- | utils/v4l2-compliance/v4l2-test-buffers.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/utils/v4l2-compliance/v4l2-test-buffers.cpp b/utils/v4l2-compliance/v4l2-test-buffers.cpp index c3929c9f..4a10a633 100644 --- a/utils/v4l2-compliance/v4l2-test-buffers.cpp +++ b/utils/v4l2-compliance/v4l2-test-buffers.cpp @@ -1821,8 +1821,8 @@ static int setupDmaBuf(struct node *expbuf_node, struct node *node, fail_on_test(exp_q.g_num_planes() < buf.g_num_planes()); for (unsigned p = 0; p < buf.g_num_planes(); p++) { if (exp_q.g_length(p) < buf.g_length(p)) - fail("exp_q.g_length(%u) < buf.g_length(%u): %u < %u\n", - p, p, exp_q.g_length(p), buf.g_length(p)); + return fail("exp_q.g_length(%u) < buf.g_length(%u): %u < %u\n", + p, p, exp_q.g_length(p), buf.g_length(p)); // This should not work! fail_on_test(node->mmap(buf.g_length(p), 0) != MAP_FAILED); q.s_fd(i, p, exp_q.g_fd(i, p)); |