On Tue, Dec 01, 2009 at 04:31:36PM +0800, Cong Wang wrote:Joe Perches wrote:On Tue, 2009-12-01 at 02:41 -0500, Amerigo Wang wrote:This is totally a personal taste, I think.This line is unreachable, remove it.[]diff --git a/security/selinux/ss/mls.c b/security/selinux/ss/mls.cI think it's better to remove the default case.
index b5407f1..a2f1034 100644
--- a/security/selinux/ss/mls.c
+++ b/security/selinux/ss/mls.c
@@ -544,7 +544,6 @@ int mls_compute_sid(struct context *scontext,
default:
return -EINVAL;
}
- return -EINVAL;
}
Either is OK. James, any comments?
I think the last unreachable return might also stop certain
versions of gcc complaining about control reaching the end of
a non void function.