linux-kernel - Re: [PATCH] [RFC] [resend] isofs: work-around for Rock Ridge+Joliet CDs with empty ISO root directory

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 31 Aug 2010 23:49:38 +0200
From:	Ondrej Zary <linux@...nbow-software.org>
To:	guenter.roeck@...csson.com
Cc:	linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] [RFC] [resend] isofs: work-around for Rock Ridge+Joliet CDs with empty ISO root directory

On Tuesday 31 August 2010 22:44:55 Guenter Roeck wrote:
> On Tue, 2010-08-31 at 14:10 -0400, Ondrej Zary wrote:
> > If a CD has both Rock Ridge and Joliet extensions and the ISO root
> > directory is empty, no files are visible. Disable Rock Ridge extensions
> > in this case and use Joliet root directory instead.
> >
> > Signed-off-by: Ondrej Zary <linux@...nbow-software.org>
> >
> > --- linux-2.6.35-rc2/fs/isofs/inode.c	2010-06-06 05:43:24.000000000 +0200
> > +++ linux-2.6.35-rc3/fs/isofs/inode.c	2010-08-24 22:43:26.000000000 +0200
> > @@ -549,6 +549,34 @@ static unsigned int isofs_get_last_sessi
> >  }
> >
> >  /*
> > + * Check if root directory is empty (has less than 3 files).
> > + *
> > + * Used to detect broken CDs where ISO root directory is empty but
> > Joliet root + * directory is OK. If such CD has Rock Ridge extensions,
> > they will be disabled + * (and Joliet used instead) or else no files
> > would be visible. + */
> > +static bool rootdir_empty(struct super_block *sb, unsigned long block)
> > +{
> > +	int offset, files, de_len;
> > +	struct iso_directory_record *de;
> > +	struct buffer_head *bh;
> > +
> > +	bh = sb_bread(sb, block);
> > +	if (!bh)
> > +		return true;
> > +	while (files < 3) {
>
> files is uninitialized.
>
> > +		de = (struct iso_directory_record *) (bh->b_data + offset);
>
> offset is uninitialized.
>
> Did you test this patch ?

Oops... I tested it and it seemed to work fine. Even gcc did not complain...

-- 
Ondrej Zary
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/