Skip to content

Fixed Crash in WinService::setFailureActions#2919

Merged
obiltschnig merged 1 commit intopocoproject:poco-1.10.1from
KevDi:bugfix/setFailureActionsCrashes
Feb 12, 2020
Merged

Fixed Crash in WinService::setFailureActions#2919
obiltschnig merged 1 commit intopocoproject:poco-1.10.1from
KevDi:bugfix/setFailureActionsCrashes

Conversation

@KevDi
Copy link
Copy Markdown
Contributor

@KevDi KevDi commented Feb 12, 2020

Currently there are 2 Situations in which the setFailureActions could crash.

  1. If the FailureActionVector has less then 3 Elements.
  2. If there is no SVC_REBOOT and SVC_RUN_COMMAND is specified. In that case the ac.lpCommand and ac.lpRebootMsg Parameters are not set.

This commit should fix boths of the Situations.
The Function now throws an InvalidArgument Exception if the Size of the Vector is larger then 3 Elements

less then 3 Elements. Also the Function crashes if no SVC_REBOOT or
SVC_RUN_COMMAND are specified. If this is the case ac.lpCommand and
ac.lpRebootMessage are not set.
@obiltschnig obiltschnig merged commit c484130 into pocoproject:poco-1.10.1 Feb 12, 2020
@obiltschnig obiltschnig added this to the Release 1.10.1 milestone Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants