tree e38358e15cfb5c2c5e8eec93b3f15a459e9b958b parent 3e6c661ddb88083186ef55ced851d5a834bc34c0 author Horms <[EMAIL PROTECTED]> Tue, 16 Aug 2005 17:38:09 +0900 committer Marcelo Tosatti <[EMAIL PROTECTED]> Tue, 16 Aug 2005 18:09:03 -0300
[PATCH] isofs driver ignore parameters It seems that Alexander Pytlev's original (simple) patch was correct. Without it the logic looks a bit like this. while (...) { if iocharset ... else if map ... if session ... if sbsector ... else if check ... ... else return 1; } Now, if iocharset, map or session are matched, then none of the if or else if clauses under sbsector will match (that is none of these clauses match iocharset, map or session), and thus the else clause will be hit, and the function will return 1 without parsing any furhter options. With Alexander's fix, the if session and if sbsector clauses become else if, and its easy to see that the return 1 won't be premeturely called. I have tested that this patch works using the testcase options iocharset=koi8-r,gid=100, and checking that gid is set correctly with the patch, and incorrectly without. Here is the patch and signoff again, just for the record. I will send a second patch to clean up the *value = 0 code that Marcelo cast concerns over - its bogus but harmless. Signed-off-by: Horms <[EMAIL PROTECTED]> fs/isofs/inode.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/isofs/inode.c b/fs/isofs/inode.c --- a/fs/isofs/inode.c +++ b/fs/isofs/inode.c @@ -335,13 +335,13 @@ static int parse_options(char *options, else if (!strcmp(value,"acorn")) popt->map = 'a'; else return 0; } - if (!strcmp(this_char,"session") && value) { + else if (!strcmp(this_char,"session") && value) { char * vpnt = value; unsigned int ivalue = simple_strtoul(vpnt, &vpnt, 0); if(ivalue < 0 || ivalue >99) return 0; popt->session=ivalue+1; } - if (!strcmp(this_char,"sbsector") && value) { + else if (!strcmp(this_char,"sbsector") && value) { char * vpnt = value; unsigned int ivalue = simple_strtoul(vpnt, &vpnt, 0); if(ivalue < 0 || ivalue >660*512) return 0; - To unsubscribe from this list: send the line "unsubscribe git-commits-24" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html