diff options
Diffstat (limited to 'odk/source')
-rw-r--r-- | odk/source/unoapploader/unx/unoapploader.c | 26 | ||||
-rw-r--r-- | odk/source/unoapploader/win/unoapploader.c | 211 |
2 files changed, 111 insertions, 126 deletions
diff --git a/odk/source/unoapploader/unx/unoapploader.c b/odk/source/unoapploader/unx/unoapploader.c index e8c9a076fdb5..e0013e97b7a1 100644 --- a/odk/source/unoapploader/unx/unoapploader.c +++ b/odk/source/unoapploader/unx/unoapploader.c @@ -32,7 +32,7 @@ #include "rtl/string.h" #include "sal/types.h" -char const* getPath(void); +char* getPath(void); char* createCommandName( char* argv0 ); static const int SEPARATOR = '/'; @@ -63,7 +63,7 @@ static const char* PATHSEPARATOR = ":"; */ int main( int argc, char *argv[] ) { - char const* path; + char* path; char* cmdname; (void) argc; /* avoid warning about unused parameter */ @@ -80,9 +80,7 @@ int main( int argc, char *argv[] ) #else static const char* ENVVARNAME = "LD_LIBRARY_PATH"; #endif - char * libpath; - int freeLibpath; - + char* libpath; char* value; char* envstr; int size; @@ -96,6 +94,7 @@ int main( int argc, char *argv[] ) pathlen + RTL_CONSTASCII_LENGTH(unoinfoSuffix) + 1); /*TODO: overflow */ if (unoinfo == NULL) { + free(path); fprintf(stderr, "Error: out of memory!\n"); exit(EXIT_FAILURE); } @@ -168,13 +167,12 @@ int main( int argc, char *argv[] ) fprintf(stderr, "Error: executing unoinfo failed!\n"); exit(EXIT_FAILURE); } - freeLibpath = 1; + free(path); } else { /* Assume an old OOo 2.x installation without unoinfo: */ - libpath = (char *) path; - freeLibpath = 0; + libpath = path; } value = getenv( ENVVARNAME ); @@ -196,10 +194,7 @@ int main( int argc, char *argv[] ) strcat( envstr, "=" ); #endif strcat( envstr, libpath ); - if ( freeLibpath != 0 ) - { - free( libpath ); - } + free( libpath ); if ( value != NULL ) { strcat( envstr, PATHSEPARATOR ); @@ -233,11 +228,12 @@ int main( int argc, char *argv[] ) * Gets the path of a UNO installation. * * @return the installation path or NULL, if no installation was specified or - * found, or if an error occurred + * found, or if an error occurred. + * Returned pointer must be released with free() */ -char const* getPath(void) +char* getPath(void) { - char const* path = cppuhelper_detail_findSofficePath(); + char* path = cppuhelper_detail_findSofficePath(); if ( path == NULL ) { diff --git a/odk/source/unoapploader/win/unoapploader.c b/odk/source/unoapploader/win/unoapploader.c index cabcfedf62ef..4ad8685924e3 100644 --- a/odk/source/unoapploader/win/unoapploader.c +++ b/odk/source/unoapploader/win/unoapploader.c @@ -36,10 +36,11 @@ #include "cppuhelper/findsofficepath.h" #include "sal/types.h" -#define MY_LENGTH(s) (sizeof (s) / sizeof *(s) - 1) +#define MY_SIZE(s) (sizeof (s) / sizeof *(s)) +#define MY_LENGTH(s) (MY_SIZE(s) - 1) -char const* getPath(void); -char* createCommandLine( char const * lpCmdLine ); +wchar_t* getPath(void); +wchar_t* createCommandLine( wchar_t const * lpCmdLine ); FILE* getErrorFile( int create ); void writeError( const char* errstr ); void closeErrorFile(void); @@ -64,28 +65,15 @@ void closeErrorFile(void); * root key HKEY_CURRENT_USER in the Windows Registry. If this key is missing, * the key is read from the root key HKEY_LOCAL_MACHINE.</p> */ -int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, - LPSTR lpCmdLine, int nCmdShow ) +int WINAPI wWinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, + LPWSTR lpCmdLine, int nCmdShow ) { - const char* ENVVARNAME = "PATH"; - const char* PATHSEPARATOR = ";"; - - char const* path = NULL; - char path2[MAX_PATH]; - char* value = NULL; - char* envstr = NULL; - char* cmdline = NULL; - size_t size; - STARTUPINFO startup_info; - PROCESS_INFORMATION process_info; - BOOL bCreate; - (void) hInstance; /* unused */ (void) hPrevInstance; /* unused */ (void) nCmdShow; /* unused */ /* get the path of the UNO installation */ - path = getPath(); + wchar_t* path = getPath(); if ( path != NULL ) { @@ -93,56 +81,54 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, MY_LENGTH(L"\"") + MAX_PATH + MY_LENGTH(L"\\unoinfo.exe\" c++")]; /* hopefully does not overflow */ - int pathsize; - SECURITY_ATTRIBUTES sec; - HANDLE temp; - HANDLE stdoutRead; - HANDLE stdoutWrite; - STARTUPINFOW startinfo; - PROCESS_INFORMATION procinfo; - int ret; cmd[0] = L'"'; - pathsize = MultiByteToWideChar(CP_ACP, 0, path, -1, cmd + 1, MAX_PATH); - if (pathsize == 0) { - writeError("Error: MultiByteToWideChar failed!\n"); - closeErrorFile(); - return 1; - } + wcscpy(cmd + 1, path); if (wcschr(cmd + 1, L'"') != NULL) { + free(path); writeError("Error: bad characters in UNO installation path!\n"); closeErrorFile(); return 1; } + size_t pathsize = wcslen(cmd); wcscpy( cmd + pathsize, &L"\\unoinfo.exe\" c++"[ pathsize == 1 || cmd[pathsize - 1] != L'\\' ? 0 : 1]); + SECURITY_ATTRIBUTES sec; sec.nLength = sizeof (SECURITY_ATTRIBUTES); sec.lpSecurityDescriptor = NULL; sec.bInheritHandle = TRUE; + HANDLE stdoutRead; + HANDLE stdoutWrite; + HANDLE temp; if (CreatePipe(&temp, &stdoutWrite, &sec, 0) == 0 || DuplicateHandle( GetCurrentProcess(), temp, GetCurrentProcess(), &stdoutRead, 0, FALSE, DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS) == 0) { + free(path); writeError("Error: CreatePipe/DuplicateHandle failed!\n"); closeErrorFile(); return 1; } - memset(&startinfo, 0, sizeof (STARTUPINFOW)); - startinfo.cb = sizeof (STARTUPINFOW); + STARTUPINFOW startinfo; + PROCESS_INFORMATION procinfo; + memset(&startinfo, 0, sizeof(startinfo)); + startinfo.cb = sizeof(startinfo); startinfo.lpDesktop = L""; startinfo.dwFlags = STARTF_USESTDHANDLES; startinfo.hStdOutput = stdoutWrite; - ret = CreateProcessW( + BOOL ret = CreateProcessW( NULL, cmd, NULL, NULL, TRUE, 0, NULL, NULL, &startinfo, &procinfo); - if (ret != 0) { + if (ret != FALSE) { + // Release result of GetPath() + free(path); + char * buf = NULL; char * tmp; DWORD n = 1000; DWORD k = 0; DWORD exitcode; - int path2size; CloseHandle(stdoutWrite); CloseHandle(procinfo.hThread); for (;;) { @@ -193,22 +179,19 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, closeErrorFile(); return 1; } - if (k == 0) { - path2size = 0; - } else { - path2size = WideCharToMultiByte( - CP_ACP, 0, (wchar_t *) buf, k / 2, path2, MAX_PATH - 1, - NULL, NULL); - if (path2size == 0) { - writeError("Error: converting unoinfo output failed!\n"); - closeErrorFile(); - return 1; - } + path = (wchar_t*)realloc(buf, k + sizeof(wchar_t)); + if (path == NULL) + { + free(buf); + writeError( + "Error: out of memory zero-terminating unoinfo output!\n"); + closeErrorFile(); + return 1; } - path2[path2size] = '\0'; - path = path2; + path[k / 2] = L'\0'; } else { if (GetLastError() != ERROR_FILE_NOT_FOUND) { + free(path); writeError("Error: calling unoinfo failed!\n"); closeErrorFile(); return 1; @@ -218,27 +201,30 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, } /* get the value of the PATH environment variable */ - value = getenv( ENVVARNAME ); + const wchar_t* ENVVARNAME = L"PATH"; + const wchar_t* PATHSEPARATOR = L";"; + wchar_t* value = _wgetenv( ENVVARNAME ); /* * add the UNO installation path to the PATH environment variable; * note that this only affects the environment variable of the current * process, the command processor's environment is not changed */ - size = strlen( ENVVARNAME ) + strlen( "=" ) + strlen( path ) + 1; + size_t size = wcslen( ENVVARNAME ) + wcslen( L"=" ) + wcslen( path ) + 1; if ( value != NULL ) - size += strlen( PATHSEPARATOR ) + strlen( value ); - envstr = (char*) malloc( size ); - strcpy( envstr, ENVVARNAME ); - strcat( envstr, "=" ); - strcat( envstr, path ); + size += wcslen( PATHSEPARATOR ) + wcslen( value ); + wchar_t* envstr = (wchar_t*) malloc( size*sizeof(wchar_t) ); + wcscpy( envstr, ENVVARNAME ); + wcscat( envstr, L"=" ); + wcscat( envstr, path ); if ( value != NULL ) { - strcat( envstr, PATHSEPARATOR ); - strcat( envstr, value ); + wcscat( envstr, PATHSEPARATOR ); + wcscat( envstr, value ); } - _putenv( envstr ); + _wputenv( envstr ); free( envstr ); + free( path ); } else { @@ -246,7 +232,7 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, } /* create the command line for the application process */ - cmdline = createCommandLine( lpCmdLine ); + wchar_t* cmdline = createCommandLine( lpCmdLine ); if ( cmdline == NULL ) { writeError( "Error: cannot create command line!\n" ); @@ -255,10 +241,12 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, } /* create the application process */ - memset( &startup_info, 0, sizeof( STARTUPINFO ) ); - startup_info.cb = sizeof( STARTUPINFO ); - bCreate = CreateProcess( NULL, cmdline, NULL, NULL, FALSE, 0, NULL, NULL, - &startup_info, &process_info ); + STARTUPINFOW startup_info; + PROCESS_INFORMATION process_info; + memset( &startup_info, 0, sizeof(startup_info) ); + startup_info.cb = sizeof(startup_info); + BOOL bCreate = CreateProcessW( NULL, cmdline, NULL, NULL, FALSE, 0, NULL, NULL, + &startup_info, &process_info ); free( cmdline ); if ( !bCreate ) { @@ -277,11 +265,12 @@ int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, * Gets the path of a UNO installation. * * @return the installation path or NULL, if no installation was specified or - * found, or if an error occurred + * found, or if an error occurred. + * Returned pointer must be released with free() */ -char const* getPath() +wchar_t* getPath() { - char const* path = cppuhelper_detail_findSofficePath(); + wchar_t* path = cppuhelper_detail_findSofficePath(); if ( path == NULL ) writeError( "Warning: getting path from Windows Registry failed!\n" ); @@ -302,38 +291,38 @@ char const* getPath() * @return the command line for the application process or NULL, if an error * occurred */ -char* createCommandLine( char const * appendix ) +wchar_t* createCommandLine( wchar_t const * appendix ) { - const char* CMDPREFIX = "_"; - const char* DQUOTE = "\""; - const char* SPACE = " "; + const wchar_t* CMDPREFIX = L"_"; + const wchar_t* DQUOTE = L"\""; + const wchar_t* SPACE = L" "; - char* cmdline = NULL; + wchar_t* cmdline = NULL; - char cmdname[ _MAX_PATH ]; - char drive[ _MAX_DRIVE ]; - char dir[ _MAX_PATH ]; - char base[ _MAX_FNAME ]; - char newbase[ _MAX_FNAME ]; - char ext[ _MAX_EXT ]; + wchar_t cmdname[ _MAX_PATH ]; + wchar_t drive[ _MAX_DRIVE ]; + wchar_t dir[ _MAX_PATH ]; + wchar_t base[ _MAX_FNAME ]; + wchar_t newbase[ _MAX_FNAME ]; + wchar_t ext[ _MAX_EXT ]; /* get the absolute path of the executable file */ - if ( GetModuleFileName( NULL, cmdname, sizeof( cmdname ) ) ) + if ( GetModuleFileNameW( NULL, cmdname, MY_SIZE( cmdname ) ) ) { /* prefix the executable file name by '_' */ - _splitpath( cmdname, drive, dir, base, ext ); - strcpy( newbase, CMDPREFIX ); - strcat( newbase, base ); - _makepath( cmdname, drive, dir, newbase, ext ); + _wsplitpath( cmdname, drive, dir, base, ext ); + wcscpy( newbase, CMDPREFIX ); + wcscat( newbase, base ); + _wmakepath( cmdname, drive, dir, newbase, ext ); /* create the command line */ - cmdline = (char*) malloc( strlen( DQUOTE ) + strlen( cmdname ) + - strlen ( DQUOTE ) + strlen( SPACE ) + strlen( appendix ) + 1 ); - strcpy( cmdline, DQUOTE ); - strcat( cmdline, cmdname ); - strcat( cmdline, DQUOTE ); - strcat( cmdline, SPACE ); - strcat( cmdline, appendix ); + cmdline = (wchar_t*) malloc( (wcslen( DQUOTE ) + wcslen( cmdname ) + + wcslen ( DQUOTE ) + wcslen( SPACE ) + wcslen( appendix ) + 1) * sizeof(wchar_t) ); + wcscpy( cmdline, DQUOTE ); + wcscat( cmdline, cmdname ); + wcscat( cmdline, DQUOTE ); + wcscat( cmdline, SPACE ); + wcscat( cmdline, appendix ); } return cmdline; @@ -356,38 +345,38 @@ char* createCommandLine( char const * appendix ) */ FILE* getErrorFile( int create ) { - const char* MODE = "w"; - const char* BASEPOSTFIX = "-error"; - const char* EXTENSION = ".log"; + const wchar_t* MODE = L"w"; + const wchar_t* BASEPOSTFIX = L"-error"; + const wchar_t* EXTENSION = L".log"; static FILE* ferr = NULL; - char fname[ _MAX_PATH ]; - char drive[ _MAX_DRIVE ]; - char dir[ _MAX_PATH ]; - char base[ _MAX_FNAME ]; - char newbase[ _MAX_FNAME ]; - char ext[ _MAX_EXT ]; + wchar_t fname[ _MAX_PATH ]; + wchar_t drive[ _MAX_DRIVE ]; + wchar_t dir[ _MAX_PATH ]; + wchar_t base[ _MAX_FNAME ]; + wchar_t newbase[ _MAX_FNAME ]; + wchar_t ext[ _MAX_EXT ]; if ( ferr == NULL && create ) { /* get the absolute path of the executable file */ - if ( GetModuleFileName( NULL, fname, sizeof( fname ) ) ) + if ( GetModuleFileNameW( NULL, fname, MY_SIZE( fname ) ) ) { /* create error file in the directory of the executable file */ - _splitpath( fname, drive, dir, base, ext ); - strcpy( newbase, base ); - strcat( newbase, BASEPOSTFIX ); - _makepath( fname, drive, dir, newbase, EXTENSION ); - ferr = fopen( fname, MODE ); + _wsplitpath( fname, drive, dir, base, ext ); + wcscpy( newbase, base ); + wcscat( newbase, BASEPOSTFIX ); + _wmakepath( fname, drive, dir, newbase, EXTENSION ); + ferr = _wfopen( fname, MODE ); if ( ferr == NULL ) { /* create error file in the temp directory */ - GetTempPath( sizeof( fname ), fname ); - strcat( fname, newbase ); - strcat( fname, EXTENSION ); - ferr = fopen( fname, MODE ); + GetTempPathW(MY_SIZE( fname ), fname ); + wcscat( fname, newbase ); + wcscat( fname, EXTENSION ); + ferr = _wfopen( fname, MODE ); } } } |