Re: [Merge] ~dirk.zimoch/epics-base:iocLogClientFixes into epics-base:7.0

[mdavidsaver via Core-talk <core-talk@aps.anl.gov>]

Review: Needs Information

What necessitates removing shutdown() and adding select()/recv()?  To my knowledge send() should return with an error if the socket is, or becomes, disconnected.

Diff comments:

> diff --git a/modules/libcom/src/log/logClient.c b/modules/libcom/src/log/logClient.c
> index 99ee671..0056a21 100644
> --- a/modules/libcom/src/log/logClient.c
> +++ b/modules/libcom/src/log/logClient.c
> @@ -507,14 +465,22 @@ logClientId epicsShareAPI logClientCreate (
>      pClient->shutdownConfirm = 0;
>  
>      epicsAtExit (logClientDestroy, (void*) pClient);

I notice that the atexit is too early.  This should be moved to the end, after success is certain.

> -    
> +
>      pClient->stateChangeNotify = epicsEventCreate (epicsEventEmpty);
>      if ( ! pClient->stateChangeNotify ) {
>          epicsMutexDestroy ( pClient->mutex );
>          free ( pClient );
>          return NULL;
>      }
> -   
> +
> +    pClient->shutdownNotify = epicsEventCreate (epicsEventEmpty);
> +    if ( ! pClient->shutdownNotify ) {
> +        epicsMutexDestroy ( pClient->mutex );
> +        epicsEventDestroy ( pClient->stateChangeNotify );
> +        free ( pClient );
> +        return NULL;
> +    }
> +
>      pClient->restartThreadId = epicsThreadCreate (
>          "logRestart", epicsThreadPriorityLow, 
>          epicsThreadGetStackSize(epicsThreadStackSmall),


-- 
https://code.launchpad.net/~dirk.zimoch/epics-base/+git/epics-base/+merge/372043
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.