diff options
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | win32/win32.c | 26 |
2 files changed, 14 insertions, 16 deletions
@@ -1,3 +1,7 @@ +Tue Mar 27 17:08:08 2012 Kazuhiro NISHIYAMA <zn@mbf.nifty.com> + + * win32/win32.c (check_if_dir): fix memory leak. + Tue Mar 27 13:13:51 2012 Nobuyoshi Nakada <nobu@ruby-lang.org> * string.c (str_new_empty): should copy also the encoding as an diff --git a/win32/win32.c b/win32/win32.c index c26d3e3cc7..310eb85340 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -5186,36 +5186,30 @@ rb_w32_uopen(const char *file, int oflag, ...) /* License: Ruby's */ static int -check_if_dir(const char *file) +check_if_wdir(const WCHAR *wfile) { - DWORD attr; - WCHAR *wfile; - - if (!(wfile = filecp_to_wstr(file, NULL))) - return FALSE; - attr = GetFileAttributesW(wfile); + DWORD attr = GetFileAttributesW(wfile); if (attr == (DWORD)-1L || !(attr & FILE_ATTRIBUTE_DIRECTORY) || check_valid_dir(wfile)) { return FALSE; } - free(wfile); errno = EISDIR; return TRUE; } /* License: Ruby's */ static int -check_if_wdir(const WCHAR *wfile) +check_if_dir(const char *file) { - DWORD attr = GetFileAttributesW(wfile); - if (attr == (DWORD)-1L || - !(attr & FILE_ATTRIBUTE_DIRECTORY) || - check_valid_dir(wfile)) { + WCHAR *wfile; + int ret; + + if (!(wfile = filecp_to_wstr(file, NULL))) return FALSE; - } - errno = EISDIR; - return TRUE; + ret = check_if_wdir(wfile); + free(wfile); + return ret; } /* License: Ruby's */ |