On Wed, 2002-05-01 at 13:18, Alexander Viro wrote:
> Yes, it is. Look for the patch I've posted yesterday (subject was
> something like "[PATCH] missing checks", IIRC).
Before I found this message I was also trying to hunt down this problem
after noticing that 2.5.12 failed unlink08 from LTP. The thing I think
broke it though was a seemlingly minor change to the handling of the
return value from exec_permission_lite in link_path_walk(). In 2.5.11
it rechecked with permission() if it got an error back from
exec_permission_lite(), but in 2.5.12 it only does this if err ==
-EAGAIN. This patch also seems to fix the problem, and simply reverts
it back to the original way it was handled rather than adding more code
to exec_permission_lite(). I'll let you decide which is the best way.
Thanks,
Paul Larson
--- linux/fs/namei.c Thu May 2 18:36:01 2002
+++ linux-fix/fs/namei.c Thu May 2 18:36:17 2002
@@ -573,7 +573,7 @@
unsigned int c;
err = exec_permission_lite(inode);
- if (err == -EAGAIN) {
+ if (err) {
unlock_nd(nd);
err = permission(inode, MAY_EXEC);
lock_nd(nd);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Tue May 07 2002 - 22:00:16 EST