diff options
| author | Andrew Wedgbury <andrew.wedgbury@realvnc.com> | 2014-04-07 12:40:35 +0100 |
|---|---|---|
| committer | Kristian Høgsberg <krh@bitplanet.net> | 2014-04-07 10:22:28 -0700 |
| commit | 9cd661e746708a76761ff1fda361c6276ee71ab1 (patch) | |
| tree | a69e2745f3d3a64e31a1e9675a25738ec231018c /shared | |
| parent | ba83db216ca8b84c250f9902fbbd4883cf63912b (diff) | |
| download | weston-9cd661e746708a76761ff1fda361c6276ee71ab1.tar.gz | |
Make sure config.h is included before any system headers
There was an issue recently in screen-share.c where config.h was not
being included, resulting in the wrong definition for off_t being used on
32 bit systems. I checked and I don't think this problem is happening
elsewhere, but to help avoid this sort of problem in the future, I went
through and made sure that config.h is included first whenever system
headers are included.
The config.h header should be included before any system headers, failing
to do this can result in the wrong type sizes being defined on certain
systems, e.g. off_t from sys/types.h
Signed-off-by: Andrew Wedgbury <andrew.wedgbury@realvnc.com>
Diffstat (limited to 'shared')
| -rw-r--r-- | shared/os-compatibility.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/shared/os-compatibility.h b/shared/os-compatibility.h index 10dbb7dd..172bb7ed 100644 --- a/shared/os-compatibility.h +++ b/shared/os-compatibility.h @@ -23,10 +23,10 @@ #ifndef OS_COMPATIBILITY_H #define OS_COMPATIBILITY_H -#include <sys/types.h> - #include "config.h" +#include <sys/types.h> + #ifdef HAVE_EXECINFO_H #include <execinfo.h> #else |