PATCH: advapi32 - services - Code cleaning and removing bugs

Created an attachment (id=7803)
Patch

What it's done here :

• Unneeded checks are removed :

if (*lpServicesReturned > 0)
{
for (dwCount = 0; dwCount < *lpServicesReturned; dwCount++)
...

now becomes:

for (dwCount = 0; dwCount < *lpServicesReturned; dwCount++)
...

• In EnumServicesStatusA :

• A buggy check is removed, instead REnumServicesStatusA performs a valid version of this test (via REnumServicesStatusW and REnumServiceGroupW, see the next bug report).
• Bad used type ENUM_SERVICE_STATUSW becomes ENUM_SERVICE_STATUSA.

• In EnumServicesStatusW :
  We use the implementation of EnumServiceGroupW by passing NULL for the group name (the same thing is done in the SCM).

• In EnumServicesStatusExW :
  Check whether the SC database handle is valid before performing any operations.