[ros-diffs] [janderwald] 37082: - Clean up resources on failure

[janderwald at svn.reactos.org]

Author: janderwald
Date: Thu Oct 30 03:23:53 2008
New Revision: 37082

URL: http://svn.reactos.org/svn/reactos?rev=37082&view=rev
Log:
- Clean up resources on failure

Modified:
    trunk/reactos/base/applications/msconfig/srvpage.c

Modified: trunk/reactos/base/applications/msconfig/srvpage.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/msconfig/srvpage.c?rev=37082&r1=37081&r2=37082&view=diff
==============================================================================
--- trunk/reactos/base/applications/msconfig/srvpage.c [iso-8859-1] (original)
+++ trunk/reactos/base/applications/msconfig/srvpage.c [iso-8859-1] Thu Oct 30 03:23:53 2008
@@ -145,7 +145,7 @@
                 if (hService != INVALID_HANDLE_VALUE)
                 {
                     /* check if service is required by the system*/
-                    if (!QueryServiceConfig2(hService, SERVICE_CONFIG_FAILURE_ACTIONS, (LPBYTE)pServiceFailureActions, 0, &BytesNeeded))
+                    if (!QueryServiceConfig2(hService, SERVICE_CONFIG_FAILURE_ACTIONS, (LPBYTE)NULL, 0, &BytesNeeded))
                     {
                         if (GetLastError() == ERROR_INSUFFICIENT_BUFFER)
                         {
@@ -156,11 +156,17 @@
                             if (!QueryServiceConfig2(hService, SERVICE_CONFIG_FAILURE_ACTIONS, (LPBYTE)pServiceFailureActions, BytesNeeded, &BytesNeeded))
                             {
                                 HeapFree(GetProcessHeap(), 0, pServiceFailureActions);
+                                HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                                CloseServiceHandle(hService);
+                                CloseServiceHandle(ScHandle);
                                 return;
                             }
                         }
                         else /* exit on failure */
                         {
+                            HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                            CloseServiceHandle(hService);
+                            CloseServiceHandle(ScHandle);
                             return;
                         }
                     }
@@ -183,22 +189,32 @@
 
                     /* get vendor of service binary */
                     BytesNeeded = 0;
-                    if (!QueryServiceConfig(hService, pServiceConfig, 0, &BytesNeeded))
+                    if (!QueryServiceConfig(hService, NULL, 0, &BytesNeeded))
                     {
                         if (GetLastError() == ERROR_INSUFFICIENT_BUFFER)
                         {
                             pServiceConfig = (LPQUERY_SERVICE_CONFIG) HeapAlloc(GetProcessHeap(), 0, BytesNeeded);
                             if (pServiceConfig == NULL)
-                                return;
-
+                            {
+                                HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                                CloseServiceHandle(hService);
+                                CloseServiceHandle(ScHandle);
+                                return;
+                            }
                             if (!QueryServiceConfig(hService, pServiceConfig, BytesNeeded, &BytesNeeded))
                             {
                                 HeapFree(GetProcessHeap(), 0, pServiceConfig);
+                                HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                                CloseServiceHandle(hService);
+                                CloseServiceHandle(ScHandle);
                                 return;
                             }
                         }
                         else /* exit on failure */
                         {
+                            HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                            CloseServiceHandle(hService);
+                            CloseServiceHandle(ScHandle);
                             return;
                         }
                     }
@@ -221,11 +237,18 @@
                     {
                         lpData = (TCHAR*) HeapAlloc(GetProcessHeap(), 0, dwLen);
                         if (lpData == NULL)
-                            return;
-
+                        {
+                            HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                            CloseServiceHandle(hService);
+                            CloseServiceHandle(ScHandle);
+                            return;
+                        }
                         if (!GetFileVersionInfo (FileName, dwHandle, dwLen, lpData))
                         {
                             HeapFree(GetProcessHeap(), 0, lpData);
+                            HeapFree(GetProcessHeap(), 0, pServiceStatus);
+                            CloseServiceHandle(hService);
+                            CloseServiceHandle(ScHandle);
                             return;
                         }