Description
In src/Servy.Core/Services/ServiceManager.cs (line 876):
catch { /* Fallback to automatic if access denied */ }
Inside the Parallel.ForEach in GetAllServices(), the service.StartType access can throw. The bare catch (not filtered to Win32Exception) silently defaults the startup type to Automatic regardless of the actual exception type.
A service that is actually Disabled will show as Automatic in the Manager UI. There is no logging.
Severity
Warning — UI displays incorrect startup type; no diagnostic trail.
Suggested fix
- Filter to
Win32Exception specifically
- Log the exception at Debug level
- Consider using a nullable or "Unknown" value instead of defaulting to Automatic
Description
In
src/Servy.Core/Services/ServiceManager.cs(line 876):Inside the
Parallel.ForEachinGetAllServices(), theservice.StartTypeaccess can throw. The bare catch (not filtered toWin32Exception) silently defaults the startup type toAutomaticregardless of the actual exception type.A service that is actually
Disabledwill show asAutomaticin the Manager UI. There is no logging.Severity
Warning — UI displays incorrect startup type; no diagnostic trail.
Suggested fix
Win32Exceptionspecifically