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)