Bug 409

Summary: visudo -s with unused alias causes segfault
Product: Sudo Reporter: David.Wood
Component: VisudoAssignee: Todd C. Miller <Todd.Miller>
Status: RESOLVED FIXED    
Severity: normal    
Priority: low    
Version: 1.7.2   
Hardware: All   
OS: All   
Attachments: visudo.c patch

Description David.Wood 2010-04-29 04:01:20 MDT 
Created attachment 270 [details]
visudo.c patch

% cat /tmp/sudoers
User_Alias    FOO=joeuser

% visudo -scf /tmp/sudoers
visudo: Error: unused User_Alias FOO
zsh: segmentation fault (core dumped)  visudo -scf /tmp/sudoers

After applying the attached patch:

% ./visudo -scf /tmp/sudoers
visudo: Error: unused User_Alias FOO
/tmp/sudoers: parse error
/tmp/sudoers: wrong owner (uid, gid) should be (0, 0)
/tmp/sudoers: bad permissions, should be mode 0440

% echo $?
1
Comment 1 Todd C. Miller 2010-04-29 18:25:26 MDT 
This is already fixed in the sudo mercurial repo and will be part of the next sudo release.