file: Move win32 utf8->wchar open wrapper to libavutil
When libavformat was changed to use the new avpriv_open function
in 51eb213d00, this silently bypassed the existing wrapper for
win32. Move the win32 wrapper into libavutil/file.c to make sure
it gets called everywhere (not just in the libavformat case).
This makes sure that non-ascii file names gets opened properly
(where file names internally are stored as utf8, but they get
converted to wchar_t and opened with _wsopen).
Signed-off-by: Martin Storsjö <martin@martin.st>
			
			
This commit is contained in:
		@@ -27,46 +27,6 @@
 | 
				
			|||||||
#include "avformat.h"
 | 
					#include "avformat.h"
 | 
				
			||||||
#include "os_support.h"
 | 
					#include "os_support.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#if defined(_WIN32) && !defined(__MINGW32CE__)
 | 
					 | 
				
			||||||
#undef open
 | 
					 | 
				
			||||||
#undef lseek
 | 
					 | 
				
			||||||
#undef stat
 | 
					 | 
				
			||||||
#undef fstat
 | 
					 | 
				
			||||||
#include <fcntl.h>
 | 
					 | 
				
			||||||
#include <io.h>
 | 
					 | 
				
			||||||
#include <windows.h>
 | 
					 | 
				
			||||||
#include <share.h>
 | 
					 | 
				
			||||||
#include <errno.h>
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
int ff_win32_open(const char *filename_utf8, int oflag, int pmode)
 | 
					 | 
				
			||||||
{
 | 
					 | 
				
			||||||
    int fd;
 | 
					 | 
				
			||||||
    int num_chars;
 | 
					 | 
				
			||||||
    wchar_t *filename_w;
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    /* convert UTF-8 to wide chars */
 | 
					 | 
				
			||||||
    num_chars = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS, filename_utf8, -1, NULL, 0);
 | 
					 | 
				
			||||||
    if (num_chars <= 0)
 | 
					 | 
				
			||||||
        goto fallback;
 | 
					 | 
				
			||||||
    filename_w = av_mallocz(sizeof(wchar_t) * num_chars);
 | 
					 | 
				
			||||||
    if (!filename_w) {
 | 
					 | 
				
			||||||
        errno = ENOMEM;
 | 
					 | 
				
			||||||
        return -1;
 | 
					 | 
				
			||||||
    }
 | 
					 | 
				
			||||||
    MultiByteToWideChar(CP_UTF8, 0, filename_utf8, -1, filename_w, num_chars);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    fd = _wsopen(filename_w, oflag, SH_DENYNO, pmode);
 | 
					 | 
				
			||||||
    av_freep(&filename_w);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    if (fd != -1 || (oflag & O_CREAT))
 | 
					 | 
				
			||||||
        return fd;
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
fallback:
 | 
					 | 
				
			||||||
    /* filename may be be in CP_ACP */
 | 
					 | 
				
			||||||
    return _sopen(filename_utf8, oflag, SH_DENYNO, pmode);
 | 
					 | 
				
			||||||
}
 | 
					 | 
				
			||||||
#endif
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
#if CONFIG_NETWORK
 | 
					#if CONFIG_NETWORK
 | 
				
			||||||
#include <fcntl.h>
 | 
					#include <fcntl.h>
 | 
				
			||||||
#if !HAVE_POLL_H
 | 
					#if !HAVE_POLL_H
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -77,11 +77,6 @@ static inline int is_dos_path(const char *path)
 | 
				
			|||||||
#endif
 | 
					#endif
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#if defined(_WIN32) && !defined(__MINGW32CE__)
 | 
					 | 
				
			||||||
int ff_win32_open(const char *filename, int oflag, int pmode);
 | 
					 | 
				
			||||||
#define open ff_win32_open
 | 
					 | 
				
			||||||
#endif
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
#if CONFIG_NETWORK
 | 
					#if CONFIG_NETWORK
 | 
				
			||||||
#if !HAVE_SOCKLEN_T
 | 
					#if !HAVE_SOCKLEN_T
 | 
				
			||||||
typedef int socklen_t;
 | 
					typedef int socklen_t;
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -36,6 +36,45 @@
 | 
				
			|||||||
#include <windows.h>
 | 
					#include <windows.h>
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#if defined(_WIN32) && !defined(__MINGW32CE__)
 | 
				
			||||||
 | 
					#undef open
 | 
				
			||||||
 | 
					#undef lseek
 | 
				
			||||||
 | 
					#undef stat
 | 
				
			||||||
 | 
					#undef fstat
 | 
				
			||||||
 | 
					#include <windows.h>
 | 
				
			||||||
 | 
					#include <share.h>
 | 
				
			||||||
 | 
					#include <errno.h>
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static int win32_open(const char *filename_utf8, int oflag, int pmode)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
					    int fd;
 | 
				
			||||||
 | 
					    int num_chars;
 | 
				
			||||||
 | 
					    wchar_t *filename_w;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    /* convert UTF-8 to wide chars */
 | 
				
			||||||
 | 
					    num_chars = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS, filename_utf8, -1, NULL, 0);
 | 
				
			||||||
 | 
					    if (num_chars <= 0)
 | 
				
			||||||
 | 
					        goto fallback;
 | 
				
			||||||
 | 
					    filename_w = av_mallocz(sizeof(wchar_t) * num_chars);
 | 
				
			||||||
 | 
					    if (!filename_w) {
 | 
				
			||||||
 | 
					        errno = ENOMEM;
 | 
				
			||||||
 | 
					        return -1;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					    MultiByteToWideChar(CP_UTF8, 0, filename_utf8, -1, filename_w, num_chars);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    fd = _wsopen(filename_w, oflag, SH_DENYNO, pmode);
 | 
				
			||||||
 | 
					    av_freep(&filename_w);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    if (fd != -1 || (oflag & O_CREAT))
 | 
				
			||||||
 | 
					        return fd;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					fallback:
 | 
				
			||||||
 | 
					    /* filename may be be in CP_ACP */
 | 
				
			||||||
 | 
					    return _sopen(filename_utf8, oflag, SH_DENYNO, pmode);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					#define open win32_open
 | 
				
			||||||
 | 
					#endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int avpriv_open(const char *filename, int flags, ...)
 | 
					int avpriv_open(const char *filename, int flags, ...)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    int fd;
 | 
					    int fd;
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user