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..