X-Git-Url: http://andersk.mit.edu/gitweb/libfaim.git/blobdiff_plain/040457cc47af3fd2ed743bda700e28a603d39eb6..78b3fb1309a7d926cb58b8322923feedd1ba47ef:/aim_msgcookie.c

diff --git a/aim_msgcookie.c b/aim_msgcookie.c
index 3481162..4ef8c8a 100644
--- a/aim_msgcookie.c
+++ b/aim_msgcookie.c
@@ -1,58 +1,101 @@
+/*
+ * Cookie Caching stuff. Adam wrote this, apparently just some
+ * derivatives of n's SNAC work. I cleaned it up, added comments.
+ * 
+ * I'm going to rewrite this stuff eventually, honest. -jbm
+ * 
+ */
 
 /*
- *
- *
+ * I'm assuming that cookies are type-specific. that is, we can have
+ * "1234578" for type 1 and type 2 concurrently. if i'm wrong, then we
+ * lose some error checking. if we assume cookies are not type-specific and are
+ * wrong, we get quirky behavior when cookies step on each others' toes.
  */
 
 #include <faim/aim.h>
 
-int aim_cachecookie(struct aim_session_t *sess,
-		    struct aim_msgcookie_t *cookie)
+/*
+ * aim_cachecookie: 
+ * appends a cookie to the cookie list for sess.
+ * - if cookie->cookie for type cookie->type is found, addtime is updated.
+ * - copies cookie struct; you need to free() it afterwards;
+ * - cookie->data is not copied, but passed along. don't free it.
+ * - newcook->addtime is updated accordingly;
+ * - cookie->type is just passed across.
+ * 
+ * returns -1 on error, 0 on success.  */
+
+faim_internal int aim_cachecookie(struct aim_session_t *sess,
+				  struct aim_msgcookie_t *cookie)
 {
   struct aim_msgcookie_t *newcook = NULL, *cur = NULL;
   
   if (!cookie)
     return -1;
 
+  if( (newcook = aim_checkcookie(sess, cookie->cookie, cookie->type)) ) {
+    newcook->addtime = time(NULL);
+    if(cookie->data != newcook->data) {
+      
+      printf("faim: cachecookie: matching cookie/type pair "
+	     "%x%x%x%x%x%x%x%x/%x has different *data. free()ing cookie copy..\n",
+	     cookie->cookie[0], cookie->cookie[1], cookie->cookie[2],
+	     cookie->cookie[3], cookie->cookie[4], cookie->cookie[5],
+	     cookie->cookie[6], cookie->cookie[7], cookie->type);
+      
+      free(cookie->data);
+    }
+    return(0);
+  }
+  
   if (!(newcook = malloc(sizeof(struct aim_msgcookie_t))))
     return -1;
   memcpy(newcook, cookie, sizeof(struct aim_msgcookie_t));
   newcook->addtime = time(NULL);
-  newcook->next = NULL;
 
+  if(newcook->next)
+    printf("faim: cachecookie: newcook->next isn't NULL ???\n");
+
+  newcook->next = NULL;
+  
   cur = sess->msgcookies;
   
   if (cur == NULL) {
     sess->msgcookies = newcook;
     return 0;
   }
+
   while (cur->next != NULL)
     cur = cur->next;
   cur->next = newcook;
-
+  
   return 0;
 }
 
