Changeset 7d471c3

Timestamp:

Feb 10, 2009, 7:04:37 PM (15 years ago)

Author:

Sam Hartman <hartmans@mit.edu>

Branches:

master, debian, release-1.10, release-1.4, release-1.5, release-1.6, release-1.7, release-1.8, release-1.9

Children:

8412869

Parents:

6b7e7a5

git-author:

Sam Hartman <hartmans@mit.edu> (02/10/09 18:56:18)

git-committer:

Sam Hartman <hartmans@mit.edu> (02/10/09 19:04:37)

Message:

message.c: do not leak call to long_zuser

long_zuser returns a string that must be freed by the caller.
owl_message_set_recipient does not free its argument but instead
duplicates it.

File:

• message.c (modified) (1 diff)

message.c

@@ -964 +964 @@
   owl_message_set_instance(m, owl_zwrite_get_instance(&z));
   if (owl_zwrite_get_numrecips(&z)>0) {
+    char *longzuser = long_zuser(owl_zwrite_get_recip_n(&z, 0));
     owl_message_set_recipient(m,
-                              long_zuser(owl_zwrite_get_recip_n(&z, 0))); /* only gets the first user, must fix */
+                              longzuser); /* only gets the first user, must fix */
+    owl_free(longzuser);
   }
   owl_message_set_opcode(m, owl_zwrite_get_opcode(&z));