Skip to content

Commit

Permalink
client: fix mixed up CID/SID in CMD_DESTROY_CHANNEL handler
Browse files Browse the repository at this point in the history
servers send SID then CID
  • Loading branch information
mdavidsaver committed Apr 7, 2020
1 parent 19d868e commit ef4bf91
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/remoteClient/clientContextImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2929,8 +2929,9 @@ class DestroyChannelHandler : public AbstractClientResponseHandler {
AbstractClientResponseHandler::handleResponse(responseFrom, transport, version, command, payloadSize, payloadBuffer);

transport->ensureData(8);
pvAccessID sid = payloadBuffer->getInt();
pvAccessID cid = payloadBuffer->getInt();
/*pvAccessID sid =*/ payloadBuffer->getInt();
(void)sid;

// TODO optimize
ClientChannelImpl::shared_pointer channel = static_pointer_cast<ClientChannelImpl>(_context.lock()->getChannel(cid));
Expand Down

0 comments on commit ef4bf91

Please sign in to comment.