summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAriadne Conill <ariadne@dereferenced.org>2022-04-04 08:21:05 -0500
committerAriadne Conill <ariadne@dereferenced.org>2022-04-04 08:21:05 -0500
commit8eddaeefa1478c1bebb89f435626565429797b04 (patch)
tree68a05354ab1c3f86a911692a6beb2df0dbec08db
parent55e20d3e5b48eea29523b633492a167118dcc3bd (diff)
downloadapk-tools-fix/portability-broken-qsort-r.tar.gz
apk-tools-fix/portability-broken-qsort-r.tar.bz2
apk-tools-fix/portability-broken-qsort-r.tar.xz
apk-tools-fix/portability-broken-qsort-r.zip
portability: check for qsort_r which does not match POSIX-next definitionfix/portability-broken-qsort-r
FreeBSD (and also Darwin) introduced its own qsort_r, which has different semantics than the one slated for inclusion in POSIX. Add a portability thunk to deal with translating between implementations.
-rw-r--r--portability/meson.build14
-rw-r--r--portability/stdlib.h26
2 files changed, 40 insertions, 0 deletions
diff --git a/portability/meson.build b/portability/meson.build
index d077345..9c9e39c 100644
--- a/portability/meson.build
+++ b/portability/meson.build
@@ -21,6 +21,20 @@ foreach f : check_functions
endforeach
+# Check for wrong (non-POSIX) qsort_r prototype
+qsort_r_test = '''
+ #define _GNU_SOURCE
+ #include <stdlib.h>
+ _Static_assert(_Generic((qsort_r),
+ void (*)(void *, size_t, size_t, void *,
+ int (*)(void *, const void *, const void *)) : 1, default: 0),
+ "Bad prototype not matched");
+'''
+if cc.compiles(qsort_r_test, name: 'Test qsort_r non-POSIX prototype')
+ add_project_arguments('-DHAVE_BROKEN_QSORT_R', language: 'c')
+endif
+
+
if libportability_src.length() > 0
libportability = static_library(
'portability',
diff --git a/portability/stdlib.h b/portability/stdlib.h
index 6254c7c..d213ae7 100644
--- a/portability/stdlib.h
+++ b/portability/stdlib.h
@@ -1,3 +1,4 @@
+#pragma once
#include_next <stdlib.h>
#ifdef NEED_QSORT_R
@@ -5,3 +6,28 @@ void qsort_r(void *base, size_t nmemb, size_t size,
int (*compar)(const void *, const void *, void *),
void *arg);
#endif
+
+#ifdef HAVE_BROKEN_QSORT_R
+struct __apk_qsort_r_compat_arg {
+ int (*compar)(const void *, const void *, void *);
+ void *arg;
+};
+
+static inline int __apk_qsort_r_compar_compat(void *arg, const void *a, const void *b)
+{
+ struct __apk_qsort_r_compat_arg *compat_arg = arg;
+ return compat_arg->compar(a, b, compat_arg->arg);
+}
+
+static inline void __apk_qsort_r(void *base, size_t nmemb, size_t size,
+ int (*compar)(const void *, const void *, void *),
+ void *arg)
+{
+ struct __apk_qsort_r_compat_arg compat_arg = {
+ .compar = compar,
+ .arg = arg,
+ };
+ qsort_r(base, nmemb, size, &compat_arg, __apk_qsort_r_compar_compat);
+}
+#define qsort_r(...) __apk_qsort_r(__VA_ARGS__)
+#endif