2016-05-13 12:08:29 +00:00
|
|
|
/*
|
|
|
|
* Copyright (C) 2016 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/>.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include <config.h>
|
|
|
|
|
|
|
|
#include "virmock.h"
|
|
|
|
#include <unistd.h>
|
|
|
|
#include <fcntl.h>
|
2016-04-18 12:10:33 +00:00
|
|
|
#include <sys/file.h>
|
2016-05-16 15:15:46 +00:00
|
|
|
#include <sys/stat.h>
|
2016-05-13 12:08:29 +00:00
|
|
|
|
2020-01-14 17:38:59 +00:00
|
|
|
#include "virsocket.h"
|
2016-05-13 12:08:29 +00:00
|
|
|
#include "configmake.h"
|
2016-04-18 12:10:33 +00:00
|
|
|
#include "virstring.h"
|
|
|
|
#include "viralloc.h"
|
|
|
|
#include "virfile.h"
|
2016-05-13 12:08:29 +00:00
|
|
|
|
|
|
|
static int (*real_open)(const char *path, int flags, ...);
|
|
|
|
static FILE *(*real_fopen)(const char *path, const char *mode);
|
|
|
|
static int (*real_access)(const char *path, int mode);
|
2018-07-08 07:24:34 +00:00
|
|
|
static int (*real_connect)(int fd, const struct sockaddr *addr, socklen_t addrlen);
|
2016-05-13 12:08:29 +00:00
|
|
|
|
2016-04-18 12:10:33 +00:00
|
|
|
static const char *progname;
|
|
|
|
const char *output;
|
|
|
|
|
|
|
|
#define VIR_FILE_ACCESS_DEFAULT abs_builddir "/test_file_access.txt"
|
|
|
|
|
2016-05-13 12:08:29 +00:00
|
|
|
static void init_syms(void)
|
|
|
|
{
|
|
|
|
if (real_open)
|
|
|
|
return;
|
|
|
|
|
|
|
|
VIR_MOCK_REAL_INIT(open);
|
|
|
|
VIR_MOCK_REAL_INIT(fopen);
|
|
|
|
VIR_MOCK_REAL_INIT(access);
|
2018-07-08 07:24:34 +00:00
|
|
|
VIR_MOCK_REAL_INIT(connect);
|
2016-05-13 12:08:29 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
static void
|
2018-07-08 07:32:03 +00:00
|
|
|
printFile(const char *file,
|
|
|
|
const char *func)
|
2016-05-13 12:08:29 +00:00
|
|
|
{
|
2016-04-18 12:10:33 +00:00
|
|
|
FILE *fp;
|
|
|
|
const char *testname = getenv("VIR_TEST_MOCK_TESTNAME");
|
|
|
|
|
|
|
|
if (!progname) {
|
|
|
|
progname = getenv("VIR_TEST_MOCK_PROGNAME");
|
|
|
|
|
|
|
|
if (!progname)
|
|
|
|
return;
|
|
|
|
|
|
|
|
output = getenv("VIR_TEST_FILE_ACCESS_OUTPUT");
|
|
|
|
if (!output)
|
|
|
|
output = VIR_FILE_ACCESS_DEFAULT;
|
|
|
|
}
|
|
|
|
|
2020-07-28 12:29:32 +00:00
|
|
|
if (!(fp = real_fopen(output, "w"))) {
|
2019-10-02 15:30:36 +00:00
|
|
|
fprintf(stderr, "Unable to open %s: %s\n", output, g_strerror(errno));
|
2016-04-18 12:10:33 +00:00
|
|
|
abort();
|
|
|
|
}
|
|
|
|
|
|
|
|
if (flock(fileno(fp), LOCK_EX) < 0) {
|
2019-10-02 15:30:36 +00:00
|
|
|
fprintf(stderr, "Unable to lock %s: %s\n", output, g_strerror(errno));
|
2016-04-18 12:10:33 +00:00
|
|
|
fclose(fp);
|
|
|
|
abort();
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Now append the following line into the output file:
|
2018-07-08 07:32:03 +00:00
|
|
|
* $file: $progname: $func: $testname */
|
2016-04-18 12:10:33 +00:00
|
|
|
|
2018-07-08 07:32:03 +00:00
|
|
|
fprintf(fp, "%s: %s: %s", file, func, progname);
|
2016-04-18 12:10:33 +00:00
|
|
|
if (testname)
|
|
|
|
fprintf(fp, ": %s", testname);
|
|
|
|
|
|
|
|
fputc('\n', fp);
|
|
|
|
|
|
|
|
flock(fileno(fp), LOCK_UN);
|
|
|
|
fclose(fp);
|
|
|
|
}
|
|
|
|
|
2018-07-08 07:32:03 +00:00
|
|
|
#define CHECK_PATH(path) \
|
|
|
|
checkPath(path, __FUNCTION__)
|
|
|
|
|
2016-04-18 12:10:33 +00:00
|
|
|
static void
|
2018-07-08 07:32:03 +00:00
|
|
|
checkPath(const char *path,
|
|
|
|
const char *func)
|
2016-04-18 12:10:33 +00:00
|
|
|
{
|
|
|
|
char *fullPath = NULL;
|
|
|
|
char *relPath = NULL;
|
|
|
|
char *crippledPath = NULL;
|
|
|
|
|
2021-04-20 04:44:12 +00:00
|
|
|
if (!g_path_is_absolute(path))
|
2019-10-22 13:26:14 +00:00
|
|
|
relPath = g_strdup_printf("./%s", path);
|
2016-04-18 12:10:33 +00:00
|
|
|
|
2018-05-03 08:04:38 +00:00
|
|
|
/* Le sigh. virFileCanonicalizePath() expects @path to exist, otherwise
|
|
|
|
* it will return an error. So if we are called over an non-existent
|
|
|
|
* file, this could return an error. In that case do our best and hope
|
|
|
|
* we will catch possible errors. */
|
|
|
|
if ((fullPath = virFileCanonicalizePath(relPath ? relPath : path))) {
|
2016-04-18 12:10:33 +00:00
|
|
|
path = fullPath;
|
|
|
|
} else {
|
|
|
|
/* Yeah, our worst nightmares just became true. Path does
|
|
|
|
* not exist. Cut off the last component and retry. */
|
2019-10-20 11:49:46 +00:00
|
|
|
crippledPath = g_strdup(relPath ? relPath : path);
|
2016-04-18 12:10:33 +00:00
|
|
|
|
|
|
|
virFileRemoveLastComponent(crippledPath);
|
|
|
|
|
2018-05-03 08:04:38 +00:00
|
|
|
if ((fullPath = virFileCanonicalizePath(crippledPath)))
|
2016-04-18 12:10:33 +00:00
|
|
|
path = fullPath;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2019-03-12 14:11:47 +00:00
|
|
|
if (!STRPREFIX(path, abs_top_srcdir) &&
|
|
|
|
!STRPREFIX(path, abs_top_builddir)) {
|
2018-07-08 07:32:03 +00:00
|
|
|
printFile(path, func);
|
2016-04-18 12:10:33 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
VIR_FREE(crippledPath);
|
|
|
|
VIR_FREE(relPath);
|
|
|
|
VIR_FREE(fullPath);
|
|
|
|
|
|
|
|
return;
|
2016-05-13 12:08:29 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
int open(const char *path, int flags, ...)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
init_syms();
|
|
|
|
|
2018-07-08 07:32:03 +00:00
|
|
|
CHECK_PATH(path);
|
2016-05-13 12:08:29 +00:00
|
|
|
|
|
|
|
if (flags & O_CREAT) {
|
|
|
|
va_list ap;
|
|
|
|
mode_t mode;
|
|
|
|
va_start(ap, flags);
|
2018-04-30 15:30:12 +00:00
|
|
|
mode = (mode_t) va_arg(ap, int);
|
2016-05-13 12:08:29 +00:00
|
|
|
va_end(ap);
|
|
|
|
ret = real_open(path, flags, mode);
|
|
|
|
} else {
|
|
|
|
ret = real_open(path, flags);
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
FILE *fopen(const char *path, const char *mode)
|
|
|
|
{
|
|
|
|
init_syms();
|
|
|
|
|
2018-07-08 07:32:03 +00:00
|
|
|
CHECK_PATH(path);
|
2016-05-13 12:08:29 +00:00
|
|
|
|
|
|
|
return real_fopen(path, mode);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
int access(const char *path, int mode)
|
|
|
|
{
|
|
|
|
init_syms();
|
|
|
|
|
2018-07-08 07:32:03 +00:00
|
|
|
CHECK_PATH(path);
|
2016-05-13 12:08:29 +00:00
|
|
|
|
|
|
|
return real_access(path, mode);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2019-05-06 14:11:57 +00:00
|
|
|
#define VIR_MOCK_STAT_HOOK \
|
|
|
|
do { \
|
|
|
|
init_syms(); \
|
|
|
|
checkPath(path, "stat"); \
|
|
|
|
} while (0)
|
2016-05-13 12:08:29 +00:00
|
|
|
|
tests: fix mocking of stat() / lstat() functions
Quite a few of the tests have a need to mock the stat() / lstat()
functions and they are taking somewhat different & inconsistent
approaches none of which are actually fully correct. This is shown
by fact that 'make check' fails on 32-bit hosts. Investigation
revealed that the code was calling into the native C library impl,
not getting intercepted by our mocks.
The POSIX stat() function might resolve to any number of different
symbols in the C library.
The may be an additional stat64() function exposed by the headers
too.
On 64-bit hosts the stat & stat64 functions are identical, always
refering to the 64-bit ABI.
On 32-bit hosts they refer to the 32-bit & 64-bit ABIs respectively.
Libvirt uses _FILE_OFFSET_BITS=64 on 32-bit hosts, which causes the
C library to transparently rewrite stat() calls to be stat64() calls.
Libvirt will never see the 32-bit ABI from the traditional stat()
call. We cannot assume this rewriting is done using a macro. It might
be, but on GLibC it is done with a magic __asm__ statement to apply
the rewrite at link time instead of at preprocessing.
In GLibC there may be two additional functions exposed by the headers,
__xstat() and __xstat64(). When these exist, stat() and stat64() are
transparently rewritten to call __xstat() and __xstat64() respectively.
The former symbols will not actally exist in the library at all, only
the header. The leading "__" indicates the symbols are a private impl
detail of the C library that applications should not care about.
Unfortunately, because we are trying to mock replace the C library,
we need to know about this internal impl detail.
With all this in mind the list of functions we have to mock will depend
on several factors
- If _FILE_OFFSET_BITS is set, then we are on a 32-bit host, and we
only need to mock stat64 and __xstat64. The other stat / __xstat
functions exist, but we'll never call them so they can be ignored
for mocking.
- If _FILE_OFFSET_BITS is not set, then we are on a 64-bit host and
we should mock stat, stat64, __xstat & __xstat64. Either may be
called by app code.
- If __xstat & __xstat64 exist, then stat & stat64 will not exist
as symbols in the library, so the latter should not be mocked.
The same all applies to lstat()
These rules are complex enough that we don't want to duplicate them
across every mock file, so this centralizes all the logic in a helper
file virmockstathelper.c that should be #included when needed. The
code merely need to provide a filename rewriting callback called
virMockStatRedirect(). Optionally VIR_MOCK_STAT_HOOK can be defined
as a macro if further processing is needed inline.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
2019-04-01 16:47:25 +00:00
|
|
|
#include "virmockstathelpers.c"
|
virtestmock: Mock stat() properly
There is a lot to explain, but I try to make it as short as
possible. I'd start by pasting some parts of sys/stat.h:
extern int stat (const char *__restrict __file,
struct stat *__restrict __buf) __THROW __nonnull ((1, 2));
extern int __REDIRECT_NTH (stat, (const char *__restrict __file,
struct stat *__restrict __buf), stat64)
__nonnull ((1, 2));
__extern_inline int
__NTH (stat (const char *__path, struct stat *__statbuf))
{
return __xstat (_STAT_VER, __path, __statbuf);
}
Only one of these is effective at once, due to some usage of
the mess we are dealing with in here. So, basically, while
compiling or linking stat() in our code can be transformed into
some other func. Or a dragon.
Now, if you read stat(2) manpage, esp. "C library/kernel
differences" section, you'll learn that glibc uses some tricks
for older applications to work. I haven't gotten around actual
code that does this, but based on my observations, if 'stat'
symbol is found, glibc assumes it's dealing with ancient
application. Unfortunately, it can be just ours stat coming from
our mock. Therefore, calling stat() from a test will end up in
our mock. But since glibc is not exposing the symbol anymore, our
call of real_stat() will SIGSEGV immediately as the pointer to
function is NULL. Therefore, we should expose only those symbols
we know glibc has.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
2016-05-17 14:26:53 +00:00
|
|
|
|
2019-10-14 12:45:03 +00:00
|
|
|
static int virMockStatRedirect(const char *path G_GNUC_UNUSED, char **newpath G_GNUC_UNUSED)
|
virtestmock: Mock stat() properly
There is a lot to explain, but I try to make it as short as
possible. I'd start by pasting some parts of sys/stat.h:
extern int stat (const char *__restrict __file,
struct stat *__restrict __buf) __THROW __nonnull ((1, 2));
extern int __REDIRECT_NTH (stat, (const char *__restrict __file,
struct stat *__restrict __buf), stat64)
__nonnull ((1, 2));
__extern_inline int
__NTH (stat (const char *__path, struct stat *__statbuf))
{
return __xstat (_STAT_VER, __path, __statbuf);
}
Only one of these is effective at once, due to some usage of
the mess we are dealing with in here. So, basically, while
compiling or linking stat() in our code can be transformed into
some other func. Or a dragon.
Now, if you read stat(2) manpage, esp. "C library/kernel
differences" section, you'll learn that glibc uses some tricks
for older applications to work. I haven't gotten around actual
code that does this, but based on my observations, if 'stat'
symbol is found, glibc assumes it's dealing with ancient
application. Unfortunately, it can be just ours stat coming from
our mock. Therefore, calling stat() from a test will end up in
our mock. But since glibc is not exposing the symbol anymore, our
call of real_stat() will SIGSEGV immediately as the pointer to
function is NULL. Therefore, we should expose only those symbols
we know glibc has.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
2016-05-17 14:26:53 +00:00
|
|
|
{
|
tests: fix mocking of stat() / lstat() functions
Quite a few of the tests have a need to mock the stat() / lstat()
functions and they are taking somewhat different & inconsistent
approaches none of which are actually fully correct. This is shown
by fact that 'make check' fails on 32-bit hosts. Investigation
revealed that the code was calling into the native C library impl,
not getting intercepted by our mocks.
The POSIX stat() function might resolve to any number of different
symbols in the C library.
The may be an additional stat64() function exposed by the headers
too.
On 64-bit hosts the stat & stat64 functions are identical, always
refering to the 64-bit ABI.
On 32-bit hosts they refer to the 32-bit & 64-bit ABIs respectively.
Libvirt uses _FILE_OFFSET_BITS=64 on 32-bit hosts, which causes the
C library to transparently rewrite stat() calls to be stat64() calls.
Libvirt will never see the 32-bit ABI from the traditional stat()
call. We cannot assume this rewriting is done using a macro. It might
be, but on GLibC it is done with a magic __asm__ statement to apply
the rewrite at link time instead of at preprocessing.
In GLibC there may be two additional functions exposed by the headers,
__xstat() and __xstat64(). When these exist, stat() and stat64() are
transparently rewritten to call __xstat() and __xstat64() respectively.
The former symbols will not actally exist in the library at all, only
the header. The leading "__" indicates the symbols are a private impl
detail of the C library that applications should not care about.
Unfortunately, because we are trying to mock replace the C library,
we need to know about this internal impl detail.
With all this in mind the list of functions we have to mock will depend
on several factors
- If _FILE_OFFSET_BITS is set, then we are on a 32-bit host, and we
only need to mock stat64 and __xstat64. The other stat / __xstat
functions exist, but we'll never call them so they can be ignored
for mocking.
- If _FILE_OFFSET_BITS is not set, then we are on a 64-bit host and
we should mock stat, stat64, __xstat & __xstat64. Either may be
called by app code.
- If __xstat & __xstat64 exist, then stat & stat64 will not exist
as symbols in the library, so the latter should not be mocked.
The same all applies to lstat()
These rules are complex enough that we don't want to duplicate them
across every mock file, so this centralizes all the logic in a helper
file virmockstathelper.c that should be #included when needed. The
code merely need to provide a filename rewriting callback called
virMockStatRedirect(). Optionally VIR_MOCK_STAT_HOOK can be defined
as a macro if further processing is needed inline.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
2019-04-01 16:47:25 +00:00
|
|
|
return 0;
|
virtestmock: Mock stat() properly
There is a lot to explain, but I try to make it as short as
possible. I'd start by pasting some parts of sys/stat.h:
extern int stat (const char *__restrict __file,
struct stat *__restrict __buf) __THROW __nonnull ((1, 2));
extern int __REDIRECT_NTH (stat, (const char *__restrict __file,
struct stat *__restrict __buf), stat64)
__nonnull ((1, 2));
__extern_inline int
__NTH (stat (const char *__path, struct stat *__statbuf))
{
return __xstat (_STAT_VER, __path, __statbuf);
}
Only one of these is effective at once, due to some usage of
the mess we are dealing with in here. So, basically, while
compiling or linking stat() in our code can be transformed into
some other func. Or a dragon.
Now, if you read stat(2) manpage, esp. "C library/kernel
differences" section, you'll learn that glibc uses some tricks
for older applications to work. I haven't gotten around actual
code that does this, but based on my observations, if 'stat'
symbol is found, glibc assumes it's dealing with ancient
application. Unfortunately, it can be just ours stat coming from
our mock. Therefore, calling stat() from a test will end up in
our mock. But since glibc is not exposing the symbol anymore, our
call of real_stat() will SIGSEGV immediately as the pointer to
function is NULL. Therefore, we should expose only those symbols
we know glibc has.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
2016-05-17 14:26:53 +00:00
|
|
|
}
|
2018-07-08 07:24:34 +00:00
|
|
|
|
|
|
|
|
|
|
|
int connect(int sockfd, const struct sockaddr *addr, socklen_t addrlen)
|
|
|
|
{
|
|
|
|
init_syms();
|
|
|
|
|
2020-01-14 17:38:59 +00:00
|
|
|
#ifndef WIN32
|
2018-07-08 07:24:34 +00:00
|
|
|
if (addrlen == sizeof(struct sockaddr_un)) {
|
|
|
|
struct sockaddr_un *tmp = (struct sockaddr_un *) addr;
|
|
|
|
if (tmp->sun_family == AF_UNIX)
|
2018-07-08 07:32:03 +00:00
|
|
|
CHECK_PATH(tmp->sun_path);
|
2018-07-08 07:24:34 +00:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
|
|
|
return real_connect(sockfd, addr, addrlen);
|
|
|
|
}
|