summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFabio Estevam <festevam@denx.de>2024-01-15 22:16:49 -0300
committerKhem Raj <raj.khem@gmail.com>2024-01-19 09:51:05 -0800
commitca628a65c7fc4a85fae8336fc8fe082b850661ea (patch)
tree50a33dcb136fc75bdf25b8dcce18484eda76ec15
parent3e11b88fe22ea679c43fc16a5a3cbdd11b1640d8 (diff)
downloadmeta-openembedded-ca628a65c7fc4a85fae8336fc8fe082b850661ea.tar.gz
v4l-utils: Remove unneeded musl patch
The musl issue that 0004-Do-not-use-getsubopt.patch fixes has already been addressed by the following v4l-utils upstream commit: commit 6e19bb895a16a9a247524ac526ab47f576cc36f6 Author: Hans Verkuil <hverkuil-cisco@xs4all.nl> Date: Thu Nov 25 10:59:22 2021 +0100 v4l-utils: use v4l_getsubopt instead of getsubopt Android doesn't have getsubopt at all, and some libc implementations use a getsubopt variant that behaves slightly different. So add a new v4l-getsubopt.h header that either just uses the glibc function (if glibc is detected) or uses a static inline v4l_getsubopt that is copied from glibc. This should avoid such problems. Reported-by: Marian Buschsieweke <marian.buschsieweke@ovgu.de> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Remove the unneeded patch. Signed-off-by: Fabio Estevam <festevam@denx.de> Signed-off-by: Khem Raj <raj.khem@gmail.com>
-rw-r--r--meta-oe/recipes-multimedia/v4l2apps/v4l-utils/0004-Do-not-use-getsubopt.patch60
-rw-r--r--meta-oe/recipes-multimedia/v4l2apps/v4l-utils_1.26.1.bb1
2 files changed, 0 insertions, 61 deletions
diff --git a/meta-oe/recipes-multimedia/v4l2apps/v4l-utils/0004-Do-not-use-getsubopt.patch b/meta-oe/recipes-multimedia/v4l2apps/v4l-utils/0004-Do-not-use-getsubopt.patch
deleted file mode 100644
index fb844acbe4..0000000000
--- a/meta-oe/recipes-multimedia/v4l2apps/v4l-utils/0004-Do-not-use-getsubopt.patch
+++ /dev/null
@@ -1,60 +0,0 @@
1From 6e7e52de7afe29597016952a7317faf9c3ea3268 Mon Sep 17 00:00:00 2001
2From: Khem Raj <raj.khem@gmail.com>
3Date: Sat, 30 Nov 2019 18:50:34 -0800
4Subject: [PATCH] Do not use getsubopt
5
6POSIX says that behavior when subopts list is empty is undefined.
7musl libs will set value to NULL which leads to crash.
8
9Simply avoid getsubopt, since we cannot rely on it.
10
11Imported from Alpine Linux
12
13Upstream-Status: Pending
14
15Signed-off-by: Khem Raj <raj.khem@gmail.com>
16
17Adapt patch to 1.23.0.
18
19(v4l-utils rev fd544473800d02e90bc289434cc44e5aa8fadd0f).
20
21%% original patch: 0007-Do-not-use-getsubopt.patch
22
23Signed-off-by: Daniel Gomez <daniel@qtec.com>
24---
25 utils/v4l2-ctl/v4l2-ctl-common.cpp | 18 ++++++++++--------
26 1 file changed, 10 insertions(+), 8 deletions(-)
27
28diff --git a/utils/v4l2-ctl/v4l2-ctl-common.cpp b/utils/v4l2-ctl/v4l2-ctl-common.cpp
29index d77f7104..838c297d 100644
30--- a/utils/v4l2-ctl/v4l2-ctl-common.cpp
31+++ b/utils/v4l2-ctl/v4l2-ctl-common.cpp
32@@ -994,15 +994,17 @@ static bool parse_subset(char *optarg)
33
34 static bool parse_next_subopt(char **subs, char **value)
35 {
36- static char *const subopts[] = {
37- nullptr
38- };
39- int opt = v4l_getsubopt(subs, subopts, value);
40+ char *p = *subs;
41+ *value = *subs;
42
43- if (opt < 0 || *value)
44- return false;
45- fprintf(stderr, "Missing suboption value\n");
46- return true;
47+ while (*p && *p != ',')
48+ p++;
49+
50+ if (*p)
51+ *p++ = '\0';
52+
53+ *subs = p;
54+ return false;
55 }
56
57 void common_cmd(const std::string &media_bus_info, int ch, char *optarg)
58--
592.35.1
60
diff --git a/meta-oe/recipes-multimedia/v4l2apps/v4l-utils_1.26.1.bb b/meta-oe/recipes-multimedia/v4l2apps/v4l-utils_1.26.1.bb
index 4ae1a18c9e..594b51b439 100644
--- a/meta-oe/recipes-multimedia/v4l2apps/v4l-utils_1.26.1.bb
+++ b/meta-oe/recipes-multimedia/v4l2apps/v4l-utils_1.26.1.bb
@@ -28,7 +28,6 @@ PACKAGECONFIG[v4l2-tracer] = ",-Dv4l2-tracer=disabled,json-c"
28SRC_URI = "\ 28SRC_URI = "\
29 git://git.linuxtv.org/v4l-utils.git;protocol=https;branch=stable-1.26 \ 29 git://git.linuxtv.org/v4l-utils.git;protocol=https;branch=stable-1.26 \
30 file://0001-keytable-meson-Restrict-the-installation-of-50-rc_ke.patch \ 30 file://0001-keytable-meson-Restrict-the-installation-of-50-rc_ke.patch \
31 file://0004-Do-not-use-getsubopt.patch \
32" 31"
33 32
34SRCREV = "4aee01a027923cab1e40969f56f8ba58d3e6c0d1" 33SRCREV = "4aee01a027923cab1e40969f56f8ba58d3e6c0d1"