linux-kernel - Re: btrfs: add ability to change features via sysfs

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>] [day] [month] [year] [list]

Message-ID: <20140131153535.GA30957@redhat.com>
Date:	Fri, 31 Jan 2014 10:35:35 -0500
From:	Dave Jones <davej@...hat.com>
To:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Cc:	clm@...com, jeffm@...e.com
Subject: Re: btrfs: add ability to change features via sysfs

On Fri, Jan 31, 2014 at 04:12:00AM +0000, Linux Kernel wrote:
 > Gitweb:     http://git.kernel.org/linus/;a=commit;h=ba631941ef09c10e229661219dbd1707e56131d8
 > Commit:     ba631941ef09c10e229661219dbd1707e56131d8
 > Parent:     79da4fa4d9dcf8c948ef8b5848f747ef08f6e732
 > Author:     Jeff Mahoney <jeffm@...e.com>
 > AuthorDate: Fri Nov 1 13:07:01 2013 -0400
 > Committer:  Chris Mason <clm@...com>
 > CommitDate: Tue Jan 28 13:19:27 2014 -0800
 > 
 >     btrfs: add ability to change features via sysfs
 
 
 > +static int can_modify_feature(struct btrfs_feature_attr *fa)
 > +{
 > +	int val = 0;
 > +	u64 set, clear;
 > +	switch (fa->feature_set) {
 > +	case FEAT_COMPAT:
 > +		set = BTRFS_FEATURE_COMPAT_SAFE_SET;
 > +		clear = BTRFS_FEATURE_COMPAT_SAFE_CLEAR;
 > +		break;
 > +	case FEAT_COMPAT_RO:
 > +		set = BTRFS_FEATURE_COMPAT_RO_SAFE_SET;
 > +		clear = BTRFS_FEATURE_COMPAT_RO_SAFE_CLEAR;
 > +		break;
 > +	case FEAT_INCOMPAT:
 > +		set = BTRFS_FEATURE_INCOMPAT_SAFE_SET;
 > +		clear = BTRFS_FEATURE_INCOMPAT_SAFE_CLEAR;
 > +		break;
 > +	default:
 > +		BUG();
 > +	}
 > +
 > +	if (set & fa->feature_bit)
 > +		val |= 1;
 > +	if (clear & fa->feature_bit)
 > +		val |= 2;
 > +
 > +	return val;
 > +}

Coverity flagged 9 new issues in btrfs last night.

It flagged this (and a bunch of similar areas) because the CLEAR defines are 0.

Doing if (0 & fa->feature_bit) is always going to be false.

There's 72 other btrfs bugs at http://scan.coverity.com (twice that of say, ext* or xfs).
Many of them look real, though some look complicated enough that they really need
someone familiar with how btrfs works to know "can't happen". If some of you want
to sign up, I can get the access approved quickly.

thanks,

	Dave

--
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/