From 56150eba928407b5b1410219292caf095db5ee63 Mon Sep 17 00:00:00 2001 From: nobu Date: Sun, 28 Jul 2013 01:54:30 +0000 Subject: win32.c: fix pipe name formatting * win32/win32.c (rb_w32_pipe): fix pipe name formatting. as "%x" may not contain '0' at all, fill at fixed position instead. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@42207 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- win32/win32.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'win32') diff --git a/win32/win32.c b/win32/win32.c index 291dc2aebe..55a73b4056 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -5654,8 +5654,12 @@ int rb_w32_pipe(int fds[2]) { static DWORD serial = 0; - char name[] = "\\\\.\\pipe\\ruby0000000000000000-0000000000000000"; - char *p; + static const char prefix[] = "\\\\.\\pipe\\ruby"; + const int width_of_prefix = (int)sizeof(prefix) - 1; + const int width_of_pid = (int)sizeof(rb_pid_t) * 2; + const int width_of_serial = (int)sizeof(serial) * 2; + const int width_of_ids = width_of_pid + 1 + width_of_serial + 1; + char name[sizeof(prefix) + width_of_ids]; SECURITY_ATTRIBUTES sec; HANDLE hRead, hWrite, h; int fdRead, fdWrite; @@ -5665,8 +5669,9 @@ rb_w32_pipe(int fds[2]) if (!cancel_io) return _pipe(fds, 65536L, _O_NOINHERIT); - p = strchr(name, '0'); - snprintf(p, strlen(p) + 1, "%"PRI_PIDT_PREFIX"x-%lx", rb_w32_getpid(), serial++); + memcpy(name, prefix, width_of_prefix); + snprintf(name + width_of_prefix, width_of_ids, "%.*"PRI_PIDT_PREFIX"x-%.*lx", + width_of_pid, rb_w32_getpid(), width_of_serial, serial++); sec.nLength = sizeof(sec); sec.lpSecurityDescriptor = NULL; -- cgit v1.2.3