summaryrefslogtreecommitdiffstats
path: root/debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch
diff options
context:
space:
mode:
authorDaniel Baumann <daniel.baumann@progress-linux.org>2024-04-28 07:41:07 +0000
committerDaniel Baumann <daniel.baumann@progress-linux.org>2024-04-28 07:42:00 +0000
commit45b4f719eac15a8a7857c50a539ff21d809ec3bb (patch)
tree866e43ed8237b2bff203037b824fba7d5561525d /debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch
parentBuilding with gcc-10 instead of gcc-12. (diff)
downloadlinux-45b4f719eac15a8a7857c50a539ff21d809ec3bb.tar.xz
linux-45b4f719eac15a8a7857c50a539ff21d809ec3bb.zip
Adding patch to fix FTBFS in drivers/media/pci/cx18 with gcc-10 (Closes: #1027456).
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch')
-rw-r--r--debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch41
1 files changed, 41 insertions, 0 deletions
diff --git a/debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch b/debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch
new file mode 100644
index 000000000..7c7b20ea3
--- /dev/null
+++ b/debian/patches/bugfix/all/media-cx18-avoid-strlen-call-that-triggers-ice-in-gc.patch
@@ -0,0 +1,41 @@
+From: Ben Hutchings <benh@debian.org>
+Date: Sun, 5 Mar 2023 19:05:14 +0100
+Subject: media: cx18: Avoid strlen() call that triggers ICE in gcc 10
+Bug-Debian: https://bugs.debian.org/1027456
+
+Commit e9a40e1585d7 "fortify: Do not cast to "unsigned char"",
+included in 6.2 and backported into 6.1.2, triggers an ICE in
+bullseye's gcc 10 when compiling cx18-i2c.c. This change specifically
+affects use of the strlen() function.
+
+The compiler has been fixed in sid but not yet in bullseye. Until
+that happens, we need a workaround here.
+
+init_cx18_i2c() initialises i2c_adapter instances from a template and
+then appends to their names by calling sprintf(...name +
+strlen(...name), ...). Avoid using strlen() here by rewriting the
+entire name field instead of appending.
+
+Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
+---
+ drivers/media/pci/cx18/cx18-i2c.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/drivers/media/pci/cx18/cx18-i2c.c b/drivers/media/pci/cx18/cx18-i2c.c
+index a83435245251..a6ce1c204937 100644
+--- a/drivers/media/pci/cx18/cx18-i2c.c
++++ b/drivers/media/pci/cx18/cx18-i2c.c
+@@ -229,8 +229,8 @@ int init_cx18_i2c(struct cx18 *cx)
+ /* Setup adapter */
+ cx->i2c_adap[i] = cx18_i2c_adap_template;
+ cx->i2c_adap[i].algo_data = &cx->i2c_algo[i];
+- sprintf(cx->i2c_adap[i].name + strlen(cx->i2c_adap[i].name),
+- " #%d-%d", cx->instance, i);
++ sprintf(cx->i2c_adap[i].name, "%s #%d-%d",
++ cx18_i2c_adap_template.name, cx->instance, i);
+ i2c_set_adapdata(&cx->i2c_adap[i], &cx->v4l2_dev);
+ cx->i2c_adap[i].dev.parent = &cx->pci_dev->dev;
+ }
+--
+2.30.2
+