From: Arnaud Giersch <arnaud.giersch@iut-bm.univ-fcomte.fr>
Date: Thu, 5 Jun 2014 20:43:08 +0000 (+0200)
Subject: Fix memory leaks.
X-Git-Tag: v3_12~993
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/0a30c2f40d805ac9a3b69fb7504a5d88eeb13b57?hp=4eec9abfb98b06e0d0e8922e423e51d0daae0a7d

Fix memory leaks.
---

diff --git a/examples/msg/cloud/master_worker_vm.c b/examples/msg/cloud/master_worker_vm.c
index 273af4c247..af24f85a88 100644
--- a/examples/msg/cloud/master_worker_vm.c
+++ b/examples/msg/cloud/master_worker_vm.c
@@ -14,6 +14,8 @@
 XBT_LOG_NEW_DEFAULT_CATEGORY(msg_test,
                              "Messages specific for this msg example");
 
+#define MAXMBOXLEN 64
+
 /** @addtogroup MSG_examples
  *
  *  - <b>cloud/masterslave_virtual_machines.c: Master/workers
@@ -139,8 +141,8 @@ int master_fun(int argc, char *argv[])
 
   XBT_INFO("# Shutdown the half of worker processes gracefuly. The remaining half will be forcibly killed.");
   for (i = 0; i < nb_workers; i++) {
-    char mbox[64];
-    sprintf(mbox, "MBOX:WRK%02d", i);
+    char mbox[MAXMBOXLEN];
+    snprintf(mbox, MAXMBOXLEN, "MBOX:WRK%02d", i);
     msg_task_t finalize = MSG_task_create("finalize", 0, 0, 0);
     MSG_task_send(finalize, mbox);
   }
@@ -167,7 +169,8 @@ int master_fun(int argc, char *argv[])
 int worker_fun(int argc, char *argv[])
 {
   const char *pr_name = MSG_process_get_name(MSG_process_self());
-  char *mbox = bprintf("MBOX:%s", pr_name);
+  char mbox[MAXMBOXLEN];
+  snprintf(mbox, MAXMBOXLEN, "MBOX:%s", pr_name);
 
   XBT_INFO("%s is listenning on mailbox(%s)", pr_name, mbox);
 
@@ -193,8 +196,6 @@ int worker_fun(int argc, char *argv[])
     MSG_task_destroy(task);
   }
 
-  free(mbox);
-
   return 0;
 }