sc: Don't silently fail when something doesn't work as expected.

Signed-off-by: Dmitry Timoshkov <dmitry@baikal.ru>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Dmitry Timoshkov 2022-03-18 17:10:52 +03:00 committed by Alexandre Julliard
parent 8b38c91d83
commit 1447872b0c
1 changed files with 9 additions and 9 deletions

View File

@ -207,7 +207,7 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
} }
if (!parse_create_params( argc - 3, argv + 3, &cp )) if (!parse_create_params( argc - 3, argv + 3, &cp ))
{ {
WINE_WARN("failed to parse create parameters\n"); WINE_ERR("failed to parse create parameters\n");
CloseServiceHandle( manager ); CloseServiceHandle( manager );
return 1; return 1;
} }
@ -219,7 +219,7 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
CloseServiceHandle( service ); CloseServiceHandle( service );
ret = TRUE; ret = TRUE;
} }
else WINE_TRACE("failed to create service %lu\n", GetLastError()); else WINE_ERR("failed to create service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"description" )) else if (!wcsicmp( argv[1], L"description" ))
{ {
@ -229,10 +229,10 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
SERVICE_DESCRIPTIONW sd; SERVICE_DESCRIPTIONW sd;
sd.lpDescription = argc > 3 ? (WCHAR *)argv[3] : NULL; sd.lpDescription = argc > 3 ? (WCHAR *)argv[3] : NULL;
ret = ChangeServiceConfig2W( service, SERVICE_CONFIG_DESCRIPTION, &sd ); ret = ChangeServiceConfig2W( service, SERVICE_CONFIG_DESCRIPTION, &sd );
if (!ret) WINE_TRACE("failed to set service description %lu\n", GetLastError()); if (!ret) WINE_ERR("failed to set service description %lu\n", GetLastError());
CloseServiceHandle( service ); CloseServiceHandle( service );
} }
else WINE_TRACE("failed to open service %lu\n", GetLastError()); else WINE_ERR("failed to open service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"failure" )) else if (!wcsicmp( argv[1], L"failure" ))
{ {
@ -247,10 +247,10 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
HeapFree( GetProcessHeap(), 0, sfa.lpsaActions ); HeapFree( GetProcessHeap(), 0, sfa.lpsaActions );
} }
else else
WINE_WARN("failed to parse failure parameters\n"); WINE_ERR("failed to parse failure parameters\n");
CloseServiceHandle( service ); CloseServiceHandle( service );
} }
else WINE_TRACE("failed to open service %lu\n", GetLastError()); else WINE_ERR("failed to open service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"delete" )) else if (!wcsicmp( argv[1], L"delete" ))
{ {
@ -261,7 +261,7 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
if (!ret) WINE_TRACE("failed to delete service %lu\n", GetLastError()); if (!ret) WINE_TRACE("failed to delete service %lu\n", GetLastError());
CloseServiceHandle( service ); CloseServiceHandle( service );
} }
else WINE_TRACE("failed to open service %lu\n", GetLastError()); else WINE_ERR("failed to open service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"start" )) else if (!wcsicmp( argv[1], L"start" ))
{ {
@ -272,7 +272,7 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
if (!ret) WINE_TRACE("failed to start service %lu\n", GetLastError()); if (!ret) WINE_TRACE("failed to start service %lu\n", GetLastError());
CloseServiceHandle( service ); CloseServiceHandle( service );
} }
else WINE_TRACE("failed to open service %lu\n", GetLastError()); else WINE_ERR("failed to open service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"stop" )) else if (!wcsicmp( argv[1], L"stop" ))
{ {
@ -283,7 +283,7 @@ int __cdecl wmain( int argc, const WCHAR *argv[] )
if (!ret) WINE_TRACE("failed to stop service %lu\n", GetLastError()); if (!ret) WINE_TRACE("failed to stop service %lu\n", GetLastError());
CloseServiceHandle( service ); CloseServiceHandle( service );
} }
else WINE_TRACE("failed to open service %lu\n", GetLastError()); else WINE_ERR("failed to open service %lu\n", GetLastError());
} }
else if (!wcsicmp( argv[1], L"sdset" )) else if (!wcsicmp( argv[1], L"sdset" ))
{ {