diff options
author | Vesa-Matti Kari <vmkari@cc.helsinki.fi> | 2008-08-06 18:24:51 +0300 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2008-08-07 08:56:16 +1000 |
commit | 421fae06be9e0dac45747494756b3580643815f9 (patch) | |
tree | 8b390e53636092477c82304b7f7f10524df6fd1b /security/selinux/ss/services.c | |
parent | 15446235367fa4a621ff5abfa4b6ebbe25b33763 (diff) | |
download | linux-421fae06be9e0dac45747494756b3580643815f9.tar.gz linux-421fae06be9e0dac45747494756b3580643815f9.tar.bz2 linux-421fae06be9e0dac45747494756b3580643815f9.zip |
selinux: conditional expression type validation was off-by-one
expr_isvalid() in conditional.c was off-by-one and allowed
invalid expression type COND_LAST. However, it is this header file
that needs to be fixed. That way the if-statement's disjunction's
second component reads more naturally, "if expr type is greater than
the last allowed value" ( rather than using ">=" in conditional.c):
if (expr->expr_type <= 0 || expr->expr_type > COND_LAST)
Signed-off-by: Vesa-Matti Kari <vmkari@cc.helsinki.fi>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/selinux/ss/services.c')
0 files changed, 0 insertions, 0 deletions