summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
e005efa)
It was only used by one command, so move the command into
the big case statement and remove a now unnecessary function.
Spring cleaning.
Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>
-/* Simple commands are those which need only respond with a return value. */
-static int process_simple_client_cmd(int command, char *recv_buf)
-{
- int result;
-
- switch(command) {
- case SET_SOCK_PATH:
- {
- struct ustcomm_single_field *sock_msg;
- sock_msg = (struct ustcomm_single_field *)recv_buf;
- result = ustcomm_unpack_single_field(sock_msg);
- if (result < 0) {
- return result;
- }
- return setenv("UST_DAEMON_SOCKET", sock_msg->field, 1);
- }
-
- default:
- return -EINVAL;
- }
-
- return 0;
-}
-
-
static int process_trace_cmd(int command, char *trace_name)
{
int result;
static int process_trace_cmd(int command, char *trace_name)
{
int result;
+ case SET_SOCK_PATH:
+ {
+ struct ustcomm_single_field *sock_msg;
+ sock_msg = (struct ustcomm_single_field *)recv_buf;
+ result = ustcomm_unpack_single_field(sock_msg);
+ if (result < 0) {
+ reply_header->result = -EINVAL;
+ goto send_response;
+ }
+
+ reply_header->result = setenv("UST_DAEMON_SOCKET",
+ sock_msg->field, 1);
+
+ goto send_response;
+ }
case START:
case SETUP_TRACE:
case ALLOC_TRACE:
case START:
case SETUP_TRACE:
case ALLOC_TRACE:
- reply_header->result =
- process_simple_client_cmd(recv_header->command,
- recv_buf);
- goto send_response;
+ reply_header->result = -EINVAL;