Fix referenced before assignment in sysvinit module

[MichaelMayorov]

SUMMARY

Using pattern keyword in service module leads to UnboundLocalError, e.g.

- name: Start redundant-router
  service: enabled=yes state=started name=redundant-router pattern=redundant-router

Here is ansible-playbook output for this error

fatal: [10-0-16-3.ams.flops.ru]: FAILED! => {
    "changed": false,
    "module_stderr": "Shared connection to 10-0-16-3.ams.flops.ru closed.\r\n",
    "module_stdout": "Traceback (most recent call last):\r\n  File \"/tmp/ansible_CSMBg8/ansible_module_sysvinit.py\", line 351, in <module>\r\n    main()\r\n  File \"/tmp/ansible_CSMBg8/ansible_module_sysvinit.py\", line 232, in main\r\n    if not worked:\r\nUnboundLocalError: local variable 'worked' referenced before assignment\r\n",
    "msg": "MODULE FAILURE",
    "rc": 1
}

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

lib/ansible/modules/system/sysvinit.py

ANSIBLE VERSION

2.6.1

ADDITIONAL INFORMATION

[pilou-]

shipit

[pilou-]

@bcoca review message needs to contains LGTM or shipit, otherwise ansibot ignores the review when counting the approvals.

[robinmulder]

This also affects 2.6.3, please backport to 2.6.

[resmo]

created a backport #45988

[resmo]

backport #45204