Bart Blanquart wrote:

> 
> Current webrev at http://cr.opensolaris.org/~bartbl/6251549-20090225/.
> 

This is OK, but it would also work fine if you get rid of strdup() and 
the corresponding free() at line 136.

Line 155 seems to have an unused variable: old_userauth_cnt. Doesn't 
lint complain about this?

        Scott


-- 
Scott Rotondo
Principal Engineer, Solaris Security Technologies
President, Trusted Computing Group
Phone/FAX: +1 408 850 3655 (Internal x68278)

Reply via email to