Re: list of patches for cryptsetup 1.0.5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Jonas, 

First of all thanks for maintaining the list of patches!  Please make
liberal use of your commit privileges to the SVN repo. These are all
reasonable and small fixes, just Cc me and commit if you feel like.

Here are a few comments regarding the patches:

> Desc:   Improve the upstream manpage

Should be merged.

> Desc:   detect read-only devices automatically

In SVN.
 
> Desc:   stop cryptsetup from segfaulting with invalid device

In SVN.

> Desc:   check for UID == 0 before actually doing anything

As others pointed out, I'm not sure if that's really neccessary.

> Desc:   rename luksInit to luksFormat in libcrytsetup.h

In SVN.

> Desc:   run udevsettle after dm device creation

Should be merged.

> Desc:   fix some gcc warnings about unused or unitialized variables

Should be merged.

> Desc:   fix implicit function declarations

Should be merged.

> Desc:   fix segfault when trying to open a non existing device

I think that's fixed in SVN. Have to check.

> Desc:   use set_error instead of printf in library to report errors

Should be merged.

The cryptsetup SVN changes slightly refactoring a bit of stuff. If
some patch fails to apply or something else seems wrong, feel free to ask.

Thanks to all contributors.
-- 
Fruhwirth Clemens - http://clemens.endorphin.org 

---------------------------------------------------------------------
dm-crypt mailing list - http://www.saout.de/misc/dm-crypt/
To unsubscribe, e-mail: dm-crypt-unsubscribe@xxxxxxxx
For additional commands, e-mail: dm-crypt-help@xxxxxxxx


[Index of Archives]     [Device Mapper Devel]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]     [Fedora Docs]

  Powered by Linux