removed FormatMessage/Win32 macro from log.h and added a

more portable wolf_system_error_msg in errors module

3 files changed, 56 insertions, 18 deletions

diff --git a/src/errors.c b/src/errors.c
index 5de1c6b..efd3f15 100644
--- a/src/errors.c
+++ b/src/errors.c
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2010 Andreas Baumann <abaumann@yahoo.com>
+    Copyright (C) 2008 Andreas Baumann <abaumann@yahoo.com>
 
     This program is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -18,7 +18,12 @@
 #include "errors.h"
 
 #include "port/gettext.h"		/* for localization */
-#include "port/string.h"		/* for strncpy */
+#include "port/string.h"		/* for strncpy, strerror_r */
+
+#if defined _WIN32
+#define WIN32_MEAN_AND_LEAN
+#include <windows.h>
+#endif /* defined _WIN32 */
 
 char *wolf_error_msg( const wolf_error_t error, char *buf, size_t buflen ) {
 	switch( error ) {
@@ -61,3 +66,36 @@ char *wolf_error_msg( const wolf_error_t error, char *buf, size_t buflen ) {
 	return buf;
 }
 
+char *wolf_system_error_msg( char *buf, size_t buflen ) {
+#ifdef _WIN32
+	LPTSTR errbuf;
+	DWORD errbuf_len;
+	DWORD res;
+	DWORD last_error;
+
+	last_error = GetLastError( );
+	
+	res = FormatMessage(
+		FORMAT_MESSAGE_ALLOCATE_BUFFER |
+		FORMAT_MESSAGE_FROM_SYSTEM |
+		FORMAT_MESSAGE_IGNORE_INSERTS |
+		FORMAT_MESSAGE_MAX_WIDTH_MASK,
+		NULL,			/* message is from system */
+		last_error,		/* there is a message with that id */
+		0,			/* default language preference */
+		(LPTSTR)&errbuf,	/* buffer allocated internally with LocalAlloc */
+		0,			/* minimum allocation size */
+		NULL );			/* no arguments */
+		
+	if( res == 0 ) {
+		strlcpy( buf, _( "No message available" ), buflen );
+	} else {
+		strlcpy( buf, errbuf, buflen );
+		LocalFree( errbuf );
+	}
+	
+	return buf;
+#else
+	(void)strerror_r( errno, buf, buflen );
+#endif
+}
diff --git a/src/library/loader.c b/src/library/loader.c
index 2f91278..106df7f 100644
--- a/src/library/loader.c
+++ b/src/library/loader.c
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2010 Andreas Baumann <abaumann@yahoo.com>
+    Copyright (C) 2008 Andreas Baumann <abaumann@yahoo.com>
 
     This program is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
diff --git a/src/service/service.c b/src/service/service.c
index 3b30c55..e63ce11 100644
--- a/src/service/service.c
+++ b/src/service/service.c
@@ -54,7 +54,7 @@ wolf_error_t wolf_service_install(	wolf_service_params_t params ) {
 		SC_MANAGER_ALL_ACCESS );
 	
 	if( scm == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_OPEN_SCM_FOR_INSTALL,
 			_( "Unable to open the service control manager to register service '%s': %s (%d)" ),
 			params.service_name, errbuf, GetLastError( ) );
@@ -68,7 +68,7 @@ wolf_error_t wolf_service_install(	wolf_service_params_t params ) {
 		MAX_PATH );
 
 	if( res == MAX_PATH ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_GET_PATH_OF_BINARY,
 			_( "Unable to get full path of the binary of service '%s': %s (%d)" ),
 			params.service_name, errbuf, GetLastError( ) );
@@ -93,7 +93,7 @@ wolf_error_t wolf_service_install(	wolf_service_params_t params ) {
 		NULL );
 
 	if( service == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_CREATE_SERVICE,
 			_( "Unable to create service '%s': %s (%d)" ),
 			params.service_name, errbuf, GetLastError( ) );
@@ -105,7 +105,7 @@ wolf_error_t wolf_service_install(	wolf_service_params_t params ) {
 	descr.lpDescription = (LPTSTR)params.service_description;
 	rt = ChangeServiceConfig2( service, SERVICE_CONFIG_DESCRIPTION, &descr );
 	if( !rt ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_SET_SERVICE_DESCRIPTION,
 			_( "Unable to set description of service '%s': %s (%d)" ),
 			params.service_name, errbuf, GetLastError( ) );
@@ -141,7 +141,7 @@ wolf_error_t wolf_service_remove(	LPCTSTR service_name ) {
 		SC_MANAGER_ALL_ACCESS );
 	
 	if( scm == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_OPEN_SCM_FOR_UNINSTALL,
 			_( "Unable to open the service control manager to uninstall '%s': %s (%d)" ),
 			service_name, errbuf, GetLastError( ) );
@@ -152,7 +152,7 @@ wolf_error_t wolf_service_remove(	LPCTSTR service_name ) {
 	service = OpenService( scm, service_name, SERVICE_ALL_ACCESS );
 	
 	if( service == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_OPEN_SERVICE,
 			_( "Unable to open service '%s': %s (%d)" ),
 			service_name, errbuf, GetLastError( ) );
@@ -162,7 +162,7 @@ wolf_error_t wolf_service_remove(	LPCTSTR service_name ) {
 
 	res = DeleteService( service );
 	if( res == 0 ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_DELETE_SERVICE,
 			_( "Unable to uninstall service '%s': %s (%d)" ),
 			service_name, errbuf, GetLastError( ) );
@@ -256,7 +256,7 @@ static void wolf_service_report_status(	DWORD current_state,
 
 	res = SetServiceStatus( service_status_handle, &service_status );
 	if( !res ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_REPORT_STATUS,
 			_( "Unable to report state '%s' (%d) of service '%s' to SCM '%s (%d)" ),
 			wolf_service_state_to_str( current_state ), current_state,
@@ -312,7 +312,7 @@ void WINAPI wolf_service_main( DWORD argc, LPTSTR *argv ) {
 		SERVICE_NAME, wolf_service_ctrl_handler );
 	
 	if( service_status_handle == 0 ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_REGISTER_SERVICE_CTRL_HANDLER,
 			_( "Unable to register service control handler function for service '%s': %s (%d)" ),
 			SERVICE_NAME, errbuf, GetLastError( ) );
@@ -333,7 +333,7 @@ void WINAPI wolf_service_main( DWORD argc, LPTSTR *argv ) {
 		FALSE,		/* not signalled */
 		NULL );		/* no name */
 	if( service_stop_event == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_CREATE_STOP_EVENT,
 			_( "Unable to create the stop event for service '%s': %s (%d)" ),
 			SERVICE_NAME, errbuf, GetLastError( ) );
@@ -369,7 +369,7 @@ wolf_service_event_t wolf_service_events_suspend_console( int timeout, wolf_erro
 			return WOLF_SERVICE_NO_EVENT;
 
 		default:
-			WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+			wolf_system_error_msg( errbuf, 512 );
 			wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_WAIT_FOR_OBJECT_FAILED,
 				_( "Waiting for events in the service '%s' failed: %s (%d)" ),
 				SERVICE_NAME, errbuf, GetLastError( ) );
@@ -395,7 +395,7 @@ wolf_service_event_t wolf_service_events_suspend_service( int timeout, wolf_erro
 			return WOLF_SERVICE_NO_EVENT;
 
 		default:
-			WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+			wolf_system_error_msg( errbuf, 512 );
 			wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_WAIT_FOR_OBJECT_FAILED,
 				_( "Waiting for events in the service '%s' failed: %s (%d)" ),
 				SERVICE_NAME, errbuf, GetLastError( ) );
@@ -457,7 +457,7 @@ wolf_error_t wolf_service_start_console( 	LPTSTR service_name,
 		FALSE,		/* not signalled */
 		NULL );		/* no name */
 	if( service_stop_event == NULL ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_CREATE_STOP_EVENT,
 			_( "Unable to create the stop event for service '%s': %s (%d)" ),
 			SERVICE_NAME, errbuf, GetLastError( ) );
@@ -467,7 +467,7 @@ wolf_error_t wolf_service_start_console( 	LPTSTR service_name,
 	/* catch console events for proper termination of the service in console mode */
 	res = SetConsoleCtrlHandler( wolf_service_console_ctrl_handler, TRUE );	
 	if( !res ) {
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_REGISTER_CONSOLE_CTRL_HANDLER,
 			_( "Unable to register the console control handler for foreground service '%s': %s (%d)" ),
 			service_name, errbuf, GetLastError( ) );
@@ -508,7 +508,7 @@ wolf_error_t wolf_service_start( 	LPTSTR service_name,
 			return WOLF_ERR_INVALID_STATE;
 		}
 
-		WOLF_LOG_GET_LAST_ERROR( GetLastError( ), errbuf, 512 );
+		wolf_system_error_msg( errbuf, 512 );
 		wolf_log( WOLF_LOG_ERR, WOLF_CATEGORY_SERVICE, WOLF_MSG_SERVICE_CANT_DISPATCH_SERVICE,
 			_( "Unable to dispatch service '%s': %s (%d)" ),
 			service_name, errbuf, GetLastError( ) );