-struct aim_msgcookie_t *aim_uncachecookie(struct aim_session_t *sess, 
-					  char *cookie)
+/*
+ * aim_uncachecookie:
+ * takes a cookie string and grabs the cookie struct associated with
+ * it. removes struct from chain.  returns the struct if found, or
+ * NULL on not found.
+ */
+
+faim_internal struct aim_msgcookie_t *aim_uncachecookie(struct aim_session_t *sess, unsigned char *cookie, int type)
 {
   struct aim_msgcookie_t *cur;
 
-  if (!cookie)
+  if (!cookie || !sess->msgcookies)
     return NULL;
 
-  if (!sess->msgcookies)
-    return NULL;
+  cur = sess->msgcookies;
 
-  if (memcmp(sess->msgcookies->cookie, cookie, 8) == 0) {
-    cur = sess->msgcookies;
+  if ( (memcmp(cur->cookie, cookie, 8) == 0) && (cur->type == type) ) {
     sess->msgcookies = cur->next;
     return cur;
   } 
 
-  cur = sess->msgcookies;
   while (cur->next) {
-    if (memcmp(cur->next->cookie, cookie, 8) == 0) {
+    if ( (memcmp(cur->next->cookie, cookie, 8) == 0) && (cur->next->type == type) ) {
       struct aim_msgcookie_t *tmp;
       
       tmp = cur->next;
@@ -65,10 +108,20 @@ struct aim_msgcookie_t *aim_uncachecookie(struct aim_session_t *sess,
 }
 
 /*
+ * aim_purgecookies:
+ * purge out old cookies
+ *
+ * finds old cookies, calls uncache on them.  
+ *
+ * this is highly inefficient, but It Works. and i don't feel like
+ * totally rewriting this. it might have some concurrency issues as
+ * well, if i rewrite it.
+ *
+ * i'll avoid the puns.  
  */
-int aim_purgecookies(struct aim_session_t *sess)
+
+faim_export int aim_purgecookies(struct aim_session_t *sess, int maxage)
 {
-  int maxage = 5*60;
   struct aim_msgcookie_t *cur;
   struct aim_msgcookie_t *remed = NULL;
   time_t curtime;
@@ -78,22 +131,93 @@ int aim_purgecookies(struct aim_session_t *sess)
   curtime = time(&curtime);
  
   while (cur) {
-    if ( (cur) && (((cur->addtime) + maxage) < curtime)) {
+    if ( (cur->addtime) > (curtime - maxage) ) {
 #if DEBUG > 1
       printf("aimmsgcookie: WARNING purged obsolete message cookie %x%x%x%x %x%x%x%x\n",
 	     cur->cookie[0], cur->cookie[1], cur->cookie[2], cur->cookie[3],
 	     cur->cookie[4], cur->cookie[5], cur->cookie[6], cur->cookie[7]);
 #endif
-      remed = aim_uncachecookie(sess, cur->cookie);
+
+      remed = aim_uncachecookie(sess, cur->cookie, cur->type);
       if (remed) {
 	if (remed->data)
 	  free(remed->data);
 	free(remed);
       }
     }
+
     cur = cur->next;
+
   }
   
   return 0;
 }
 
+faim_internal struct aim_msgcookie_t *aim_mkcookie(unsigned char *c, int type, void *data) 
+{
+  struct aim_msgcookie_t *cookie;
+
+  if(!c)
+    return(NULL);
+
+  if( (cookie = calloc(1, sizeof(struct aim_msgcookie_t))) == NULL)
+    return(NULL);
+  
+  cookie->data = data;
+
+  cookie->type = type;
+  
+  memcpy(cookie->cookie, c, 8);
+  
+  return(cookie);
+}
+  
+faim_internal struct aim_msgcookie_t *aim_checkcookie(struct aim_session_t *sess, unsigned char *cookie, int type)
+{
+  struct aim_msgcookie_t *cur;
+  
+  if(!sess->msgcookies)
+    return NULL;
+
+  cur = sess->msgcookies;
+
+  if( (memcmp(cur->cookie, cookie, 8) == 0) && (cur->type == type))
+    return(cur);
+
+  while( (cur = cur->next) )
+    if( (memcmp(cur->cookie, cookie, 8) == 0) && (cur->type == type))
+      return(cur);
+
+  return(NULL);
+}
+
+static int aim_freecookie(struct aim_msgcookie_t *cookie) {
+  return(0);
+} 
+
+faim_internal int aim_msgcookie_gettype(int reqclass) {
+  /* XXX: hokey-assed. needs fixed. */
+  switch(reqclass) {
+  case AIM_CAPS_BUDDYICON:
+    return AIM_COOKIETYPE_OFTICON;
+    break;
+  case AIM_CAPS_VOICE:
+    return AIM_COOKIETYPE_OFTVOICE;
+    break;
+  case AIM_CAPS_IMIMAGE:
+    return AIM_COOKIETYPE_OFTIMAGE;
+    break;
+  case AIM_CAPS_CHAT:
+    return AIM_COOKIETYPE_CHAT;
+    break;
+  case AIM_CAPS_GETFILE:
+    return AIM_COOKIETYPE_OFTGET;
+    break;
+  case AIM_CAPS_SENDFILE:
+    return AIM_COOKIETYPE_OFTSEND;
+    break;
+  default:
+    return AIM_COOKIETYPE_UNKNOWN;
+    break;
+  }           
+}