Changeset ff7449e for aim.c

Timestamp:

Jul 19, 2011, 3:30:24 AM (13 years ago)

Author:

David Benjamin <davidben@mit.edu>

Branches:

master, release-1.10, release-1.9

Children:

d8f22b6

Parents:

837bd81

git-author:

David Benjamin <davidben@mit.edu> (07/19/11 03:16:31)

git-committer:

David Benjamin <davidben@mit.edu> (07/19/11 03:30:24)

Message:

Don't leak realmsg when... what is this I don't even

I don't know what in the world this code is doing after the return, but
it leaks!

File:

• aim.c (modified) (1 diff)

aim.c

@@ -1064 +1064 @@
   g_free(wrapmsg);
   g_free(nz_screenname);
+  g_free(realmsg);
 
   return(1);
 
-  owl_function_debugmsg("faimtest_parse_incoming_im_chan1: icbm: message: %s\n", realmsg);
-  
-  if (args->icbmflags & AIM_IMFLAGS_MULTIPART) {
-    aim_mpmsg_section_t *sec;
-    int z;
-
-    owl_function_debugmsg("faimtest_parse_incoming_im_chan1: icbm: multipart: this message has %d parts\n", args->mpmsg.numparts);
-    
-    for (sec = args->mpmsg.parts, z = 0; sec; sec = sec->next, z++) {
-      if ((sec->charset == 0x0000) || (sec->charset == 0x0003) || (sec->charset == 0xffff)) {
-        owl_function_debugmsg("faimtest_parse_incoming_im_chan1: icbm: multipart:   part %d: charset 0x%04x, subset 0x%04x, msg = %s\n", z, sec->charset, sec->charsubset, sec->data);
-      } else {
-        owl_function_debugmsg("faimtest_parse_incoming_im_chan1: icbm: multipart:   part %d: charset 0x%04x, subset 0x%04x, binary or UNICODE data\n", z, sec->charset, sec->charsubset);
-      }
-    }
-  }
-  
-  if (args->icbmflags & AIM_IMFLAGS_HASICON) {
-    /* aim_send_im(sess, userinfo->sn, AIM_IMFLAGS_BUDDYREQ, "You have an icon"); */
-    owl_function_debugmsg("faimtest_parse_incoming_im_chan1: icbm: their icon: iconstamp = %ld, iconlen = 0x%08x, iconsum = 0x%04x\n", args->iconstamp, args->iconlen, args->iconsum);
-  }
-
-  g_free(realmsg);
-
-  return(1);
+  /* TODO: Multipart? See history from before 1.8 release. */
 }