Temporary variable service may leak memory at abnormal cases in function sm_service_table_add.

[YeHuiSheng]

In the function sm_service_table_add, the temporary variable service point to new memory by malloc.
However, if sm_service_go_active_exists or sm_service_enable_exists doesn't return SM_OKAY, the function sm_service_table_add will return before add the temporary variable into list _services.
Then, the temporary variable service will leak the memory which is malloc in the function sm_service_table_add.

So, we need to add free function in these abnormal cases.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/672896

[Ghada Khalil (gkhalil)]

Low priority / Not Gating - code cleanup suggestion / no noted system impact

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/672896
Committed: https://git.openstack.org/cgit/starlingx/ha/commit/?id=5696e1e56ddb906dfdd6fb59b01d3f65cb4a5441
Submitter: Zuul
Branch: master

commit 5696e1e56ddb906dfdd6fb59b01d3f65cb4a5441
Author: YeHuiSheng <email address hidden>
Date: Fri Jul 26 15:17:23 2019 +0800

    Modify memory leak in some abnormal cases by adding free function.

    Change-Id: I0076635d9c19d39637bf56a3404a4e4ad1f9506f
    Closes-Bug: #1837976
    Signed-off-by: YeHuiSheng <email address hidden>