* use setresuid() instead of setruid() if available

* if fix for the non-fp resampler

git-svn-id: file:///home/lennart/svn/public/pulseaudio/trunk@314 fefdeb5f-60dc-0310-8127-8f9354f1896f

diff --git a/configure.ac b/configure.ac
index 4db6b5b9f8e7b50cbb4d4ff16432f4e23c3f9f8a..3ab9491ff0a086989e4e79741be0e5846b85b4aa 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -96,6 +96,9 @@ AC_SUBST(CAP_LIBS)
 
 AC_CHECK_HEADERS(sys/capability.h)
 
+AC_CHECK_FUNCS(setresuid)
+AC_CHECK_FUNCS(setreuid)
+
 PKG_CHECK_MODULES(LIBSAMPLERATE, [ samplerate >= 0.1.0 ])
 AC_SUBST(LIBSAMPLERATE_CFLAGS)
 AC_SUBST(LIBSAMPLERATE_LIBS)

diff --git a/polyp/caps.c b/polyp/caps.c
index 34d97eb56ddb9d5af8171f1c5e2a5d064b1f50d7..93fca89f1f17b4571553bc0782b4c485ccdb7f2a 100644 (file)
--- a/polyp/caps.c
+++ b/polyp/caps.c
@@ -23,6 +23,9 @@
 #include <config.h>
 #endif
 
+/* setresuid() is only available on GNU */
+#define _GNU_SOURCE
+
 #include <assert.h>
 #include <unistd.h>
 #include <errno.h>
@@ -41,13 +44,17 @@ void pa_drop_root(void) {
     
     if (uid == 0 || geteuid() != 0)
         return;
-    
-/*     pa_log(__FILE__": dropping root rights.\n"); */
-    
-    setreuid(uid, uid);
 
-/*    setuid(uid);
-    seteuid(uid);*/
+    /*     pa_log(__FILE__": dropping root rights.\n"); */
+
+#if defined(HAVE_SETRESUID)
+    setresuid(uid, uid, uid);
+#elif defined(HAVE_SETREUID)
+    setreuid(uid, uid);
+#else
+    setuid(uid);
+    seteuid(uid);
+#endif
 }
 
 #ifdef HAVE_SYS_CAPABILITY_H

diff --git a/polyp/resampler.c b/polyp/resampler.c
index 377aa79758f6cf4be039a6ef6c752a8597706f06..68c0129ad3ac7a7f4c8293a582cdd8ad69085ba6 100644 (file)
--- a/polyp/resampler.c
+++ b/polyp/resampler.c
@@ -342,8 +342,8 @@ static void trivial_run(struct pa_resampler *r, const struct pa_memchunk *in, st
             unsigned j;
             
             j = (i->o_counter * r->i_ss.rate / r->o_ss.rate);
-            assert(j >= i->i_counter);
-            j = j - i->i_counter;
+            
+            j = j > i->i_counter ? j - i->i_counter : 0;
             
             if (j >= nframes)
                 break;