git://ftp.safe.ca / safe / jmp / linux-2.6 / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 2933392) | patch
fix affs parse_options()
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 24 Jan 2010 05:06:22 +0000 (00:06 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 27 Jan 2010 03:22:25 +0000 (22:22 -0500)
commit217686e98321a4ff4c1a6cc535e511e37c5d2dbf
tree8b68ea06a602d92d52e6c5a9d3c976a8991cccc7tree | snapshot
parent29333920a5a46edcc9b728e2cf0134d5a9b516eecommit | diff
fix affs parse_options()

Error handling in that sucker got broken back in 2003.  If function
returns 0 on failure, it's not nice to add return -EINVAL into it.
Adding return 1 on other failure exits is also not a good thing (and
yes, original success exits with 1 and some of failure exits with 0
are still there; so's the original logics in callers).

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/affs/super.c diff | blob | history
Full containers within linux kernel