Bug 486

Summary:	redundant code in common/aix.c::getlimits()
Product:	Sudo	Reporter:	dlaigle
Component:	Sudo	Assignee:	Todd C. Miller <Todd.Miller>
Status:	RESOLVED FIXED
Severity:	low
Priority:	low
Version:	1.8.1
Hardware:	IBM
OS:	AIX

Description dlaigle 2011-03-30 02:59:36 MDT

The 'getuserattr()' call will always end up by seeking in the "default" stanza on local files, regardless of the user database (I used LDAP).
Thus, if an attribute in not defined for a user but exists in "default", 'getuserattr(user)' will return the default value.
You don't need to explicitly call 'getuserattr("default")' and therefore you can remove this statement from the code...

Regards

- Dominique

Comment 1 Todd C. Miller 2011-03-30 08:59:52 MDT

Thanks, I'll remove that line in the next release.

Comment 2 Todd C. Miller 2011-04-11 08:02:13 MDT

Fixed in sudo 1.8.1