isofs: Fix isofs_show_options()
authorDavid Howells <dhowells@redhat.com>
Tue, 11 Jul 2017 15:22:59 +0000 (16:22 +0100)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 13 Jul 2017 16:30:43 +0000 (12:30 -0400)
The isofs patch needs a small fix to handle a signed/unsigned comparison that
the compiler didn't flag - thanks to Dan for catching it.

It should be noted, however, the session number handing appears to be incorrect
between where it is parsed and where it is used.

Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/isofs/inode.c

index f80ee600d1bcd248105d13d25115a532842888e0..8cf898a59730dff80eec216b069cd415fbd5ca70 100644 (file)
@@ -492,7 +492,7 @@ static int isofs_show_options(struct seq_file *m, struct dentry *root)
 
        if (sbi->s_check)               seq_printf(m, ",check=%c", sbi->s_check);
        if (sbi->s_mapping)             seq_printf(m, ",map=%c", sbi->s_mapping);
 
        if (sbi->s_check)               seq_printf(m, ",check=%c", sbi->s_check);
        if (sbi->s_mapping)             seq_printf(m, ",map=%c", sbi->s_mapping);
-       if (sbi->s_session != -1)       seq_printf(m, ",session=%u", sbi->s_session);
+       if (sbi->s_session != 255)      seq_printf(m, ",session=%u", sbi->s_session - 1);
        if (sbi->s_sbsector != -1)      seq_printf(m, ",sbsector=%u", sbi->s_sbsector);
 
        if (root->d_sb->s_blocksize != 1024)
        if (sbi->s_sbsector != -1)      seq_printf(m, ",sbsector=%u", sbi->s_sbsector);
 
        if (root->d_sb->s_blocksize != 1024)