Fixes when HAVE_LIBC_VERSIONED_SYMBOLS is not defined

fuse_loop_mt and fuse_new had not been defined when
HAVE_LIBC_VERSIONED_SYMBOLS had not been set and additionally,
fuse_new_31 was missing in the version script and was therefore
an unusable symbol.

This also adds a test for unset HAVE_LIBC_VERSIONED_SYMBOLS.

diff --git a/include/fuse.h b/include/fuse.h
index 9897c85b4d04f57d3a6ca40f9372b92622c59dd1..2888d2b497c11f3f24f9fc02e91509b6c1b7ca5e 100644 (file)
--- a/include/fuse.h
+++ b/include/fuse.h
@@ -948,8 +948,15 @@ struct fuse *fuse_new_30(struct fuse_args *args, const struct fuse_operations *o
                         size_t op_size, void *private_data);
 #define fuse_new(args, op, size, data) fuse_new_30(args, op, size, data)
 #else
+#if (defined(HAVE_LIBC_VERSIONED_SYMBOLS))
 struct fuse *fuse_new(struct fuse_args *args, const struct fuse_operations *op,
                      size_t op_size, void *private_data);
+#else /* HAVE_LIBC_VERSIONED_SYMBOLS */
+struct fuse *fuse_new_31(struct fuse_args *args,
+                     const struct fuse_operations *op,
+                     size_t op_size, void *user_data);
+#define fuse_new(args, op, size, data) fuse_new_31(args, op, size, data)
+#endif /* HAVE_LIBC_VERSIONED_SYMBOLS */
 #endif
 
 /**
@@ -1046,9 +1053,14 @@ int fuse_loop_mt_32(struct fuse *f, struct fuse_loop_config *config);
  *
  * See also: fuse_loop()
  */
+#if (defined(HAVE_LIBC_VERSIONED_SYMBOLS))
 int fuse_loop_mt(struct fuse *f, struct fuse_loop_config *config);
+#else
+#define fuse_loop_mt(f, config) fuse_loop_mt_312(f, config)
+#endif /* HAVE_LIBC_VERSIONED_SYMBOLS */
 #endif
 
+
 /**
  * Get the current context
  *

diff --git a/lib/fuse_versionscript b/lib/fuse_versionscript
index 7e50e75078017072a8fe18708aa861b1cfbe0730..a1e9ed8df4737339ed63be63ad61a562f4d5ae96 100644 (file)
--- a/lib/fuse_versionscript
+++ b/lib/fuse_versionscript
@@ -139,6 +139,7 @@ FUSE_3.1 {
                fuse_lib_help;
                fuse_invalidate_path;
                fuse_new_30;
+               fuse_new_31;
                fuse_new;
 } FUSE_3.0;
 

diff --git a/test/travis-build.sh b/test/travis-build.sh
index 267b6e7c1b89e8afec9e5e58bbd2c015a2966349..22b3e6adb9517ccd32828ed7e0f0bfa9ea622dad 100755 (executable)
--- a/test/travis-build.sh
+++ b/test/travis-build.sh
@@ -50,15 +50,20 @@ for CC in gcc gcc-7 gcc-10 clang; do
 done
 (cd build-$CC; sudo ninja install)
 
-# Sanitized build
-CC=clang
-CXX=clang++
-for san in undefined address; do
+sanitized_build()
+{
+    san=$1
+    additonal_option=$2
+
     echo "=== Building with clang and ${san} sanitizer ==="
-    mkdir build-${san}; cd build-${san}
+    [ -n ${additonal_option} ] || echo "Additional option: ${additonal_option}"
+
+    mkdir build-${san}; pushd build-${san}
+
     # b_lundef=false is required to work around clang
     # bug, cf. https://groups.google.com/forum/#!topic/mesonbuild/tgEdAXIIdC4
-    meson -D b_sanitize=${san} -D b_lundef=false -D werror=true "${SOURCE_DIR}" \
+    meson -D b_sanitize=${san} -D b_lundef=false -D werror=true\
+           ${additonal_option} "${SOURCE_DIR}" \
            || (cat meson-logs/meson-log.txt; false)
     ninja
 
@@ -66,11 +71,27 @@ for san in undefined address; do
     sudo ${TEST_CMD}
     sudo chown root:root util/fusermount3
     sudo chmod 4755 util/fusermount3
-    # Cleanup temporary files (since they're now owned by root)
+    # Cleanup temporary files (since they are now owned by root)
     sudo rm -rf test/.pytest_cache/ test/__pycache__
 
     ${TEST_CMD}
-    cd ..
+    
+    popd
+    rm -fr build-${san}
+}
+
+# Sanitized build
+CC=clang
+CXX=clang++
+for san in undefined address; do
+    sanitized_build ${san}
+done
+
+# Sanitized build without libc versioned symbols
+CC=clang
+CXX=clang++
+for san in undefined address; do
+    sanitized_build ${san} "-Ddisable-libc-symbol-version=true"
 done
 
 # Documentation.