[Libstoragemgmt-devel] [PATCH] tester.c: Fix seg. fault if no systems returned

[Tony Asleson]

Avoid blowing up if the plug-in returns nothing. This is unexpected as
we are using simulators, but it can and does happen when the plug-in
has an issue.

Signed-off-by: Tony Asleson <***@redhat.com>
---
test/tester.c | 18 ++++++++++--------
1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/test/tester.c b/test/tester.c
index d5ce169..82e52fb 100644
--- a/test/tester.c
+++ b/test/tester.c
@@ -985,16 +985,18 @@ START_TEST(test_systems)
G(rc, lsm_system_list, c, &sys, &count, LSM_CLIENT_FLAG_RSVD);
fail_unless(count == 1);

- id = lsm_system_id_get(sys[0]);
- fail_unless(id != NULL);
- fail_unless(strcmp(id, SYSTEM_ID) == 0, "%s", id);
+ if( count ) {
+ id = lsm_system_id_get(sys[0]);
+ fail_unless(id != NULL);
+ fail_unless(strcmp(id, SYSTEM_ID) == 0, "%s", id);

- name = lsm_system_name_get(sys[0]);
- fail_unless(name != NULL);
- fail_unless(strcmp(name, SYSTEM_NAME) == 0);
+ name = lsm_system_name_get(sys[0]);
+ fail_unless(name != NULL);
+ fail_unless(strcmp(name, SYSTEM_NAME) == 0);

- status = lsm_system_status_get(sys[0]);
- fail_unless(status == LSM_SYSTEM_STATUS_OK, "status = %x", status);
+ status = lsm_system_status_get(sys[0]);
+ fail_unless(status == LSM_SYSTEM_STATUS_OK, "status = %x", status);
+ }

G(rc, lsm_system_record_array_free, sys, count);
}