Changeset 1154 for trunk/xcache/xc_shm_mmap.c

Timestamp:

2012-10-18T09:49:35+02:00 (7 months ago)

Author:

moo

Message:

avoid using rand() without seed. use getpid

File:

• trunk/xcache/xc_shm_mmap.c (modified) (2 diffs)

trunk/xcache/xc_shm_mmap.c

@@ -151 +151 @@
     int ro_ok;
     volatile void *romem;
-    char tmpname[sizeof(TMP_PATH) - 1 + 100];
+    char tmpname[sizeof(TMP_PATH) - 1 + 4 * 10 + 100] = { 0 };
     const char *errstr = NULL;
     const char *path = (const char *) arg1;
+    static int instanceId = 0;
 
     CHECK(shm = calloc(1, sizeof(xc_shm_t)), "shm OOM");
@@ -159 +160 @@
 
     if (path == NULL || !path[0]) {
-        static int inc = 0;
-        snprintf(tmpname, sizeof(tmpname) - 1, "%s.%d.%d.%d.%d", TMP_PATH, (int) getuid(), (int) getpid(), inc ++, rand());
+        snprintf(tmpname, sizeof(tmpname) - 1, "%s.%d.%d.%d", TMP_PATH, (int) getuid(), (int) getpid(), ++instanceId);
         path = tmpname;
     }
 #ifdef ZEND_WIN32
     else {
-        static int inc2 = 0;
-        snprintf(tmpname, sizeof(tmpname) - 1, "%s.%d.%d.%d.%d", path, (int) getuid(), (int) getpid(), inc2 ++, rand());
+        snprintf(tmpname, sizeof(tmpname) - 1, "%s.%d.%d.%d", path, (int) getuid(), (int) getpid(), ++instanceId);
         path = tmpname;
     }