Commit d6553780 authored by Scott Vokes's avatar Scott Vokes
Browse files

Store a pointer to the session for unlocking, since *op may be freed on error.

parent 02110007

src/lib/kinetic_operation.c

+3 −2
Original line number Diff line number Diff line
@@ -54,13 +54,14 @@ static KineticStatus send_request_in_lock(KineticOperation* const op); 


KineticStatus KineticOperation_SendRequest(KineticOperation* const op)

{

    KineticSession *session = op->session;

    KineticOperation_ValidateOperation(op);

    

    if (!KineticRequest_LockSend(op->session)) {

    if (!KineticRequest_LockSend(session)) {

        return KINETIC_STATUS_CONNECTION_ERROR;

    }

    KineticStatus status = send_request_in_lock(op);

    KineticRequest_UnlockSend(op->session);

    KineticRequest_UnlockSend(session);

    return status;

}