oknet commented on a change in pull request #6910:
URL: https://github.com/apache/trafficserver/pull/6910#discussion_r447065299



##########
File path: contrib/openssl/async_engine.c
##########
@@ -176,13 +186,16 @@ static void async_pause_job(void) {
     OSSL_ASYNC_FD *writefd;
     char buf = DUMMY_CHAR;
 
-    if ((job = ASYNC_get_current_job()) == NULL)
+    if ((job = ASYNC_get_current_job()) == NULL) {
+        printf("No job\n");
         return;
+    }
 
     waitctx = ASYNC_get_wait_ctx(job);
 
     if (ASYNC_WAIT_CTX_get_fd(waitctx, engine_id, &pipefds[0],

Review comment:
       If we got an existed `fd` from `waitctx`, it means there is an async job 
still in progress. It should return immediately.
   
   > An ASYNC_WAIT_CTX should only be used for one ASYNC_JOB at any one time, 
but can be reused after an ASYNC_JOB has finished for a subsequent ASYNC_JOB.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to