The intent is that this library is going to be called every time
to check if we are not touching anything outside srcdir or
builddir.
Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
---
cfg.mk | 2 +-
tests/Makefile.am | 13 +++-
tests/testutils.c | 33 +++++++++
tests/testutils.h | 29 ++------
tests/vircgroupmock.c | 6 +-
tests/virpcimock.c | 6 +-
tests/virtestmock.c | 180 ++++++++++++++++++++++++++++++++++++++++++++++++++
7 files changed, 241 insertions(+), 28 deletions(-)
create mode 100644 tests/virtestmock.c
virCapsPtr virTestGenericCapsInit(void);
diff --git a/tests/vircgroupmock.c b/tests/vircgroupmock.c
index cfc51e8..395254b 100644
--- a/tests/vircgroupmock.c
+++ b/tests/vircgroupmock.c
@@ -416,8 +416,10 @@ static void init_syms(void)
LOAD_SYM(fopen);
LOAD_SYM(access);
- LOAD_SYM_ALT(lstat, __lxstat);
- LOAD_SYM_ALT(stat, __xstat);
+ LOAD_SYM(lstat);
+ LOAD_SYM(__lxstat);
+ LOAD_SYM(stat);
+ LOAD_SYM(__xstat);
LOAD_SYM(mkdir);
LOAD_SYM(open);
}
I wondered why this change ^^ was necessary, but then realized the tests
would crash for some reason that is unknown to me.
diff --git a/tests/virtestmock.c b/tests/virtestmock.c
new file mode 100644
index 0000000..f138e98
--- /dev/null
+++ b/tests/virtestmock.c
@@ -0,0 +1,180 @@
+/*
+ * Copyright (C) 2014 Red Hat, Inc.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library. If not, see
+ * <
http://www.gnu.org/licenses/>.
+ *
+ * Author: Michal Privoznik <mprivozn(a)redhat.com>
+ */
+
+#include <config.h>
+#include <unistd.h>
+#include <sys/types.h>
+#include <stdio.h>
+#include <dlfcn.h>
+#include <sys/stat.h>
+#include <fcntl.h>
+
+#include "internal.h"
+#include "configmake.h"
+
+static int (*realopen)(const char *path, int flags, ...);
+static FILE *(*realfopen)(const char *path, const char *mode);
+static int (*realaccess)(const char *path, int mode);
+static int (*realstat)(const char *path, struct stat *sb);
+static int (*real__xstat)(int ver, const char *path, struct stat *sb);
+static int (*reallstat)(const char *path, struct stat *sb);
+static int (*real__lxstat)(int ver, const char *path, struct stat *sb);
+
+static void init_syms(void)
+{
+ if (realopen)
+ return;
+
+#define LOAD_SYM(name) \
+ do { \
+ if (!(real ## name = dlsym(RTLD_NEXT, #name))) { \
+ fprintf(stderr, "Cannot find real '%s' symbol\n", #name);
\
+ abort(); \
+ } \
+ } while (0)
+
+#define LOAD_SYM_ALT(name1, name2) \
+ do { \
+ if (!(real ## name1 = dlsym(RTLD_NEXT, #name1)) && \
+ !(real ## name2 = dlsym(RTLD_NEXT, #name2))) { \
+ fprintf(stderr, "Cannot find real '%s' or '%s'
symbol\n", #name1, #name2); \
+ abort(); \
+ } \
+ } while (0)
+
+ LOAD_SYM(open);
+ LOAD_SYM(fopen);
+ LOAD_SYM(access);
+ LOAD_SYM_ALT(stat, __xstat);
+ LOAD_SYM_ALT(lstat, __lxstat);
+}
+
+static void
+checkPath(const char *path)
+{
+ if (!STRPREFIX(path, abs_topsrcdir) &&
+ !STRPREFIX(path, abs_topbuilddir)) {
+ /* Okay, this is a dummy check and spurious print.
+ * But this is going to be replaced soon. */
+ fprintf(stderr, "*** %s ***\n", path);
+ }
+}
+
+
+int open(const char *path, int flags, ...)
+{
+ int ret;
+
+ init_syms();
+
+ checkPath(path);
+
+ if (flags & O_CREAT) {
+ va_list ap;
+ mode_t mode;
+ va_start(ap, flags);
+ mode = va_arg(ap, mode_t);
+ va_end(ap);
+ ret = realopen(path, flags, mode);
+ } else {
+ ret = realopen(path, flags);
+ }
+ return ret;
+}
+
+FILE *fopen(const char *path, const char *mode)
+{
+ init_syms();
+
+ checkPath(path);
+
+ return realfopen(path, mode);
+}
+
+
+int access(const char *path, int mode)
+{
+ init_syms();
+
+ checkPath(path);
+
+ return realaccess(path, mode);
+}
+
+int stat(const char *path, struct stat *sb)
+{
+ init_syms();
+
+ checkPath(path);
+
+ if (!realstat)
+ return real__xstat(1, path, sb);
+
+ return realstat(path, sb);
+}
+
+int
+__xstat(int ver, const char *path, struct stat *sb)
+{
+ init_syms();
+
+ checkPath(path);
+ if (!real__xstat) {
+ if (ver == 1) {
+ return realstat(path, sb);
+ } else {
+ fprintf(stderr, "Not handled __xstat(ver=%d)", ver);
+ abort();
+ }
+ }
+
+ return real__xstat(ver, path, sb);
+}
+
+int
+lstat(const char *path, struct stat *sb)
+{
+ init_syms();
+
+ checkPath(path);
+
+ if (!reallstat)
+ return real__lxstat(1, path, sb);
+
+ return reallstat(path, sb);
+}
+
+int
+__lxstat(int ver, const char *path, struct stat *sb)
+{
+ init_syms();
+
+ checkPath(path);
+ if (!real__lxstat) {
+ if (ver == 1) {
+ return reallstat(path, sb);
+ } else {
+ fprintf(stderr, "Not handled __lxstat(ver=%d)", ver);
+ abort();
+ }
+ }
+
+ return real__lxstat(ver, path, sb);
+}
The patch works, it also looks reasonable to me, but I'm far from being
a master in this matter, so let's wait if someone would like to chime in
and express their opinion. If not, I say ACK.
Erik