X-Git-Url: http://andersk.mit.edu/gitweb/libfaim.git/blobdiff_plain/646c6b5273a2272632e50e7393f090f7c8bf06eb..d2587300fe3ca6bf664c93b5f6258d275fe75155:/utils/faimtest/faimtest.c

diff --git a/utils/faimtest/faimtest.c b/utils/faimtest/faimtest.c
index c403ab7..e10d494 100644
--- a/utils/faimtest/faimtest.c
+++ b/utils/faimtest/faimtest.c
@@ -38,12 +38,7 @@
  *
  */
 
-/*
-  Current status:
-
- */
-
-#include <aim.h> 
+#include "faimtest.h"
 
 static char *dprintf_ctime(void)
 {
@@ -154,11 +149,15 @@ static char *msgerrreasons[] = {
 static int msgerrreasonslen = 25;
 
 static char *screenname,*password,*server=NULL;
+static char *proxy = NULL, *proxyusername = NULL, *proxypass = NULL;
 static char *ohcaptainmycaptain = NULL;
 static int connected = 0;
 
-FILE *listingfile;
-char *listingpath;
+struct aim_session_t aimsess;
+int keepgoing = 1;
+
+static FILE *listingfile;
+static char *listingpath;
 
 static void faimtest_debugcb(struct aim_session_t *sess, int level, const char *format, va_list va)
 {
@@ -237,12 +236,81 @@ int initwsa(void)
 }
 #endif /* _WIN32 */
 
+int faimtest_init(void)
+{
+  struct aim_conn_t *stdinconn = NULL;
+
+  if (!(stdinconn = aim_newconn(&aimsess, 0, NULL))) {
+    dprintf("unable to create connection for stdin!\n");
+    return -1;
+  }
+
+  stdinconn->fd = STDIN_FILENO;
+
+  return 0;
+}
+
+int logout(void)
+{
+
+  if (ohcaptainmycaptain)
+    aim_send_im(&aimsess, aim_getconn_type(&aimsess, AIM_CONN_TYPE_BOS), ohcaptainmycaptain, 0, "ta ta...");
+
+  aim_logoff(&aimsess);
+
+  if (faimtest_init() == -1)
+    dprintf("faimtest_init failed\n");
+
+  return 0;
+}
+
+int login(const char *sn, const char *passwd)
+{
+  struct aim_conn_t *authconn;
+
+  if (sn)
+    screenname = strdup(sn);
+  if (passwd)
+    password = strdup(passwd);
+
+  if (proxy)
+    aim_setupproxy(&aimsess, proxy, proxyusername, proxypass);
+
+  if (!screenname || !password) {
+    dprintf("need SN and password\n");
+    return -1;
+  }
+
+  if (!(authconn = aim_newconn(&aimsess, AIM_CONN_TYPE_AUTH, server?server:FAIM_LOGIN_SERVER))) {
+    dprintf("faimtest: internal connection error while in aim_login.  bailing out.\n");
+    return -1;
+  } else if (authconn->fd == -1) {
+    if (authconn->status & AIM_CONN_STATUS_RESOLVERR) {
+      dprintf("faimtest: could not resolve authorizer name\n");
+    } else if (authconn->status & AIM_CONN_STATUS_CONNERR) {
+      dprintf("faimtest: could not connect to authorizer\n");
+    }
+    aim_conn_kill(&aimsess, &authconn);
+    return -1;
+  }
+
+  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_FLAPVER, faimtest_flapversion, 0);
+  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_CONNCOMPLETE, faimtest_conncomplete, 0);
+  aim_conn_addhandler(&aimsess, authconn, 0x0017, 0x0007, faimtest_parse_login, 0);
+  aim_conn_addhandler(&aimsess, authconn, 0x0017, 0x0003, faimtest_parse_authresp, 0);    
+
+  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_DEBUGCONN_CONNECT, faimtest_debugconn_connect, 0);
+
+  /* If the connection is in progress, this will just be queued */
+  aim_request_login(&aimsess, authconn, screenname);
+  dprintf("faimtest: login request sent\n");
+
+  return 0;
+}
+
 int main(int argc, char **argv)
 {
-  struct aim_session_t aimsess;
-  struct aim_conn_t *authconn = NULL, *waitingconn = NULL;
-  int keepgoing = 1;
-  char *proxy, *proxyusername, *proxypass;
+  struct aim_conn_t *waitingconn = NULL;
   int i;
   int selstat = 0;
 
@@ -267,7 +335,6 @@ int main(int argc, char **argv)
     case 'c': ohcaptainmycaptain = optarg; break;
     case 'h':
     default:
-    usage:
       printf("faimtest\n");
       printf(" Options: \n");
       printf("    -u name       Screen name ($SCREENNAME)\n");
@@ -282,9 +349,6 @@ int main(int argc, char **argv)
     }
   }
 
