[SSSD] [PATCH] Include sys/types.h for types id_t and uid_t

Jakub Hrozek jhrozek at redhat.com
Mon Sep 2 21:02:44 UTC 2013


On Mon, Sep 02, 2013 at 07:12:51PM +0200, Lukas Slebodnik wrote:
> ehlo,
> 
> I checked some manual pages, where [ug]?id types are used and each manual page
> suggest to include header file "sys/types.h". This header file was indirectly
> included in some files on linux, but it is not portable.
> 
> man getgid
>   #include <unistd.h>
>   #include <sys/types.h>
> 
>   gid_t getgid(void);
>   gid_t getegid(void);
> -------------------------
> man getpwuid
>   #include <sys/types.h>
>   #include <pwd.h>     
> 
>   struct passwd *getpwuid(uid_t uid);
> 

I think the change is correct. The POSIX information I checked
(http://pubs.opengroup.org/onlinepubs/009695399/basedefs/pwd.h.html for
example) say that "The gid_t and uid_t types shall be defined as
described in <sys/types.h>."

But I wonder if these are the only files that needed fixing? uid_t seems
to be used all over the place..



More information about the sssd-devel mailing list