Changeset 1151f0b

Timestamp:

Jan 10, 2008, 1:28:59 AM (16 years ago)

Author:

Nelson Elhage <nelhage@mit.edu>

Branches:

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

Children:

9506feb

Parents:

44f32fb

Message:

sprintf -> snprintf

File:

• zephyr.c (modified) (1 diff)

zephyr.c

@@ -563 +563 @@
     }
   } else if (!strcmp(retnotice->z_message, ZSRVACK_NOTSENT)) {
+    #define BUFFLEN 1024
     if (retnotice->z_recipient == NULL
         || *retnotice->z_recipient == NULL
         || *retnotice->z_recipient == '@') {
-      char buff[1024];
+      char buff[BUFFLEN];
       owl_function_error("No one subscribed to class %s", retnotice->z_class);
-      sprintf(buff, "Could not send message to class %s: no one subscribed.\n", retnotice->z_class);
+      snprintf(buff, BUFFLEN, "Could not send message to class %s: no one subscribed.\n", retnotice->z_class);
       owl_function_adminmsg("", buff);
     } else {
-      char buff[1024];
+      char buff[BUFFLEN];
       tmp = short_zuser(retnotice->z_recipient);
       owl_function_error("%s: Not logged in or subscribing.", tmp);
-      sprintf(buff, "Could not send message to %s: not logged in or subscribing to", tmp);
+      snprintf(buff, BUFFLEN, "Could not send message to %s: not logged in or subscribing to", tmp);
       if(strcmp(retnotice->z_class, "message")) {
-        sprintf(buff, "%s class %s, instance %s.\n", buff,
-                retnotice->z_class,
-                retnotice->z_class_inst);
+        snprintf(buff, BUFFLEN,
+                 "%s class %s, instance %s.\n", buff,
+                 retnotice->z_class,
+                 retnotice->z_class_inst);
       } else {
-        sprintf(buff, "%s messages.\n", buff);
+        snprintf(buff, BUFFLEN,
+                 "%s messages.\n", buff);
       }
       owl_function_adminmsg("", buff);