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:	Mon, 5 May 2014 16:11:41 +0200 (CEST)
From:	Lukáš Czerner <lczerner@...hat.com>
To:	"Theodore Ts'o" <tytso@....edu>
cc:	Ext4 Developers List <linux-ext4@...r.kernel.org>
Subject: Re: [PATCH 3/3] mke2fs: check for a partition table and warn if
 present

On Mon, 5 May 2014, Theodore Ts'o wrote:

> Date: Mon, 5 May 2014 09:58:04 -0400
> From: Theodore Ts'o <tytso@....edu>
> To: Lukáš Czerner <lczerner@...hat.com>
> Cc: Ext4 Developers List <linux-ext4@...r.kernel.org>
> Subject: Re: [PATCH 3/3] mke2fs: check for a partition table and warn if
>     present
> 
> On Mon, May 05, 2014 at 03:52:05PM +0200, Lukáš Czerner wrote:
> > > +	ret = check_partition_table(device);
> > 
> > This can be actually used to check more than just partitions. So we
> > can use this approach to check for all rather than having separate
> > checks for file system signatures and partitions.
> 
> The issue is that e2fsprogs gets compiled for systems other than just
> Linux.  I don't want to be like the assholes who work on systemd and
> GNOME that simply screw over *BSD systems.  This is why I keep our
> internal version of blkid in e2fsprogs, even if I do plan to use the
> system blkid by default for 1.43.
> 
> If I had infinite amounts of free time, I'd backport the the new
> blkid_probe interfaces to our internal version of blkid, but since I
> don't, I prefer use the old interfaces for blkid as much as possible,
> since that's the path of least resistance in terms of continuing to
> support non-Linux users of e2fsprogs.

Fair enough. But we should still make the use of system libblkid by
default if you do not have any objections.

Also it'll be great to mention that in the commit description that
this is the reason why we still try to use the old approach.

> 
> > Also in your check_partition_table() you do not disable probing for
> > supeblocks even though you do not look to them afterwards so it's
> > not a big deal. But again I think that we can use it to check for
> > all signatures.
> 
> How do you disable probing for superblocks with the new blkid interface?

blkid_probe_enable_superblocks(pr, 0);

Thanks!
-Lukas

> 
> Cheers,
> 
> 					- Ted
> 

Powered by blists - more mailing lists