[janderwald] 29453: - randomize named pipe name - remove hardcoded path in named pipe reader class

Author: janderwald
Date: Mon Oct  8 13:01:33 2007
New Revision: 29453

URL: http://svn.reactos.org/svn/reactos?rev=29453&view=rev
Log:
- randomize named pipe name
- remove hardcoded path in named pipe reader class

Modified:
    trunk/reactos/tools/sysreg/namedpipe_reader.cpp
    trunk/reactos/tools/sysreg/rosboot_test.cpp

Modified: trunk/reactos/tools/sysreg/namedpipe_reader.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/sysreg/namedpipe_read...
==============================================================================
--- trunk/reactos/tools/sysreg/namedpipe_reader.cpp (original)
+++ trunk/reactos/tools/sysreg/namedpipe_reader.cpp Mon Oct  8 13:01:33 2007
@@ -50,7 +50,7 @@
    		return false;
    	}
 #ifndef __LINUX__
-		h_Pipe = CreateFile("\\.\pipe\qemu", //PipeCmd.c_str(),
+		h_Pipe = CreateFile(PipeCmd.c_str(),
    		                GENERIC_WRITE | GENERIC_READ,
    		                0,
    		                NULL,
@@ -139,7 +139,7 @@
         long  offset = 0;
         size_t start_size = vect.size ();
         char * start = buffer;
-        buffer[cbRead] = _T('\0');
+        buffer[cbRead] = '\0';
    	char * end = strstr(buffer, s_LineBreak);

//cout << "extractLines entered with append_line: " << append_line << " cbRead: " << cbRead << "buffer: " << buffer << endl;
@@ -148,7 +148,7 @@
         {
             if (end)
             {
-                end[0] = _T('\0');
+                end[0] = '\0';
                 string line = start;
                 end += (sizeof(s_LineBreak) / sizeof(char));
                 start = end;

Modified: trunk/reactos/tools/sysreg/rosboot_test.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/sysreg/rosboot_test.c...
==============================================================================
--- trunk/reactos/tools/sysreg/rosboot_test.cpp (original)
+++ trunk/reactos/tools/sysreg/rosboot_test.cpp Mon Oct  8 13:01:33 2007
@@ -27,7 +27,8 @@
 #include <assert.h>
 #include <math.h>
 #include <signal.h>
-			
+#include <io.h>
+
 namespace Sysreg_
 {
    using std::vector;
@@ -273,7 +274,7 @@
     {
         string pipe;
         string qemudir;
-
+        char pipename[] = "qemuXXXXXX";
         if (m_MaxMem.length() == 0)
         {
             /* set default memory size to 64M */
@@ -281,14 +282,35 @@
         }

#ifdef __LINUX__
-        pipe = "pipe:/tmp/qemu";
-		m_Src = "/tmp/qemu";
+
+        if (_mktemp(pipename))
+        {
+            string temp = pipename;
+            m_Src = "/tmp/" + temp;
+            pipe = "pipe:" + m_Src;
+        }
+        else
+        {
+            pipe = "pipe:/tmp/qemu";
+		    m_Src = "/tmp/qemu";
+        }
+
         qemudir = "/usr/share/qemu";
         m_DebugPort = "pipe";
 #else		
-		pipe = "pipe:qemu";
-		m_Src = "\\.\pipe\qemu";
+        if (_mktemp(pipename))
+        {
+            string temp = pipename;
+    		pipe = "pipe:" + temp;
+		    m_Src = "\\.\pipe\" + temp;
+        }
+        else
+        {
+    		pipe = "pipe:qemu";
+		    m_Src = "\\.\pipe\qemu";
+        }
         m_DebugPort = "pipe";
+
         if (!getQemuDir(qemudir))
         {
             return false;