Util-Linux package development
 help / color / mirror / Atom feed
From: Petr Uzel <puzel@suse.cz>
To: util-linux <util-linux@vger.kernel.org>
Subject: Re: [PATCH] libblkid: make blkid_do_wipe() work with probes with offset
Date: Tue, 19 Apr 2016 11:05:02 +0200	[thread overview]
Message-ID: <20160419090502.GA20078@skipper.site> (raw)
In-Reply-To: <20160418180018.wgjfgaule76mcfyb@ws.net.home>

[-- Attachment #1: Type: text/plain, Size: 983 bytes --]

On Mon, Apr 18, 2016 at 08:00:18PM +0200, Karel Zak wrote:
> On Mon, Apr 18, 2016 at 04:22:05PM +0200, Petr Uzel wrote:
> > When a probe is created with an offset, e.g. via
> > blkid_probe_set_device(), this offset is correctly used when looking for
> > the signatures, but is not respected by blkid_do_wipe() function.
> > Therefore the signature is removed from an invalid location.
> 
> Wow, excellent catch. 

Thanks, feels good to again contribute a fix after such a long time :)

> How did you found it? Do you use libblkid in some partitioning tool or
> installer?

Yes, I'm trying to implement a --wipesignatures for parted - this
would wipe the signatures from a new partition, _before_ the kernel
is informed. This is to avoid mess like RAID autoassembly...

[If only there was some sane mechanism lock a device to temporarily
prevent udev from touching it :/]

> 
> Applied, thanks!

Thanks,

        Petr


-- 
Petr Uzel
TL SUSE L3 Team 2

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2016-04-19  9:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-18 14:22 [PATCH] libblkid: make blkid_do_wipe() work with probes with offset Petr Uzel
2016-04-18 18:00 ` Karel Zak
2016-04-19  9:05   ` Petr Uzel [this message]
2016-04-19  9:18     ` Karel Zak
2016-04-22 14:38       ` Petr Uzel
2016-05-13 13:50       ` Karel Zak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160419090502.GA20078@skipper.site \
    --to=puzel@suse.cz \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox