diff options
| author | Al Viro | 2005-08-19 23:42:16 +0200 |
|---|---|---|
| committer | Linus Torvalds | 2005-08-20 02:57:19 +0200 |
| commit | 2fb1e3086df9b454538491fba8121298da37cd23 (patch) | |
| tree | d84587462e0378c5aa9c5581d6920d76242b8c65 /fs/cifs/misc.c | |
| parent | Merge master.kernel.org:/pub/scm/linux/kernel/git/bart/ide-2.6 (diff) | |
| download | kernel-qcow2-linux-2fb1e3086df9b454538491fba8121298da37cd23.tar.gz kernel-qcow2-linux-2fb1e3086df9b454538491fba8121298da37cd23.tar.xz kernel-qcow2-linux-2fb1e3086df9b454538491fba8121298da37cd23.zip | |
[PATCH] jffs2: fix symlink error handling
The current calling conventions for ->follow_link() are already fairly
complex.
What we have is
1) you can return -error; then you must release nameidata yourself
and ->put_link() will _not_ be called.
2) you can do nd_set_link(nd, ERR_PTR(-error)) and return 0
3) you can do nd_set_link(nd, path) and return 0
4) you can return 0 (after having moved nameidata yourself)
jffs2 follow_link() is broken - it has an exit where it returns
-EIO and leaks nameidata.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/cifs/misc.c')
0 files changed, 0 insertions, 0 deletions