[pkg-cryptsetup-devel] Bug#507721: Bug#507721: cryptsetup: Sometimes initrd ends up missing conf/conf.d/cryptroot file in it

Jonas Meurer jonas at freesources.org
Wed Dec 17 22:33:03 UTC 2008


On 17/12/2008 Christian Jaeger wrote:
> Jonas Meurer wrote:
> > if [ "$(dmsetup table $depnode 2> /dev/null | cut -d' ' -f3)" != "crypt" ]; then
> > 	get_lvm_deps "$depnode"
> >   
> 
> It seems you have missed that in the first line above, $depnode is *not*
> quoted. So going these extra steps to be safe and quote the variable in
> the second line is pointless. This is what I did mean when I said
> ~"since it is used unquoted anyway above".
> 
> Correct would be:
> 
> if [ "$(dmsetup table "$depnode" 2> /dev/null | cut -d' ' -f3)" != "crypt" ]; then

You're right, but i'm not sure about the quotes inside quotes. Is that a
clean solution either?

greetings,
 jonas





More information about the pkg-cryptsetup-devel mailing list