Bug#768577: Patch applied upstream

2015-02-13 Thread Martin Pitt
Control: tag -1 fixed 218-1

Quentin Lefebvre [2014-11-24 15:35 +0100]:
 For your information, a patch has been applied upstream.
 Here is the link:
 http://cgit.freedesktop.org/systemd/systemd/commit/?id=8a52210c93

This was part of 218, thus is fixed in experimental.

I realize that this might not be sufficient to make all current cases
work, but as far as I understand it that's about as far as upstream
wants to go with bug for bug compatibility?

Thanks,

Martin
-- 
Martin Pitt| http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#768577: Patch applied upstream

2014-11-24 Thread Quentin Lefebvre

Hi,

For your information, a patch has been applied upstream.
Here is the link:
http://cgit.freedesktop.org/systemd/systemd/commit/?id=8a52210c93

Cheers,
Quentin


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#768577: Patch applied upstream

2014-11-24 Thread intrigeri
Hi Quentin,

Quentin Lefebvre wrote (24 Nov 2014 14:35:45 GMT) :
 For your information, a patch has been applied upstream.
 Here is the link:
 http://cgit.freedesktop.org/systemd/systemd/commit/?id=8a52210c93

Congrats!

Can you please try to apply the upstream patch on top of Debian
unstable's systemd, and confirm that it works and fixes the issue
for you?

Cheers,
-- 
intrigeri


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#768577: Patch applied upstream

2014-11-24 Thread Quentin Lefebvre

Hi,

On 24/11/2014 16:37, intrigeri wrote :

Quentin Lefebvre wrote (24 Nov 2014 14:35:45 GMT) :

For your information, a patch has been applied upstream.
Here is the link:
http://cgit.freedesktop.org/systemd/systemd/commit/?id=8a52210c93


Congrats!

Can you please try to apply the upstream patch on top of Debian
unstable's systemd, and confirm that it works and fixes the issue
for you?


Thanks for making me test the new patch. Actually it is a rewrite of the 
one I first proposed, and it doesn't work.

I hope the developers will agree on my original patch.

So... waiting!

Cheers,
Quentin


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#768577: Patch applied upstream

2014-11-24 Thread Quentin Lefebvre

So here is the point of view of the developers.
The upstream patch works provided that hash=plain is mentioned in 
/etc/cryptab.


To summarize:
- when a user creates a plain dm-crypt device providing a --hash 
parameter along with a key file

- he *should* be aware of the fact that the hash parameter is ignored
- and as a consequence, he should write hash=plain in /etc/crypttab
- in short, it's a cryptsetup bug, and systemd won't be compatible with 
cryptsetup's bug...


Let's say that's fine. It may be worth documenting this.

Please note that this patch basically changes nothing about the 
aforementioned bug, so I'm not convinced it should be applied in Debian, 
and I don't attach it.
The trick with hash=plain already works with Debian's current version 
of systemd.


Best regards,
Quentin


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#768577: Patch applied upstream

2014-11-24 Thread intrigeri
Hi,

Quentin Lefebvre wrote (24 Nov 2014 21:19:56 GMT) :
 So here is the point of view of the developers.
 The upstream patch works provided that hash=plain is mentioned in 
 /etc/cryptab.

 To summarize:
 - when a user creates a plain dm-crypt device providing a --hash parameter 
 along with
 a key file
 - he *should* be aware of the fact that the hash parameter is ignored
 - and as a consequence, he should write hash=plain in /etc/crypttab
 - in short, it's a cryptsetup bug, and systemd won't be compatible with 
 cryptsetup's
 bug...

Well, cryptsetup(8) makes it quite clear that hash processing is only
used on *passphrases*. See the NOTES ON PASSPHRASE PROCESSING FOR
PLAIN MODE section. So, IMO that's not a bug in cryptsetup, but
rather the intended and documented way it works. Maybe it's worth
pointing systemd upstream at that piece of documentation.

Cheers,
-- 
intrigeri


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org