Bug 461416

Summary:	plymouth ask-for-password --command sends garbage to command
Product:	[Fedora] Fedora	Reporter:	Milan Broz <mbroz>
Component:	plymouth	Assignee:	Ray Strode [halfline] <rstrode>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	jeff, krh, pvrabec
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-10-06 13:48:45 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Milan Broz 2008-09-07 17:43:34 UTC

initcript contain this line (system with encrypted PV)

plymouth ask-for-password --command "cryptsetup luksOpen /dev/sda2 luks-sda2"

Running in text mode, it requires password twice, because first attempt fails (even witout touching keyboard!)

As I found, cryptsetup always receive on standard input 4 chars, no idea where they are from, but these are completely garbage not password.

Just tried to replace crypsetup with simple test program
while (read(0, &c, 1) == 1)
  ...

And I see the same.

Version-Release number of selected component (if applicable):

plymouth-0.6.0-0.2008.08.27.2.fc10.i386
mkinitrd-6.0.62-1.fc10.i386

Plymouth ask-for-password should probably drain the output before asking password again.

Comment 1 Ray Strode [halfline] 2008-09-07 21:07:57 UTC

Are you still seeing this with the latest plymouth?

I did some fixes on friday, including:

* Fri Sep  5 2008 Ray Strode <rstrode> 0.5.0-0.2008.09.05.1
- Fix some confusion with password handling in details plugin


Make sure you rebuild your initrd after updating with

/sbin/mkinitrd -f /boot/initrd-$(uname -r).img $(uname -r)

Comment 2 Milan Broz 2008-09-08 11:57:29 UTC

Seems that it works with 0.6.0-0.2008.09.05.3.f10, but tested on other confuguration.

I'll verify it on my test system later this week.

Comment 3 Ray Strode [halfline] 2008-09-23 18:00:49 UTC

marking MODIFIED waiting verification

Comment 4 Milan Broz 2008-10-06 11:26:44 UTC

Works with F10-beta, thanks.