Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 49040
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Tavis Ormandy (RETIRED) <taviso@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Ti Leggett <leggett@mcs.anl.gov>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
sudo-1.6.7_p5-r1.ebuild sudo with IUSE=skey, also cleaned up a little text/plain Eric Brown 2004-04-26 12:54 0000 1.03 KB Details
sudo-1.6.7_p5-r1.ebuild corrected text/plain Eric Brown 2004-04-26 15:31 0000 1.07 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 49040 depends on: Show dependency tree
Bug 49040 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2004-04-26 09:09 0000 
sudo since at least version 1.6.6 has had s/key support for it. Can sudo get a
skey USE flag added that enables this support?

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

------- Comment #1 From Eric Brown 2004-04-26 12:54:12 0000 ------- 
Created an attachment (id=30107) [details]
sudo with IUSE=skey, also cleaned up a little

changed use pam && myconf="--with-pam" || myconf="--without-pam" to
myconf="`use_with pam pam`"

also the skey flag was added

------- Comment #2 From Ti Leggett 2004-04-26 13:25:42 0000 ------- 
One minor addition. Shouldn't:

DEPEND="pam? ( >=sys-libs/pam-0.73-r1 )"

be

DEPEND="pam? ( >=sys-libs/pam-0.73-r1 )
    skey? ( >=app-admin/skey-1.1.5-r1 )"

------- Comment #3 From Eric Brown 2004-04-26 15:31:54 0000 ------- 
Created an attachment (id=30115) [details]
corrected

------- Comment #4 From Ti Leggett 2004-04-26 16:51:44 0000 ------- 
Heh. Missed this in the first pass:

Replace:
	use skey \
		&& myconf=" --with-skey"

with:
	myconf="${myconf} `use_with skey skey`"

Otherwise your --with(out)-pam configure option will be wiped.

------- Comment #5 From Tavis Ormandy (RETIRED) 2004-09-17 12:45:41 0000 ------- 
sudo wont support pam with other authorisation methods, even though it decides
to link with -lskey.

you can see in sudo_auth.c, if pam is enabled, everything else is ignored.

So I'll add support for it and add an einfo that says pam + skey is bad, I've
tried it out and it works okay here.

Thanks for the suggestion guys.
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug