Changeset 5838:52ca1a88734e in roaraudio for libroar/basic.c

Timestamp:

01/07/13 02:36:01 (11 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

handle setting of server name in connection object in a better way. also added new command "servername" to roarctl to debug this a bit. This is one little step needed by ##337

File:

• libroar/basic.c (modified) (7 diffs)

libroar/basic.c

@@ -46 +46 @@
  int r;
  char fhstr[12];
+ size_t len;
 
  if ( !strncmp(server, "+fork=", 6) ) {
@@ -78 +79 @@
   daemonimage = "roard";
 
+ len = roar_mm_strlen(daemonimage) + 9; // 9 = '+fork=' + 'X:' + '\0'
+ con->server_name = roar_mm_malloc(len);
+ if ( con->server_name != NULL ) {
+  snprintf(con->server_name, len, "+fork=%s%s", mode == NORMAL ? "d:" : "!", daemonimage);
+ }
+
  if ( socketpair(AF_UNIX, SOCK_STREAM, 0, socks) == -1 ) {
   roar_err_from_errno();
@@ -164 +171 @@
  int fh = -1;
  int err;
+ int ret;
 
  if ( con == NULL || server == NULL ) {
@@ -177 +185 @@
    return -1;
   con->flags |= ROAR_CON_FLAGS_VIO;
+  con->server_name = roar_mm_strdup(server);
   return 0;
  } else if ( !strcmp(server, "+fork") || !strncmp(server, "+fork=", 6) ) {
   return _start_server(con, server, type, flags, timeout);
  } else if ( !strcmp(server, "+internal") || !strncmp(server, "+internal=", 10) ) {
-  return _connect_internal(con, server, type, flags, timeout);
+  ret = _connect_internal(con, server, type, flags, timeout);
+  if ( ret == 0 )
+   con->server_name = roar_mm_strdup(server);
+  return ret;
  }
 
@@ -217 +229 @@
 
   if ( *obj == 0 ) {
-#ifdef DN_MAXOBJL
    roar_mm_strlcat(user_sock, ROAR_DEFAULT_OBJECT, sizeof(user_sock)-1);
    user_sock[sizeof(user_sock)-1] = 0;
-#else
-   ROAR_ERR("roar_connect_raw(*): size of DECnet object unknown.");
-#endif
-  }
-   ROAR_DBG("roar_connect_raw(*): user_sock='%s'", user_sock);
+  }
+
+  ROAR_DBG("roar_connect_raw(*): user_sock='%s'", user_sock);
  }
 
@@ -265 +274 @@
  }
 
+ con->server_name = roar_mm_strdup(server);
+
  roar_err_set(ROAR_ERROR_NONE);
  return 0;
@@ -449 +460 @@
   return -1;
 
- // TODO: this should be moved to be within _connect_server() so it also stores server names of auto located servers.
- if ( server != NULL ) {
-  con->server_name = roar_mm_strdup(server);
- }
-
  return 0;
 }