External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-11 12:25:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

avoid malfunction when virFileResolveLink is applied to non-POSIX FS

Browse Source 
The virFileResolveLink utility function relied on the POSIX guarantee
that stat.st_size of a symlink is the length of the value.  However,
on some types of file systems, it is invalid, so do not rely on it.
Use gnulib's areadlink module instead.
* bootstrap (modules): Add areadlink.
* src/util/util.c: Include "areadlink.h".
Let areadlink perform the readlink and malloc.
* configure.in (AC_CHECK_FUNCS): Remove readlink.  No need,
since it's presence is guaranteed by gnulib.
This commit is contained in:
Jim Meyering 2009-12-15 08:27:53 +01:00
parent f3cbd24ddc
commit 5baa463541
3 changed files with 5 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
bootstrap
View File

@ -65,6 +65,7 @@ gnulib_tool=$GNULIB_SRCDIR/gnulib-tool
<$gnulib_tool || exit
modules='
areadlink
base64
c-ctype
close

2
configure.in
View File

@ -83,7 +83,7 @@ dnl Use --disable-largefile if you don't want this.
AC_SYS_LARGEFILE
dnl Availability of various common functions (non-fatal if missing).
AC_CHECK_FUNCS([cfmakeraw regexec uname sched_getaffinity getuid getgid posix_fallocate mmap readlink])
AC_CHECK_FUNCS([cfmakeraw regexec uname sched_getaffinity getuid getgid posix_fallocate mmap])
dnl Availability of various not common threadsafe functions
AC_CHECK_FUNCS([strerror_r strtok_r getmntent_r getgrnam_r getpwuid_r])

27
src/util/util.c
View File

@ -64,6 +64,7 @@
#include <mntent.h>
#endif
#include "areadlink.h"
#include "virterror_internal.h"
#include "logging.h"
#include "event.h"
@ -1059,10 +1060,7 @@ int virFileLinkPointsTo(const char *checkLink,
int virFileResolveLink(const char *linkpath,
char **resultpath)
{
#ifdef HAVE_READLINK
struct stat st;
char *buf;
int n;
*resultpath = NULL;
@ -1075,28 +1073,9 @@ int virFileResolveLink(const char *linkpath,
return 0;
}
/* Posix says that 'st_size' field from
* result of an lstat() call is filled with
* number of bytes in the destination
* filename.
*/
if (VIR_ALLOC_N(buf, st.st_size + 1) < 0)
return -ENOMEM;
*resultpath = areadlink (linkpath);
if ((n = readlink(linkpath, buf, st.st_size)) < 0) {
VIR_FREE(buf);
return -errno;
}
buf[n] = '\0';
*resultpath = buf;
return 0;
#else
if (!(*resultpath = strdup(linkpath)))
return -ENOMEM;
return 0;
#endif
return *resultpath == NULL ? -1 : 0;
}
/*