On Wed, Oct 22, 2008 at 4:23 AM, Vlad Seryakov <[EMAIL PROTECTED]> wrote: > Update of /cvsroot/naviserver/modules/nsssl > In directory fdv4jf1.ch3.sourceforge.com:/tmp/cvs-serv22913/modules/nsssl > > Modified Files: > nsssl.c > Log Message: > new accept status, SSL driver works now > > > Index: nsssl.c > =================================================================== > *** 286,298 **** > > if (sslPtr == NULL) { > ! sslPtr = SSL_new(drvPtr->ctx); > if (sslPtr == NULL) { > Ns_Log(Error, "%d: SSL session init error for %s: [%s]", > sock->sock, ns_inet_ntoa(sock->sa.sin_addr), strerror(errno)); > return NS_DRIVER_ACCEPT_ERROR; > } > sock->arg = sslPtr; > ! SSL_set_fd(sslPtr, sock->sock); > ! SSL_set_accept_state(sslPtr); > ! > } > return NS_DRIVER_ACCEPT_DATA; > --- 293,306 ---- > > if (sslPtr == NULL) { > ! sslPtr = ns_calloc(1, sizeof(SSLContext)); > ! sslPtr->ssl = SSL_new(drvPtr->ctx); > if (sslPtr == NULL) { > Ns_Log(Error, "%d: SSL session init error for %s: [%s]", > sock->sock, ns_inet_ntoa(sock->sa.sin_addr), strerror(errno)); > + ns_free(sslPtr); > return NS_DRIVER_ACCEPT_ERROR; > } > sock->arg = sslPtr; > ! SSL_set_fd(sslPtr->ssl, sock->sock); > ! SSL_set_accept_state(sslPtr->ssl); > } > return NS_DRIVER_ACCEPT_DATA; > ***************
The check for (sslPtr == NULL) happens after it's already been used. ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ naviserver-devel mailing list naviserver-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/naviserver-devel