Bug 189 - Errors in auth/securid5.c
Errors in auth/securid5.c
Status: RESOLVED FIXED
Product: Sudo
Classification: Unclassified
Component: Sudo
1.6.8
Sun Solaris 2.x
: normal high
Assigned To: Todd C. Miller
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-07-27 12:55 MDT by Charles Conn
Modified: 2005-07-27 23:17 MDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Charles Conn 2005-07-27 12:55:14 MDT
securid5.c contains a case statement that sets rval for various SecurID 
conditions (ie: user entered a bad passcode, user is in new pin mode etc).  
This case statement never sets rval upon SUCCESS, which makes the function 
return an undeclared int rval every time the user enters a correct passcode.

After line 172 in securid5.c you should add:

        case ACM_OK:
                rval = ACM_OK;
                break;


There are also a number of things a user must know in order to get sudo 
compiled in a SecurID 5.* environment that are not in the documentation (they 
refer to outdated 4.* era .h and .a files).  Let me know if you want a 
description of that as well.
Comment 1 Todd C. Miller 2005-07-27 19:17:43 MDT
Fixed, thanks.  As you've no doubt surmised I don't use securid myself.  I'm 
definately interested in setup info for compiling sudo with securid5.