Re: [PATCH] Add compat function for geteuid()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 15, 2011 at 10:06:23AM -0400, Laine Stump wrote:
> On 03/15/2011 08:32 AM, Daniel P. Berrange wrote:
> >* configure.ac: Check for geteuid()
> >* src/util/util.h: Compat for geteuid()
> >---
> >  configure.ac    |    2 +-
> >  src/util/util.h |    4 ++++
> >  2 files changed, 5 insertions(+), 1 deletions(-)
> >
> >diff --git a/configure.ac b/configure.ac
> >index a58ee4e..e2b2b24 100644
> >--- a/configure.ac
> >+++ b/configure.ac
> >@@ -120,7 +120,7 @@ AC_MSG_RESULT([$have_cpuid])
> >  dnl Availability of various common functions (non-fatal if missing),
> >  dnl and various less common threadsafe functions
> >  AC_CHECK_FUNCS_ONCE([cfmakeraw regexec sched_getaffinity getuid getgid \
> >- initgroups posix_fallocate mmap kill \
> >+ geteuid initgroups posix_fallocate mmap kill \
> >   getmntent_r getgrnam_r getpwuid_r])
> >
> >  dnl Availability of pthread functions (if missing, win32 threading is
> >diff --git a/src/util/util.h b/src/util/util.h
> >index 31c3a33..276e259 100644
> >--- a/src/util/util.h
> >+++ b/src/util/util.h
> >@@ -269,6 +269,10 @@ const char *virEnumToString(const char *const*types,
> >  static inline int getuid (void) { return 0; }
> >  # endif
> >
> >+# ifndef HAVE_GETUID
> 
> This should be HAVE_GETEUID, not HAVE_GETUID ;-)

Yes, I'll push without the stupid typo !


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]