summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEli Schwartz <eschwartz93@gmail.com>2024-04-28 01:42:46 -0400
committerSam James <sam@gentoo.org>2024-04-28 07:54:26 +0100
commit054d5d2c0cfd27716cf924e7f5b335a43dcf1217 (patch)
treee7cc4cd0bb177252190eccd6b360686ac2afde75 /x11-plugins
parentsys-fs/squashfs-tools-ng: run elibtoolize in non-live ebuild (diff)
downloadgentoo-054d5d2c0cfd27716cf924e7f5b335a43dcf1217.tar.gz
gentoo-054d5d2c0cfd27716cf924e7f5b335a43dcf1217.tar.bz2
gentoo-054d5d2c0cfd27716cf924e7f5b335a43dcf1217.zip
x11-plugins/wmcalendar: add patches to fix horrible missing function decls
We kill two birds with one stone. It fails the c99 porting tracker and will not compile with Modern C compilers such as gcc 14 or clang 16, and it ALSO fixes LTO errors since -Werror=lto-type-mismatch complained when the implicit function declaration's assumed type was incorrect and didn't match the type of the actual live instance of the function. While we are at it, fix some additional (memory access) errors discovered by LTO in the form of -Waggressive-loop-optimizations. Closes: https://bugs.gentoo.org/875530 Closes: https://bugs.gentoo.org/881461 Signed-off-by: Eli Schwartz <eschwartz93@gmail.com> Signed-off-by: Sam James <sam@gentoo.org>
Diffstat (limited to 'x11-plugins')
-rw-r--r--x11-plugins/wmcalendar/files/0001-fix-obvious-typo-causing-out-of-bounds-memory-access.patch27
-rw-r--r--x11-plugins/wmcalendar/files/wmcalendar-c99.patch63
-rw-r--r--x11-plugins/wmcalendar/wmcalendar-0.5.2-r3.ebuild39
3 files changed, 129 insertions, 0 deletions
diff --git a/x11-plugins/wmcalendar/files/0001-fix-obvious-typo-causing-out-of-bounds-memory-access.patch b/x11-plugins/wmcalendar/files/0001-fix-obvious-typo-causing-out-of-bounds-memory-access.patch
new file mode 100644
index 000000000000..8565fe879b29
--- /dev/null
+++ b/x11-plugins/wmcalendar/files/0001-fix-obvious-typo-causing-out-of-bounds-memory-access.patch
@@ -0,0 +1,27 @@
+From ce49fa7696aadb61bb770bc42181d258af4407de Mon Sep 17 00:00:00 2001
+From: Eli Schwartz <eschwartz93@gmail.com>
+Date: Sun, 28 Apr 2024 01:36:55 -0400
+Subject: [PATCH] fix obvious typo causing out of bounds memory access
+
+The size in the .h file doesn't match the loop access in the .c file.
+Detected via LTO and -Waggressive-loop-optimizations
+---
+ wmCalendar.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/wmCalendar.c b/wmCalendar.c
+index bf5640b..bfe68a1 100644
+--- a/wmCalendar.c
++++ b/wmCalendar.c
+@@ -71,7 +71,7 @@ void initValues(){
+ for(i = 0; i < 12; ++i)
+ for( j = 0; j < 6; ++j)
+ xdMonth[j][i] = xeMonth[j][i] - xsMonth[j][i] + 1;
+- for(i = 0; i < 12; ++i)
++ for(i = 0; i < 10; ++i)
+ for( j = 0; j < 2; ++j)
+ xdYear[j][i] = xeYear[j][i] - xsYear[j][i] + 1;
+ for(i = 0; i < MAXBUTTON; ++i)
+--
+2.43.2
+
diff --git a/x11-plugins/wmcalendar/files/wmcalendar-c99.patch b/x11-plugins/wmcalendar/files/wmcalendar-c99.patch
new file mode 100644
index 000000000000..fb37477108cd
--- /dev/null
+++ b/x11-plugins/wmcalendar/files/wmcalendar-c99.patch
@@ -0,0 +1,63 @@
+From eb19c3c29d65b4e0e4f30d49c568846296ea4287 Mon Sep 17 00:00:00 2001
+From: Eli Schwartz <eschwartz93@gmail.com>
+Date: Fri, 29 Mar 2024 10:21:50 -0400
+Subject: [PATCH 1/2] fix missing includes/prototypes that violate c99
+
+Fixes various -Werror=implicit-function-declaration issues.
+---
+ calendar.c | 2 ++
+ calendar.h | 1 +
+ 2 files changed, 3 insertions(+)
+
+diff --git a/calendar.c b/calendar.c
+index 4da3650..c847853 100644
+--- a/calendar.c
++++ b/calendar.c
+@@ -1,4 +1,6 @@
+ #include "calendar.h"
++#include "settings.h"
++#include "calendarfunc.h"
+
+ time_t modtime;
+ struct calobj* calRoot;
+diff --git a/calendar.h b/calendar.h
+index bc9e576..7803d0c 100644
+--- a/calendar.h
++++ b/calendar.h
+@@ -33,4 +33,5 @@ int getDayType(struct icaltimetype dt);
+ int calcDayType(struct icaltimetype dt);
+ char* read_stream(char *s, size_t size, void *d);
+ void checkicalversion();
++int eventOnDay(struct icaltimetype dt, struct calobj* it);
+
+--
+2.43.2
+
+
+From a6f7c549f79a8fa50456f06352a3fd1457937c62 Mon Sep 17 00:00:00 2001
+From: Eli Schwartz <eschwartz93@gmail.com>
+Date: Sun, 28 Apr 2024 01:34:58 -0400
+Subject: [PATCH 2/2] cast a struct pointing to itself to avoid
+ -Werror=incompatible-pointer-types
+
+This is already done the other way around at the beginning of the block.
+---
+ calendar.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/calendar.c b/calendar.c
+index c847853..f8f875b 100644
+--- a/calendar.c
++++ b/calendar.c
+@@ -326,7 +326,7 @@ void deleteCalObjs(){
+ free(calRoot->text);
+ free(calRoot->comp);
+ free(calRoot);
+- calRoot = help;
++ calRoot = (struct calobj*) help;
+ }
+ for(i = 0; i < 32; i++)
+ datetype[i][1] = 0;
+--
+2.43.2
+
diff --git a/x11-plugins/wmcalendar/wmcalendar-0.5.2-r3.ebuild b/x11-plugins/wmcalendar/wmcalendar-0.5.2-r3.ebuild
new file mode 100644
index 000000000000..ff36a3411c04
--- /dev/null
+++ b/x11-plugins/wmcalendar/wmcalendar-0.5.2-r3.ebuild
@@ -0,0 +1,39 @@
+# Copyright 1999-2024 Gentoo Authors
+# Distributed under the terms of the GNU General Public License v2
+
+EAPI=7
+inherit toolchain-funcs
+
+DESCRIPTION="a calendar dockapp"
+HOMEPAGE="https://wmcalendar.sourceforge.net"
+SRC_URI="mirror://sourceforge/${PN}/${P}.tar.gz"
+
+LICENSE="GPL-2"
+SLOT="0"
+KEYWORDS="~amd64 ~ppc ~sparc ~x86"
+
+RDEPEND="
+ dev-libs/libical
+ x11-libs/gtk+:2
+ x11-libs/libX11
+ x11-libs/libXext
+ x11-libs/libXpm"
+DEPEND="${RDEPEND}"
+BDEPEND="virtual/pkgconfig"
+
+DOCS=( ../{BUGS,CHANGES,HINTS,README,TODO} )
+
+S=${WORKDIR}/${P}/Src
+
+PATCHES=( "${FILESDIR}"/${P}-exit-sin-and-cos.patch
+ "${FILESDIR}"/${P}-rename_kill_func.patch
+ "${FILESDIR}"/${P}-ical.patch
+ "${FILESDIR}"/${P}-fno-common.patch
+ "${FILESDIR}"/wmcalendar-c99.patch
+ "${FILESDIR}"/0001-fix-obvious-typo-causing-out-of-bounds-memory-access.patch
+)
+
+src_compile() {
+ tc-export CC PKG_CONFIG
+ default
+}