]> granicus.if.org Git - pdns/commitdiff
utility: use gid_t and uid_t
authorPieter Lexis <pieter.lexis@powerdns.com>
Thu, 6 Jun 2019 10:54:57 +0000 (12:54 +0200)
committerPieter Lexis <pieter.lexis@powerdns.com>
Thu, 6 Jun 2019 10:54:57 +0000 (12:54 +0200)
pdns/common_startup.cc
pdns/unix_utility.cc
pdns/utility.hh

index 07d78c1cd9f2ccb9eb2a55943891d55dd8de6a63..034e13e4199a5a450ed99e39d37959e31e6a562c 100644 (file)
@@ -499,12 +499,12 @@ void mainthread()
 {
    Utility::srandom();
 
-   int newgid=0;
-   if(!::arg()["setgid"].empty()) 
-     newgid=Utility::makeGidNumeric(::arg()["setgid"]);      
-   int newuid=0;      
-   if(!::arg()["setuid"].empty())        
-     newuid=Utility::makeUidNumeric(::arg()["setuid"]); 
+   gid_t newgid = 0;
+   if(!::arg()["setgid"].empty())
+     newgid=Utility::makeGidNumeric(::arg()["setgid"]);
+   uid_t newuid = 0;
+   if(!::arg()["setuid"].empty())
+     newuid=Utility::makeUidNumeric(::arg()["setuid"]);
    
    g_anyToTcp = ::arg().mustDo("any-to-tcp");
    g_8bitDNS = ::arg().mustDo("8bit-dns");
index 6e5d817a09d3e81016ffcc3b3d691acfae617408..201c73fc0d7a2a3a52b644a94d6e53e0d1131f9c 100644 (file)
@@ -124,7 +124,7 @@ void Utility::usleep(unsigned long usec)
 
 
 // Drops the program's group privileges.
-void Utility::dropGroupPrivs( int uid, int gid )
+void Utility::dropGroupPrivs( uid_t uid, gid_t gid )
 {
   if(gid) {
     if(setgid(gid)<0) {
@@ -152,7 +152,7 @@ void Utility::dropGroupPrivs( int uid, int gid )
 
 
 // Drops the program's user privileges.
-void Utility::dropUserPrivs( int uid )
+void Utility::dropUserPrivs( uid_t uid )
 {
   if(uid) {
     if(setuid(uid)<0) {
@@ -181,9 +181,9 @@ int Utility::gettimeofday( struct timeval *tv, void *tz )
 
 
 // Retrieves a gid using a groupname.
-int Utility::makeGidNumeric(const string &group)
+gid_t Utility::makeGidNumeric(const string &group)
 {
-  int newgid;
+  gid_t newgid;
   if(!(newgid=atoi(group.c_str()))) {
     errno=0;
     struct group *gr=getgrnam(group.c_str());
@@ -198,9 +198,9 @@ int Utility::makeGidNumeric(const string &group)
 
 
 // Retrieves an uid using a username.
-int Utility::makeUidNumeric(const string &username)
+uid_t Utility::makeUidNumeric(const string &username)
 {
-  int newuid;
+  uid_t newuid;
   if(!(newuid=atoi(username.c_str()))) {
     struct passwd *pw=getpwnam(username.c_str());
     if(!pw) {
index 024fc089fb111a5b6f66f239e60d2f32ebb39cca..626131d8bb42c6fc7efb8c240c522899a0356ef1 100644 (file)
@@ -121,10 +121,10 @@ public:
   static const char *inet_ntop( int af, const char *src, char *dst, size_t size );
 
   //! Retrieves a gid using a groupname.
-  static int makeGidNumeric( const string & group );
+  static gid_t makeGidNumeric( const string & group );
   
   //! Retrieves an uid using an username.
-  static int makeUidNumeric( const string & username );
+  static uid_t makeUidNumeric( const string & username );
 
   //! Writes a vector.
   static int writev( Utility::sock_t socket, const iovec *vector, size_t count );
@@ -133,10 +133,10 @@ public:
   static void srandom(void);
 
   //! Drops the program's group privileges.
-  static void dropGroupPrivs( int uid, int gid );
+  static void dropGroupPrivs( uid_t uid, gid_t gid );
 
   //! Drops the program's user privileges.
-  static void dropUserPrivs( int uid );
+  static void dropUserPrivs( uid_t uid );
   
   //! Sets the socket into Bind-any mode
   static void setBindAny ( int af, Utility::sock_t socket );