-  if (!screenname || !password)
-    goto usage;
-
 #ifdef _WIN32
   if (initwsa() != 0) {
     dprintf("faimtest: could not initialize windows sockets\n");
@@ -311,88 +375,54 @@ int main(int argc, char **argv)
     free(listingname);
   }
 
-  if (proxy)
-    aim_setupproxy(&aimsess, proxy, proxyusername, proxypass);
+  faimtest_init();
 
-  authconn = aim_newconn(&aimsess, AIM_CONN_TYPE_AUTH, server?server:FAIM_LOGIN_SERVER);
-
-  if (authconn == NULL) {
-    dprintf("faimtest: internal connection error while in aim_login.  bailing out.\n");
-    return -1;
-  } else if (authconn->fd == -1) {
-    if (authconn->status & AIM_CONN_STATUS_RESOLVERR) {
-      dprintf("faimtest: could not resolve authorizer name\n");
-    } else if (authconn->status & AIM_CONN_STATUS_CONNERR) {
-      dprintf("faimtest: could not connect to authorizer\n");
-    }
-    aim_conn_kill(&aimsess, &authconn);
-    return -1;
-  }
-
-  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_FLAPVER, faimtest_flapversion, 0);
-  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_CONNCOMPLETE, faimtest_conncomplete, 0);
-  aim_conn_addhandler(&aimsess, authconn, 0x0017, 0x0007, faimtest_parse_login, 0);
-  aim_conn_addhandler(&aimsess, authconn, 0x0017, 0x0003, faimtest_parse_authresp, 0);    
-
-  aim_conn_addhandler(&aimsess, authconn, AIM_CB_FAM_SPECIAL, AIM_CB_SPECIAL_DEBUGCONN_CONNECT, faimtest_debugconn_connect, 0);
-
-  /* If the connection is in progress, this will just be queued */
-  aim_request_login(&aimsess, authconn, screenname);
-  dprintf("faimtest: login request sent\n");
+  cmd_init();
 
   while (keepgoing) {
     waitingconn = aim_select(&aimsess, NULL, &selstat);
 
-    switch(selstat) {
-    case -1: /* error */
+    if (selstat == -1) { /* error */
       keepgoing = 0; /* fall through and hit the aim_logoff() */
-      break;
-
-    case 0: /* no events pending */
-      break;
-
-    case 1: /* outgoing data pending */
+    } else if (selstat == 0) { /* no events pending */
+      keepgoing = 0;
+    } else if (selstat == 1) { /* outgoing data pending */
       aim_tx_flushqueue(&aimsess);
-      break;
-
-    case 2: /* incoming data pending */
-      if (waitingconn->type == AIM_CONN_TYPE_RENDEZVOUS_OUT) {
-	if (aim_handlerendconnect(&aimsess, waitingconn) < 0) {
-	  dprintf("connection error (rend out)\n");
-	  aim_conn_kill(&aimsess, &waitingconn);
-	}
-	break;
-      }
-            
-      if (aim_get_command(&aimsess, waitingconn) >= 0) {
-	aim_rxdispatch(&aimsess);
+    } else if (selstat == 2) { /* incoming data pending */
+      if (waitingconn->fd == STDIN_FILENO) {
+	cmd_gotkey();
       } else {
-	dvprintf("connection error (type 0x%04x:0x%04x)\n", waitingconn->type, waitingconn->subtype);
-	/* we should have callbacks for all these, else the library will do the conn_kill for us. */
-	if(waitingconn->type == AIM_CONN_TYPE_RENDEZVOUS) {
-	  dprintf("connection error: rendezvous connection. you forgot register a disconnect callback, right?\n");	  
-	  aim_conn_kill(&aimsess, &waitingconn);
-	} else
-	  aim_conn_kill(&aimsess, &waitingconn);
-	if (!aim_getconn_type(&aimsess, AIM_CONN_TYPE_BOS)) {
-	  dprintf("major connection error\n");
-	  keepgoing = 0;
+	if (waitingconn->type == AIM_CONN_TYPE_RENDEZVOUS_OUT) {
+	  if (aim_handlerendconnect(&aimsess, waitingconn) < 0) {
+	    dprintf("connection error (rend out)\n");
+	    aim_conn_kill(&aimsess, &waitingconn);
+	  }
+	} else {
+	  if (aim_get_command(&aimsess, waitingconn) >= 0) {
+	    aim_rxdispatch(&aimsess);
+	  } else {
+	    dvprintf("connection error (type 0x%04x:0x%04x)\n", waitingconn->type, waitingconn->subtype);
+	    /* we should have callbacks for all these, else the library will do the conn_kill for us. */
+	    if(waitingconn->type == AIM_CONN_TYPE_RENDEZVOUS) {
+	      dprintf("connection error: rendezvous connection. you forgot register a disconnect callback, right?\n");	  
+	      aim_conn_kill(&aimsess, &waitingconn);
+	    } else
+	      aim_conn_kill(&aimsess, &waitingconn);
+	    if (!aim_getconn_type(&aimsess, AIM_CONN_TYPE_BOS)) {
+	      dprintf("major connection error\n");
+	    }
+	  }
 	}
       }
-      
-      break;
-    
-    default:
-      break; /* invalid */
     }
   }
 
-  /* Close up */
-  dprintf("AIM just decided we didn't need to be here anymore, closing up...\n");
-  
   /* close up all connections, dead or no */
   aim_logoff(&aimsess); 
 
+  printf("\n");
+  cmd_uninit();
+
   /* Get out */
   exit(0);
 }
@@ -885,20 +915,144 @@ int faimtest_parse_userinfo(struct aim_session_t *sess, struct command_rx_struct
   return 1;
 }
 
+static int faimtest_handlecmd(struct aim_session_t *sess, struct command_rx_struct *command, struct aim_userinfo_s *userinfo, char *tmpstr)
+{
+
+  if (!strncmp(tmpstr, "disconnect", 10)) {
+
+    logout();
+
+  } else if (strstr(tmpstr, "goodday")) {
+
+      aim_send_im(sess, command->conn, userinfo->sn, AIM_IMFLAGS_ACK, "Good day to you too.");
+
+  } else if (strstr(tmpstr, "warnme")) {
+
+    dprintf("faimtest: icbm: sending non-anon warning\n");
+    aim_send_warning(sess, command->conn, userinfo->sn, 0);
+
+  } else if (strstr(tmpstr, "anonwarn")) {
+
+    dprintf("faimtest: icbm: sending anon warning\n");
+    aim_send_warning(sess, command->conn, userinfo->sn, AIM_WARN_ANON);
+
+  } else if (strstr(tmpstr, "setdirectoryinfo")) {
+
+    dprintf("faimtest: icbm: sending backwards profile data\n");
+    aim_setdirectoryinfo(sess, command->conn, "tsrif", "elddim", "tsal", "nediam", "emankcin", "teerts", "ytic", "etats", "piz", 0, 1);
+
+  } else if (strstr(tmpstr, "setinterests")) {
+
+    dprintf("faimtest: icbm: setting fun interests\n");
+    aim_setuserinterests(sess, command->conn, "interest1", "interest2", "interest3", "interest4", "interest5", 1);
+
+  } else if (!strncmp(tmpstr, "getfile", 7)) {
+
+    if (!ohcaptainmycaptain) {
+
+      aim_send_im(sess, command->conn, userinfo->sn, AIM_IMFLAGS_ACK, "I have no owner!");
+
+    } else {
+      struct aim_conn_t *newconn;
+
+      newconn = aim_getfile_initiate(sess, command->conn, (strlen(tmpstr) < 8)?ohcaptainmycaptain:tmpstr+8);
+      dvprintf("faimtest: getting file listing from %s\n", (strlen(tmpstr) < 8)?ohcaptainmycaptain:tmpstr+8);
+      aim_conn_addhandler(sess, newconn,  AIM_CB_FAM_OFT, AIM_CB_OFT_GETFILEINITIATE, faimtest_getfile_initiate,0);
+    }
+
+  } else if (!strncmp(tmpstr, "open chatnav", 12)) {
+
+    aim_bos_reqservice(sess, command->conn, AIM_CONN_TYPE_CHATNAV);
+
+  } else if (!strncmp(tmpstr, "create", 6)) {
+
+    aim_chatnav_createroom(sess,aim_getconn_type(sess, AIM_CONN_TYPE_CHATNAV), (strlen(tmpstr) < 7)?"WorldDomination":tmpstr+7, 0x0004);
+
+  } else if (!strncmp(tmpstr, "close chatnav", 13)) {
+    struct aim_conn_t *chatnavconn;
+
+    chatnavconn = aim_getconn_type(sess, AIM_CONN_TYPE_CHATNAV);
+    aim_conn_kill(sess, &chatnavconn);
+
+  } else if (!strncmp(tmpstr, "join", 4)) {
+
+    aim_chat_join(sess, command->conn, 0x0004, "worlddomination");
+
+  } else if (!strncmp(tmpstr, "leave", 5)) {
+
+    aim_chat_leaveroom(sess, "worlddomination");
+
+  } else if (!strncmp(tmpstr, "getinfo", 7)) {
+
+    aim_getinfo(sess, command->conn, "75784102", AIM_GETINFO_GENERALINFO);
+    aim_getinfo(sess, command->conn, "15853637", AIM_GETINFO_AWAYMESSAGE);
+
+  } else if (!strncmp(tmpstr, "open directim", 13)) {
+    struct aim_conn_t *newconn;
+
+    printf("faimtest: opening directim to %s\n", (strlen(tmpstr) < 14)?userinfo->sn:tmpstr+14);
+    newconn = aim_directim_initiate(sess, command->conn, NULL, (strlen(tmpstr) < 14)?userinfo->sn:tmpstr+14);
+    if(!newconn || newconn->fd == -1)
+      printf("connection failed!\n");
+    aim_conn_addhandler(sess, newconn,  AIM_CB_FAM_OFT, AIM_CB_OFT_DIRECTIMINITIATE, faimtest_directim_initiate,0);
+
+  } else if(!(strncmp(tmpstr, "lookup", 6))) {
+
+    aim_usersearch_address(sess, command->conn, tmpstr+7);
+
+  } else if (!strncmp(tmpstr, "reqsendmsg", 10)) {
+
+    aim_send_im(sess, command->conn, ohcaptainmycaptain, 0, "sendmsg 7900");
+
+  } else if (!strncmp(tmpstr, "reqauth", 7)) {
+
+    aim_bos_reqservice(sess, command->conn, AIM_CONN_TYPE_AUTH);
+
+  } else if (!strncmp(tmpstr, "reqconfirm", 10)) {
+
+    aim_auth_reqconfirm(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH));
+
+  } else if (!strncmp(tmpstr, "reqemail", 8)) {
+
+    aim_auth_getinfo(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), 0x0011);
+
+  } else if (!strncmp(tmpstr, "changepass", 8)) {
+
+    aim_auth_changepasswd(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), "NEWPASSWORD", "OLDPASSWORD");
+
+  } else if (!strncmp(tmpstr, "setemail", 8)) {
+
+    aim_auth_setemail(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), "NEWEMAILADDRESS");
+
+  } else if (!strncmp(tmpstr, "sendmsg", 7)) {
+    int i;
+    i = atoi(tmpstr+8);
+    if (i < 10000) {
+      char *newbuf;
+      int z;
+
+      newbuf = malloc(i+1);
+      for (z = 0; z < i; z++) {
+	newbuf[z] = (z % 10)+0x30;
+      }
+      newbuf[i] = '\0';
+      aim_send_im(sess, command->conn, userinfo->sn, 0, newbuf);
+      free(newbuf);
+    }
+
+  } else {
+
+    dprintf("unknown command.\n");
+    aim_add_buddy(sess, command->conn, userinfo->sn);
+
+  }  
+
+  return 0;
+}
+
 /*
  * The user-level Incoming ICBM callback.
  *
- * Arguments:
- *  struct command_rx_struct *  command     if you feel like doing it yourself
- *  char *                      srcsn       the source name
- *  char *                      msg         message
- *  int                         warnlevel   warning/evil level
- *  int                         flags       flags
- *  ulong                       membersince time_t of date of signup
- *  ulong                       onsince     time_t of date of singon
- *  int                         idletime    min (sec?) idle
- *  u_int                       icbmflags   sets AIM_IMFLAGS_{AWAY,ACK}
- *
  */
 int faimtest_parse_incoming_im(struct aim_session_t *sess, struct command_rx_struct *command, ...)
 {
@@ -967,93 +1121,8 @@ int faimtest_parse_incoming_im(struct aim_session_t *sess, struct command_rx_str
       }
       tmpstr = msg+i;
 
-      dvprintf("tmpstr = %s\n", tmpstr);
-      
-      if ( (strlen(tmpstr) >= 10) &&
-	   (!strncmp(tmpstr, "disconnect", 10)) ) {
-	if (ohcaptainmycaptain)
-	  aim_send_im(sess, command->conn, ohcaptainmycaptain, 0, "ta ta...");
-	aim_logoff(sess);
-      } else if (strstr(tmpstr, "goodday")) {
-	dprintf("faimtest: icbm: sending response\n");
-	aim_send_im(sess, command->conn, userinfo->sn, AIM_IMFLAGS_ACK, "Good day to you too.");
-      } else if (strstr(tmpstr, "warnme")) {
-	dprintf("faimtest: icbm: sending non-anon warning\n");
-	aim_send_warning(sess, command->conn, userinfo->sn, 0);
-      } else if (strstr(tmpstr, "anonwarn")) {
-	dprintf("faimtest: icbm: sending anon warning\n");
-	aim_send_warning(sess, command->conn, userinfo->sn, AIM_WARN_ANON);
-      } else if (strstr(tmpstr, "setdirectoryinfo")) {
-	dprintf("faimtest: icbm: sending backwards profile data\n");
-	aim_setdirectoryinfo(sess, command->conn, "tsrif", "elddim", "tsal", "nediam", "emankcin", "teerts", "ytic", "etats", "piz", 0, 1);
-      } else if (strstr(tmpstr, "setinterests")) {
-	dprintf("faimtest: icbm: setting fun interests\n");
-	aim_setuserinterests(sess, command->conn, "interest1", "interest2", "interest3", "interest4", "interest5", 1);
-      } else if (!strncmp(tmpstr, "getfile", 7)) {
-	if (!ohcaptainmycaptain) {
-	  aim_send_im(sess, command->conn, userinfo->sn, AIM_IMFLAGS_ACK, "I have no owner!");
-	} else {
-	  struct aim_conn_t *newconn;
-	  newconn = aim_getfile_initiate(sess, command->conn, (strlen(tmpstr) < 8)?ohcaptainmycaptain:tmpstr+8);
-	  dvprintf("faimtest: getting file listing from %s\n", (strlen(tmpstr) < 8)?ohcaptainmycaptain:tmpstr+8);
-	  aim_conn_addhandler(sess, newconn,  AIM_CB_FAM_OFT, AIM_CB_OFT_GETFILEINITIATE, faimtest_getfile_initiate,0);
-	}
-      } else if (!strncmp(tmpstr, "open chatnav", 12)) {
-	aim_bos_reqservice(sess, command->conn, AIM_CONN_TYPE_CHATNAV);
-      } else if (!strncmp(tmpstr, "create", 6)) {
-	aim_chatnav_createroom(sess,aim_getconn_type(sess, AIM_CONN_TYPE_CHATNAV), (strlen(tmpstr) < 7)?"WorldDomination":tmpstr+7, 0x0004);
-      } else if (!strncmp(tmpstr, "close chatnav", 13)) {
-	struct aim_conn_t *chatnavconn;
-	chatnavconn = aim_getconn_type(sess, AIM_CONN_TYPE_CHATNAV);
-	aim_conn_kill(sess, &chatnavconn);
-      } else if (!strncmp(tmpstr, "join", 4)) {
-	  aim_chat_join(sess, command->conn, 0x0004, "worlddomination");
-      } else if (!strncmp(tmpstr, "leave", 5))
-	    aim_chat_leaveroom(sess, "worlddomination");
-      else if (!strncmp(tmpstr, "getinfo", 7)) {
-	aim_getinfo(sess, command->conn, "75784102", AIM_GETINFO_GENERALINFO);
-	aim_getinfo(sess, command->conn, "15853637", AIM_GETINFO_AWAYMESSAGE);
-      } else if (!strncmp(tmpstr, "open directim", 13)) {
-	struct aim_conn_t *newconn;
-	printf("faimtest: opening directim to %s\n", (strlen(tmpstr) < 14)?userinfo->sn:tmpstr+14);
-       	newconn = aim_directim_initiate(sess, command->conn, NULL, (strlen(tmpstr) < 14)?userinfo->sn:tmpstr+14);
-	if(!newconn || newconn->fd == -1)
-	  printf("connection failed!\n");
-	aim_conn_addhandler(sess, newconn,  AIM_CB_FAM_OFT, AIM_CB_OFT_DIRECTIMINITIATE, faimtest_directim_initiate,0);
-      } else if(!(strncmp(tmpstr, "lookup", 6))) {
-	aim_usersearch_address(sess, command->conn, tmpstr+7);
-      } else if (!strncmp(tmpstr, "reqsendmsg", 10)) {
-	aim_send_im(sess, command->conn, ohcaptainmycaptain, 0, "sendmsg 7900");
-      } else if (!strncmp(tmpstr, "reqauth", 7)) {
-	aim_bos_reqservice(sess, command->conn, AIM_CONN_TYPE_AUTH);
-      } else if (!strncmp(tmpstr, "reqconfirm", 10)) {
-	aim_auth_reqconfirm(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH));
-      } else if (!strncmp(tmpstr, "reqemail", 8)) {
-	aim_auth_getinfo(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), 0x0011);
-      } else if (!strncmp(tmpstr, "changepass", 8)) {
-	aim_auth_changepasswd(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), "NEWPASSWORD", "OLDPASSWORD");
-      } else if (!strncmp(tmpstr, "setemail", 8)) {
-	aim_auth_setemail(sess, aim_getconn_type(sess, AIM_CONN_TYPE_AUTH), "NEWEMAILADDRESS");
-      } else if (!strncmp(tmpstr, "sendmsg", 7)) {
-	int i;
-	i = atoi(tmpstr+8);
-	if (i < 10000) {
-	  char *newbuf;
-	  int z;
-
-	  newbuf = malloc(i+1);
-	  for (z = 0; z < i; z++) {
-	    newbuf[z] = (z % 10)+0x30;
-	  }
-	  newbuf[i] = '\0';
-	  aim_send_im(sess, command->conn, userinfo->sn, 0, newbuf);
-	  free(newbuf);
-	}
-      } else {
-	dprintf("unknown command.\n");
-	aim_add_buddy(sess, command->conn, userinfo->sn);
-      }
-      
+      faimtest_handlecmd(sess, command, userinfo, tmpstr);
+
     }
   }
   /*
@@ -1245,16 +1314,21 @@ int faimtest_directim_connect(struct aim_session_t *sess, struct command_rx_stru
 int faimtest_directim_incoming(struct aim_session_t *sess, struct command_rx_struct *command, ...)
 {
   va_list ap;
-  char *sn = NULL, *msg = NULL;
+  char *msg = NULL;
   struct aim_conn_t *conn;
+  struct aim_directim_priv *priv;
 
   va_start(ap, command);
   conn = va_arg(ap, struct aim_conn_t *);
-  sn = va_arg(ap, char *);
   msg = va_arg(ap, char *);
   va_end(ap);
 
-  dvprintf("faimtest: Directim from %s: %s\n", sn, msg);
+  if(!(priv = conn->priv)) {
+    dvprintf("faimtest: directim: no private struct on conn with fd %d\n", conn->fd);
+    return -1;
+  }
+
+  dvprintf("faimtest: Directim from %s: %s\n", priv->sn, msg);
   if (!strncmp(msg, "sendmsg", 7)) {
     int i;
     i = atoi(msg+8);
@@ -1300,13 +1374,19 @@ int faimtest_directim_disconnect(struct aim_session_t *sess, struct command_rx_s
 int faimtest_directim_typing(struct aim_session_t *sess, struct command_rx_struct *command, ...)
 {
   va_list ap;
-  char *sn;
-  
+  struct aim_conn_t *conn;
+  struct aim_directim_priv *priv;
+
   va_start(ap, command);
-  sn = va_arg(ap, char *);
+  conn = va_arg(ap, struct aim_conn_t *);
   va_end(ap);
+  
+  if(!(priv = (struct aim_directim_priv *)conn->priv)) {
+    dvprintf("faimtest: no private struct on conn with fd %d!\n", conn->fd);
+    return -1;
+  }
 
-  dvprintf("faimtest: ohmigod! %s has started typing (DirectIM). He's going to send you a message! *squeal*\n", sn);
+  dvprintf("faimtest: ohmigod! %s has started typing (DirectIM). He's going to send you a message! *squeal*\n", priv->sn);
   return 1;
 }