s-s-d: Fix signed vs unsigned warning when comparing to getuid and getgid

diff --git a/ChangeLog b/ChangeLog
index dfa4d13ea2b59b298a8f95f3bc8dd80d10c963cc..7fa3d489650b1d943c14f4bcd159a0e83aae7a4c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-01-21  Guillem Jover  <guillem@debian.org>
+
+       * utils/start-stop-daemon.c (main): Cast runas_uid and runas_gid to
+       uid_t and gid_t when comparing to getuid() and getgid().
+
 2008-01-21  Guillem Jover  <guillem@debian.org>
 
        * configure.ac (AC_CHECK_FUNCS): Merge arguments from both macro

diff --git a/utils/start-stop-daemon.c b/utils/start-stop-daemon.c
index cf34c6d5ade219c2dd0374b68b78b411d7a2e6fa..ed21c3c2627dea0a19c2689a2927d02955b26760 100644 (file)
--- a/utils/start-stop-daemon.c
+++ b/utils/start-stop-daemon.c
@@ -1413,11 +1413,12 @@ main(int argc, char **argv)
        if (chdir(changedir) < 0)
                fatal("Unable to chdir() to %s", changedir);
 
-       if (changeuser != NULL && getuid() != runas_uid) {
+       if (changeuser != NULL && getuid() != (uid_t)runas_uid) {
                if (setuid(runas_uid))
                        fatal("Unable to set uid to %s", changeuser);
        }
-       if (changegroup != NULL && *changegroup != '\0' && getgid() != runas_gid) {
+       if (changegroup != NULL && *changegroup != '\0' &&
+           getgid() != (gid_t)runas_gid) {
                if (!gid_in_current_groups(runas_gid))
                        if (initgroups(changeuser, runas_gid))
                                fatal("Unable to set initgroups() with gid %d",