[dsdp-tcf-dev] [Bug 321063] New: [tcf][agent] The script functionality o

[bugzilla-daemon@xxxxxxxxxxx]

https://bugs.eclipse.org/bugs/show_bug.cgi?id=321063 
Product/Component: Target Management / TCF

           Summary: [tcf][agent] The script functionality of the C client
                    doesn't work
    Classification: DSDP
           Product: Target Management
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P3
         Component: TCF
        AssignedTo: dsdp.tm.tcf-inbox@xxxxxxxxxxx
        ReportedBy: michael.sills.lavoie@xxxxxxxxx
         QAContact: martin.oberhuber@xxxxxxxxxxxxx


Build Identifier: rev 1079

Since revision 900, it's impossible to use the script functionality with the C
client.

The client report the channel as closed even if it's not : Reply error 131077:
Channel closed.

This is due to the addition of an if statement in protocol.c at line 429 (last
revision): 

if (c->peer_service_list == NULL) {
    post_event(send_command_failed, rh);
}

This statement checks if the remote agent has any services. Since the "hello"
message is not received at the time of sending the command, the client report
the connection as closed. Removing this check resolve the problem.

Removing the check is probably not the best option. Adding a sleep after a
connect command in the client is not either but I am not sure how you want to
solve this.

Reproducible: Always

Steps to Reproduce:
1. Start an agent with at least one service that you will use to send a
command.
2. Create a script file with a connect statement followed by any command (see
attachment for an example).
3. Start the client with the script you just created : client -S test
4. The client report that the channel is closed.

-- 
Configure bugmail: https://bugs.eclipse.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.