Changeset 181ea08

Timestamp:

Feb 28, 2010, 9:55:08 PM (14 years ago)

Author:

Anders Kaseorg <andersk@mit.edu>

Branches:

master, release-1.10, release-1.6, release-1.7, release-1.8, release-1.9

Children:

ce5d181

Parents:

18fb86ee

git-author:

Anders Kaseorg <andersk@mit.edu> (02/28/10 16:41:01)

git-committer:

Anders Kaseorg <andersk@mit.edu> (02/28/10 21:55:08)

Message:

owl_command_aimwrite: Fix a buffer overflow on aimwrite -m.

I’ve tested that this works at least as well as the old code.  There’s
still something wrong here because aimwrite concatenates its arguments
to get the screenname and aimwrite -m doesn’t, but I don’t care
anymore because libfaim is going away (right?  RIGHT?).

Signed-off-by: Anders Kaseorg <andersk@mit.edu>
Reviewed-by: Sam Hartman <hartmans@mit.edu>

File:

• commands.c (modified) (1 diff)

commands.c

@@ -1921 +1921 @@
         newbuff[strlen(newbuff) - 1] = '\0'; /* remove last space */
 
-      recip=owl_malloc(strlen(argv[0])+5);
-      sprintf(recip, "%s ", argv[1]);
+      recip=owl_strdup(argv[1]);
       owl_aim_send_im(recip, newbuff);
       m=owl_function_make_outgoing_aim(newbuff, recip);