diff options
| author | Eric Paris <[email protected]> | 2009-11-20 16:00:12 +0000 |
|---|---|---|
| committer | James Morris <[email protected]> | 2009-11-22 22:00:44 +0000 |
| commit | 85c3b529f8ad4d65ba86b982ef050212ae7dd976 (patch) | |
| tree | becd5d4d607faf5de3bf5d8470cacf715aef8f7a /scripts/selinux/genheaders/genheaders.c | |
| parent | selinux: Fix warnings (diff) | |
| download | kernel-85c3b529f8ad4d65ba86b982ef050212ae7dd976.tar.gz kernel-85c3b529f8ad4d65ba86b982ef050212ae7dd976.zip | |
SELinux: header generation may hit infinite loop
If a permission name is long enough the selinux class definition generation
tool will go into a infinite loop. This is because it's macro max() is
fooled into thinking it is dealing with unsigned numbers. This patch makes
sure the macro always uses signed number so 1 > -1.
Signed-off-by: Eric Paris <[email protected]>
Signed-off-by: James Morris <[email protected]>
Diffstat (limited to 'scripts/selinux/genheaders/genheaders.c')
| -rw-r--r-- | scripts/selinux/genheaders/genheaders.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/selinux/genheaders/genheaders.c b/scripts/selinux/genheaders/genheaders.c index 771b86f46194..24626968055d 100644 --- a/scripts/selinux/genheaders/genheaders.c +++ b/scripts/selinux/genheaders/genheaders.c @@ -13,7 +13,7 @@ struct security_class_mapping { #include "classmap.h" #include "initial_sid_to_string.h" -#define max(x, y) ((x > y) ? x : y) +#define max(x, y) (((int)(x) > (int)(y)) ? x : y) const char *progname; |