[asterisk-commits] jrose: trunk r346563 - in /trunk: channels/ include/asterisk/ main/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Wed Nov 30 15:32:27 CST 2011
Author: jrose
Date: Wed Nov 30 15:32:23 2011
New Revision: 346563
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=346563
Log:
Reverting 346525 due to accidental patch against trunk instead of 1.8
Modified:
trunk/channels/chan_sip.c
trunk/include/asterisk/tcptls.h
trunk/main/tcptls.c
Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=346563&r1=346562&r2=346563
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Wed Nov 30 15:32:23 2011
@@ -2813,7 +2813,14 @@
if (tcptls_session) {
ast_mutex_lock(&tcptls_session->lock);
- ast_tcptls_close_session_file(tcptls_session);
+ if (tcptls_session->f) {
+ fclose(tcptls_session->f);
+ tcptls_session->f = NULL;
+ }
+ if (tcptls_session->fd != -1) {
+ close(tcptls_session->fd);
+ tcptls_session->fd = -1;
+ }
tcptls_session->parent = NULL;
ast_mutex_unlock(&tcptls_session->lock);
@@ -26012,8 +26019,8 @@
ao2_t_ref(ca, -1, "failed to create client, getting rid of client tcptls_session arguments");
}
if (s->tcptls_session) {
- ast_tcptls_close_session_file(tcptls_session);
- s->fd = -1;
+ close(tcptls_session->fd);
+ s->fd = tcptls_session->fd = -1;
ao2_ref(s->tcptls_session, -1);
s->tcptls_session = NULL;
}
Modified: trunk/include/asterisk/tcptls.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/tcptls.h?view=diff&rev=346563&r1=346562&r2=346563
==============================================================================
--- trunk/include/asterisk/tcptls.h (original)
+++ trunk/include/asterisk/tcptls.h Wed Nov 30 15:32:23 2011
@@ -177,13 +177,6 @@
void *ast_tcptls_server_root(void *);
/*!
- * \brief Closes a tcptls session instance's file and/or file descriptor.
- * The tcptls_session will be set to NULL and it's file descriptor will be set to -1
- * by this function.
- */
-void ast_tcptls_close_session_file(struct ast_tcptls_session_instance *tcptls_session);
-
-/*!
* \brief This is a generic (re)start routine for a TCP server,
* which does the socket/bind/listen and starts a thread for handling
* accept().
Modified: trunk/main/tcptls.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/tcptls.c?view=diff&rev=346563&r1=346562&r2=346563
==============================================================================
--- trunk/main/tcptls.c (original)
+++ trunk/main/tcptls.c Wed Nov 30 15:32:23 2011
@@ -78,23 +78,9 @@
static int ssl_close(void *cookie)
{
- int cookie_fd = SSL_get_fd(cookie);
- int ret;
- if (cookie_fd > -1) {
- /*
- * According to the TLS standard, it is acceptable for an application to only send its shutdown
- * alert and then close the underlying connection without waiting for the peer's response (this
- * way resources can be saved, as the process can already terminate or serve another connection).
- */
- if ((ret = SSL_shutdown(cookie)) < 0) {
- ast_log(LOG_ERROR, "SSL_shutdown() failed: %d\n", SSL_get_error(cookie, ret));
- }
- SSL_free(cookie);
- /* adding shutdown(2) here has no added benefit */
- if (close(cookie_fd)) {
- ast_log(LOG_ERROR, "close() failed: %s\n", strerror(errno));
- }
- }
+ close(SSL_get_fd(cookie));
+ SSL_shutdown(cookie);
+ SSL_free(cookie);
return 0;
}
#endif /* DO_SSL */
@@ -159,7 +145,8 @@
if (!tcptls_session->parent->tls_cfg) {
if ((tcptls_session->f = fdopen(tcptls_session->fd, "w+"))) {
if(setvbuf(tcptls_session->f, NULL, _IONBF, 0)) {
- ast_tcptls_close_session_file(tcptls_session);
+ fclose(tcptls_session->f);
+ tcptls_session->f = NULL;
}
}
}
@@ -225,7 +212,8 @@
if (peer) {
X509_free(peer);
}
- ast_tcptls_close_session_file(tcptls_session);
+ close(tcptls_session->fd);
+ fclose(tcptls_session->f);
ao2_ref(tcptls_session, -1);
return NULL;
}
@@ -242,7 +230,7 @@
#endif /* DO_SSL */
if (!tcptls_session->f) {
- ast_tcptls_close_session_file(tcptls_session);
+ close(tcptls_session->fd);
ast_log(LOG_WARNING, "FILE * open failed!\n");
#ifndef DO_SSL
if (tcptls_session->parent->tls_cfg) {
@@ -288,9 +276,7 @@
tcptls_session = ao2_alloc(sizeof(*tcptls_session), session_instance_destructor);
if (!tcptls_session) {
ast_log(LOG_WARNING, "No memory for new session: %s\n", strerror(errno));
- if (close(fd)) {
- ast_log(LOG_ERROR, "close() failed: %s\n", strerror(errno));
- }
+ close(fd);
continue;
}
@@ -307,7 +293,7 @@
/* This thread is now the only place that controls the single ref to tcptls_session */
if (ast_pthread_create_detached_background(&launched, NULL, handle_tcptls_connection, tcptls_session)) {
ast_log(LOG_WARNING, "Unable to launch helper thread: %s\n", strerror(errno));
- ast_tcptls_close_session_file(tcptls_session);
+ close(tcptls_session->fd);
ao2_ref(tcptls_session, -1);
}
}
@@ -570,24 +556,6 @@
error:
close(desc->accept_fd);
desc->accept_fd = -1;
-}
-
-void ast_tcptls_close_session_file(struct ast_tcptls_session_instance *tcptls_session)
-{
- if (tcptls_session->f) {
- if (fclose(tcptls_session->f)) {
- ast_log(LOG_ERROR, "fclose() failed: %s\n", strerror(errno));
- }
- tcptls_session->f = NULL;
- tcptls_session->fd = -1;
- } else if (tcptls_session->fd != -1) {
- if (close(tcptls_session->fd)) {
- ast_log(LOG_ERROR, "close() failed: %s\n", strerror(errno));
- }
- tcptls_session->fd = -1;
- } else {
- ast_log(LOG_ERROR, "ast_tcptls_close_session_file invoked on session instance without file or file descriptor\n");
- }
}
void ast_tcptls_server_stop(struct ast_tcptls_session_args *desc)
More information about the asterisk-commits
mailing list