Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-RT-Original-Encoding: iso-8859-1 Content-Length: 603 Tom Yu via RT wrote: > [Line numbers refer to file after application of Jeff's patch.] > > The extra call to free_cred_contents() at line 164 will never be > executed, as there is no way for free_tgt to be 1 at that point. > Otherwise, it appears to be functionally nearly identical to my > patch. Jeff, were there any cases your patch covers which mine does > not? I didn't think there were, but I wanted to make sure I wasn't > missing something. The new patch does not cover any new cases but it is less likely to be susceptible to similar bugs in the future if changes are made. Jeffrey Altman