]> git.ipfire.org Git - thirdparty/openssh-portable.git/commitdiff
Cast *ID types to unsigned long when printing.
authorDarren Tucker <dtucker@dtucker.net>
Sat, 6 Jul 2019 03:02:34 +0000 (13:02 +1000)
committerDarren Tucker <dtucker@dtucker.net>
Sat, 6 Jul 2019 03:02:34 +0000 (13:02 +1000)
UID and GID types vary by platform so cast to u_long and use %lu when
printing them to prevent warnings.

openbsd-compat/bsd-setres_id.c
openbsd-compat/pwcache.c

index 696ae7b28136f33a0940629d06596b6288aa673f..a8e76bf60dcd78d53fd9ffda438d10516eca0589 100644 (file)
@@ -37,20 +37,20 @@ setresgid(gid_t rgid, gid_t egid, gid_t sgid)
 #if defined(HAVE_SETREGID) && !defined(BROKEN_SETREGID)
        if (setregid(rgid, egid) < 0) {
                saved_errno = errno;
-               error("setregid %u: %.100s", rgid, strerror(errno));
+               error("setregid %lu: %.100s", (u_long)rgid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
 #else
        if (setegid(egid) < 0) {
                saved_errno = errno;
-               error("setegid %u: %.100s", (u_int)egid, strerror(errno));
+               error("setegid %lu: %.100s", (u_long)egid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
        if (setgid(rgid) < 0) {
                saved_errno = errno;
-               error("setgid %u: %.100s", rgid, strerror(errno));
+               error("setgid %ul: %.100s", (u_long)rgid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
@@ -72,7 +72,7 @@ setresuid(uid_t ruid, uid_t euid, uid_t suid)
 #if defined(HAVE_SETREUID) && !defined(BROKEN_SETREUID)
        if (setreuid(ruid, euid) < 0) {
                saved_errno = errno;
-               error("setreuid %u: %.100s", ruid, strerror(errno));
+               error("setreuid %lu: %.100s", (u_long)ruid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
@@ -81,14 +81,14 @@ setresuid(uid_t ruid, uid_t euid, uid_t suid)
 # ifndef SETEUID_BREAKS_SETUID
        if (seteuid(euid) < 0) {
                saved_errno = errno;
-               error("seteuid %u: %.100s", euid, strerror(errno));
+               error("seteuid %lu: %.100s", (u_long)euid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
 # endif
        if (setuid(ruid) < 0) {
                saved_errno = errno;
-               error("setuid %u: %.100s", ruid, strerror(errno));
+               error("setuid %lu: %.100s", (u_long)ruid, strerror(errno));
                errno = saved_errno;
                ret = -1;
        }
index 5a8b78801b91138fd93575936429d1052176bd9d..826c2378ba258072269995549e3c9ab98ee2f332 100644 (file)
@@ -67,7 +67,7 @@ user_from_uid(uid_t uid, int nouser)
                if ((pw = getpwuid(uid)) == NULL) {
                        if (nouser)
                                return (NULL);
-                       (void)snprintf(nbuf, sizeof(nbuf), "%u", uid);
+                       (void)snprintf(nbuf, sizeof(nbuf), "%lu", (u_long)uid);
                }
                cp->uid = uid;
                if (cp->name != NULL)
@@ -102,7 +102,7 @@ group_from_gid(gid_t gid, int nogroup)
                if ((gr = getgrgid(gid)) == NULL) {
                        if (nogroup)
                                return (NULL);
-                       (void)snprintf(nbuf, sizeof(nbuf), "%u", gid);
+                       (void)snprintf(nbuf, sizeof(nbuf), "%lu", (u_long)gid);
                }
                cp->gid = gid;
                if (cp->name != NULL)