muthusuba / serf

Automatically exported from code.google.com/p/serf
Apache License 2.0
0 stars 0 forks source link

CredHandle not freed when APR pool is destroyed #153

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In auth_spnego_sspi.c from line 97

 if (SecIsValidHandle(&ctx->sspi_credentials)) {
        FreeCredentialsHandle(&ctx->sspi_context);
        SecInvalidateHandle(&ctx->sspi_context);
    }

I guess this should read

 if (SecIsValidHandle(&ctx->sspi_credentials)) {
        FreeCredentialsHandle(&ctx->sspi_credentials);
        SecInvalidateHandle(&ctx->sspi_credentials);
    }

Shouldn't it?

Original issue reported on code.google.com by 1983-01...@gmx.net on 20 Aug 2014 at 5:13

GoogleCodeExporter commented 9 years ago
Good catch. Fixed in r2423.

Thanks a lot!

Original comment by chemodax@gmail.com on 1 Sep 2014 at 1:34