Skip to content

Commit 0d097fa

Browse files
committed
Merge branch 'RTSDK-9816' into release
2 parents 708680f + e9a858d commit 0d097fa

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

Cpp-C/Eta/Impl/Reactor/rsslReactor.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6156,7 +6156,7 @@ static RsslRet _reactorDispatchEventFromQueue(RsslReactorImpl *pReactorImpl, Rss
61566156
pReactorTokenMgntEvent->reactorAuthTokenEvent.statusCode = pTokenSession->httpStatusCode;
61576157

61586158
// The current channel could be a WSB connection that has transitioned to the channel list, so set up the main channel here.
6159-
if (pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
6159+
if (pReactorChannel && pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
61606160
{
61616161
pCallbackChannel = &pReactorChannel->pWarmStandByHandlerImpl->mainReactorChannelImpl.reactorChannel;
61626162
_reactorSetupMainWSBReactorChannel(pCallbackChannel, pReactorChannel);
@@ -6197,7 +6197,7 @@ static RsslRet _reactorDispatchEventFromQueue(RsslReactorImpl *pReactorImpl, Rss
61976197
pReactorTokenMgntEvent->reactorAuthTokenEvent.statusCode = pReactorChannel ? pTokenSession->httpStatusCode : pReactorOAuthCredentialRenewalImpl->httpStatusCode;
61986198

61996199
// The current channel could be a WSB connection that has transitioned to the channel list, so set up the main channel here.
6200-
if (pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
6200+
if (pReactorChannel && pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
62016201
{
62026202
pCallbackChannel = &pReactorChannel->pWarmStandByHandlerImpl->mainReactorChannelImpl.reactorChannel;
62036203
_reactorSetupMainWSBReactorChannel(pCallbackChannel, pReactorChannel);
@@ -6274,7 +6274,7 @@ static RsslRet _reactorDispatchEventFromQueue(RsslReactorImpl *pReactorImpl, Rss
62746274
/* Send login message only when the watchlist is enable.
62756275
Note: Unlike above, this should never be sent in the preferred host case.
62766276
*/
6277-
if (pReactorChannel->pWatchlist && pReactorChannel->channelRole.ommConsumerRole.pLoginRequest)
6277+
if (pReactorChannel && pReactorChannel->pWatchlist && pReactorChannel->channelRole.ommConsumerRole.pLoginRequest)
62786278
{
62796279
rsslWatchlistClearProcessMsgOptions(&processOpts);
62806280
processOpts.pRdmMsg = (RsslRDMMsg*)pReactorChannel->channelRole.ommConsumerRole.pLoginRequest;
@@ -6303,7 +6303,7 @@ static RsslRet _reactorDispatchEventFromQueue(RsslReactorImpl *pReactorImpl, Rss
63036303
pReactorTokenMgntEvent->reactorAuthTokenEvent.statusCode = pReactorOAuthCredentialRenewalImpl->httpStatusCode;
63046304

63056305
// The current channel could be a WSB connection that has transitioned to the channel list, so set up the main channel here.
6306-
if (pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
6306+
if (pReactorChannel && pReactorChannel->reactorChannel.pWarmStandbyChInfo != NULL)
63076307
{
63086308
pCallbackChannel = &pReactorChannel->pWarmStandByHandlerImpl->mainReactorChannelImpl.reactorChannel;
63096309
_reactorSetupMainWSBReactorChannel(pCallbackChannel, pReactorChannel);

Cpp-C/Eta/TestTools/UnitTests/rsslVATest/reactorUnitTests.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2242,7 +2242,7 @@ TEST_F(ReactorSessionMgntTest, ConnectionRecoveryFromSocketToEncrypted_Unauthori
22422242
/* Wait until the connection is switched successfully to the encrypted connection */
22432243
ASSERT_TRUE(dispatchEvent(pConsMon, 10000) >= RSSL_RET_SUCCESS);
22442244
time_sleep(2000);
2245-
ASSERT_TRUE(dispatchEvent(pConsMon, 3000) >= RSSL_RET_SUCCESS);
2245+
dispatchEvent(pConsMon, 3000);
22462246

22472247
/* Check for token information */
22482248
ASSERT_TRUE(pConsMon->authEventStatusCode == 400);
@@ -2382,7 +2382,7 @@ TEST_P(ReactorSessionMgntTest, ConnectionRecoveryFromSocketToEncrypted_InvalidAu
23822382
/* Wait until the connection is switched successfully to the encrypted connection */
23832383
ASSERT_TRUE(dispatchEvent(pConsMon, 8000) >= RSSL_RET_SUCCESS);
23842384
time_sleep(2000);
2385-
ASSERT_TRUE(dispatchEvent(pConsMon, 3000) >= RSSL_RET_SUCCESS);
2385+
dispatchEvent(pConsMon, 3000);
23862386

23872387
/* Check for token information */
23882388
ASSERT_TRUE(pConsMon->authEventStatusCode == 0); /* No HTTP response status code as the URL is invalid*/

0 commit comments

Comments
 (0)