From: Eric Paris Date: Fri, 20 Nov 2009 16:00:12 +0000 (-0500) Subject: SELinux: header generation may hit infinite loop X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=85c3b529f8ad4d65ba86b982ef050212ae7dd976;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git 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 Signed-off-by: James Morris --- 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;