From: Serge Hallyn That should be -EINVAL for both. For that matter, the braces in the next line shouldn't be there. Signed-off-by: Serge Hallyn Signed-off-by: Andrew Morton --- security/seclvl.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff -puN security/seclvl.c~seclvl-use-securityfs-fix security/seclvl.c --- devel/security/seclvl.c~seclvl-use-securityfs-fix 2005-08-16 00:12:57.000000000 -0700 +++ devel-akpm/security/seclvl.c 2005-08-16 00:13:53.000000000 -0700 @@ -252,7 +252,7 @@ passwd_write_file(struct file * file, co } if (count < 0 || count >= PAGE_SIZE) - return -ENOMEM; + return -EINVAL; if (*ppos != 0) return -EINVAL; page = (char *)get_zeroed_page(GFP_KERNEL); _