Remove ASYNC_in_job()
authorMatt Caswell <matt@openssl.org>
Wed, 23 Sep 2015 22:12:36 +0000 (23:12 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 20 Nov 2015 23:34:34 +0000 (23:34 +0000)
The ASYNC_in_job() function is redundant. The same effect can be achieved by
using ASYNC_get_current_job().

Reviewed-by: Rich Salz <rsalz@openssl.org>
crypto/async/async.c
include/openssl/async.h
ssl/ssl_lib.c

index ed3ed73..7a4fbf5 100644 (file)
@@ -287,14 +287,6 @@ int ASYNC_pause_job(void)
     return 1;
 }
 
-int ASYNC_in_job(void)
-{
-    if(ASYNC_get_ctx())
-        return 1;
-
-    return 0;
-}
-
 static void async_empty_pool(STACK_OF(ASYNC_JOB) *pool)
 {
     ASYNC_JOB *job;
index 5b84d5b..e123261 100644 (file)
@@ -73,7 +73,6 @@ void ASYNC_free_pool(void);
 int ASYNC_start_job(ASYNC_JOB **job, int *ret, int (*func)(void *),
                          void *args, size_t size);
 int ASYNC_pause_job(void);
-int ASYNC_in_job(void);
 
 int ASYNC_get_wait_fd(ASYNC_JOB *job);
 ASYNC_JOB *ASYNC_get_current_job(void);
index 270a908..55b64bb 100644 (file)
@@ -961,7 +961,7 @@ int SSL_accept(SSL *s)
 
     args.s = s;
 
-    if((s->mode & SSL_MODE_ASYNC) && !ASYNC_in_job()) {
+    if((s->mode & SSL_MODE_ASYNC) && ASYNC_get_current_job() == NULL) {
         switch(ASYNC_start_job(&s->job, &ret, ssl_accept_intern, &args,
             sizeof(struct ssl_async_args))) {
         case ASYNC_ERR:
@@ -1031,7 +1031,7 @@ int SSL_read(SSL *s, void *buf, int num)
     args.buf = buf;
     args.num = num;
 
-    if((s->mode & SSL_MODE_ASYNC) && !ASYNC_in_job()) {
+    if((s->mode & SSL_MODE_ASYNC) && ASYNC_get_current_job() == NULL) {
         switch(ASYNC_start_job(&s->job, &ret, ssl_read_intern, &args,
             sizeof(struct ssl_async_args))) {
         case ASYNC_ERR:
@@ -1104,7 +1104,7 @@ int SSL_write(SSL *s, const void *buf, int num)
     args.buf = (void *) buf;
     args.num = num;
 
-    if((s->mode & SSL_MODE_ASYNC) && !ASYNC_in_job()) {
+    if((s->mode & SSL_MODE_ASYNC) && ASYNC_get_current_job() == NULL) {
         switch(ASYNC_start_job(&s->job, &ret, ssl_write_intern, &args,
             sizeof(struct ssl_async_args))) {
         case ASYNC_ERR: