axis-c-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From billblo...@apache.org
Subject [axis-axis2-c-core] 19/23: ssl issue fixed
Date Sat, 04 Apr 2020 15:07:27 GMT
This is an automated email from the ASF dual-hosted git repository.

billblough pushed a commit to branch 1.7.x
in repository https://gitbox.apache.org/repos/asf/axis-axis2-c-core.git

commit 9ba93f2ecd50c7e6ea5d2539d86311383247d107
Author: Korale Gamaralalage Nandika Chandrasiri Jayawardana <nandika@apache.org>
AuthorDate: Tue Sep 7 07:50:09 2010 +0000

    ssl issue fixed
---
 1.7.0/c/src/core/transport/http/sender/http_client.c    | 10 +++++++++-
 1.7.0/c/src/core/transport/http/sender/http_sender.c    | 14 ++++++++++++--
 1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.c |  8 ++------
 1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.h |  5 +++++
 1.7.0/c/src/core/transport/http/sender/ssl/ssl_utils.c  |  5 ++++-
 5 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/1.7.0/c/src/core/transport/http/sender/http_client.c b/1.7.0/c/src/core/transport/http/sender/http_client.c
index 3a8deca..5d16e5c 100644
--- a/1.7.0/c/src/core/transport/http/sender/http_client.c
+++ b/1.7.0/c/src/core/transport/http/sender/http_client.c
@@ -120,7 +120,15 @@ axis2_http_client_free(
     }
     if(-1 != http_client->sockfd)
     {
-        axutil_network_handler_close_socket(env, http_client->sockfd);
+#ifdef AXIS2_SSL_ENABLED
+		if(http_client->data_stream->stream_type == AXIS2_STREAM_SOCKET)
+		{
+			axutil_network_handler_close_socket(env, http_client->sockfd);
+			/** ssl streams of type AXIS2_STREAM_BASIC  will be handled by SSL_shutdown(); */
+		}
+#else
+		axutil_network_handler_close_socket(env, http_client->sockfd);
+#endif
         http_client->sockfd = -1;
     }
 
diff --git a/1.7.0/c/src/core/transport/http/sender/http_sender.c b/1.7.0/c/src/core/transport/http/sender/http_sender.c
index 093cfec..f899a9e 100644
--- a/1.7.0/c/src/core/transport/http/sender/http_sender.c
+++ b/1.7.0/c/src/core/transport/http/sender/http_sender.c
@@ -14,7 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
+#include <axutil_stream.h>
 #include <axis2_http_sender.h>
 #include <axutil_string.h>
 #include <axis2_http_transport.h>
@@ -36,6 +36,7 @@
 #include <axis2_util.h>
 #include <axiom_soap.h>
 #include <axutil_version.h>
+#include "ssl/ssl_stream.h"
 
 #ifdef AXIS2_LIBCURL_ENABLED
 #include "libcurl/axis2_libcurl.h"
@@ -43,6 +44,8 @@
 #define CLIENT_NONCE_LENGTH 8
 #endif
 
+
+
 struct axis2_http_sender
 {
     axis2_char_t *http_version;
@@ -1594,7 +1597,14 @@ axis2_http_sender_process_response(
         response, env));
     property = axutil_property_create(env);
     axutil_property_set_scope(property, env, AXIS2_SCOPE_REQUEST);
-    axutil_property_set_free_func(property, env, axutil_stream_free_void_arg);
+#ifdef AXIS2_SSL_ENABLED
+	if(in_stream->stream_type == AXIS2_STREAM_SOCKET)
+		axutil_property_set_free_func(property, env, axutil_stream_free_void_arg);
+	else /** SSL Streams are AXIS2_STREAM_BASIC */
+		axutil_property_set_free_func(property, env, axis2_ssl_stream_free);
+#else
+   axutil_property_set_free_func(property, env, axutil_stream_free_void_arg);
+#endif
     axutil_property_set_value(property, env, in_stream);
     axis2_msg_ctx_set_property(msg_ctx, env, AXIS2_TRANSPORT_IN, property);
     AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, "Exit:axis2_http_sender_process_response");
diff --git a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.c b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.c
index 9360f1a..0900487 100644
--- a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.c
+++ b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.c
@@ -38,10 +38,6 @@ struct ssl_stream_impl
 
 #define AXIS2_INTF_TO_IMPL(stream) ((ssl_stream_impl_t *)(stream))
 
-void AXIS2_CALL axis2_ssl_stream_free(
-    axutil_stream_t * stream,
-    const axutil_env_t * env);
-
 axutil_stream_type_t AXIS2_CALL axis2_ssl_stream_get_type(
     axutil_stream_t * stream,
     const axutil_env_t * env);
@@ -118,11 +114,11 @@ axutil_stream_create_ssl(
 
 void AXIS2_CALL
 axis2_ssl_stream_free(
-    axutil_stream_t * stream,
+    void * streamv,
     const axutil_env_t * env)
 {
     ssl_stream_impl_t *stream_impl = NULL;
-
+    axutil_stream_t* stream = (axutil_stream_t*)streamv;
     stream_impl = AXIS2_INTF_TO_IMPL(stream);
     axis2_ssl_utils_cleanup_ssl(env, stream_impl->ctx, stream_impl->ssl);
     AXIS2_FREE(env->allocator, stream_impl);
diff --git a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.h b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.h
index 396da5d..7fe9a46 100644
--- a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.h
+++ b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_stream.h
@@ -41,6 +41,11 @@ extern "C"
         axis2_char_t * key_file,
         axis2_char_t * ssl_pp);
 
+	void AXIS2_CALL
+	axis2_ssl_stream_free(
+		void *stream, 
+		const axutil_env_t *env);
+
     /** @} */
 
 #ifdef __cplusplus
diff --git a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_utils.c b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_utils.c
index 01fe993..66cb9e7 100644
--- a/1.7.0/c/src/core/transport/http/sender/ssl/ssl_utils.c
+++ b/1.7.0/c/src/core/transport/http/sender/ssl/ssl_utils.c
@@ -215,7 +215,10 @@ axis2_ssl_utils_cleanup_ssl(
 
     if (ssl)
     {
-        SSL_shutdown(ssl);
+        if(SSL_shutdown(ssl)==0)
+		{
+			SSL_free(ssl);
+		}
     }
     if (ctx)
     {


Mime
View raw message