tests/subattr: Check return value of kdump_attr_ref_get()

The return value should never be assumed to be always KDUMP_OK.

Signed-off-by: Petr Tesarik <[email protected]>

Author: ptesarik

tests/subattr.c

@@ -73,7 +73,11 @@ main(int argc, char **argv)
 	status = kdump_sub_attr_ref(ctx, &ref, "uts", &subref);
 	if (status == KDUMP_OK) {
 		status = kdump_attr_ref_get(ctx, &subref, &attr);
-		if (attr.type != KDUMP_DIRECTORY) {
+		if (status != KDUMP_OK) {
+			fprintf(stderr, "kdump_attr_ref_get failed for %s: %s\n",
+				"uts", kdump_get_err(ctx));
+			rc = TEST_FAIL;
+		} else if (attr.type != KDUMP_DIRECTORY) {
 			fprintf(stderr, "Wrong type for %s: %d\n",
 				ATTRDIR, (int) attr.type);
 			rc = TEST_FAIL;
@@ -89,7 +93,11 @@ main(int argc, char **argv)
 	status = kdump_sub_attr_ref(ctx, &ref, "uts.sysname", &subref);
 	if (status == KDUMP_OK) {
 		status = kdump_attr_ref_get(ctx, &subref, &attr);
-		if (attr.type != KDUMP_STRING) {
+		if (status != KDUMP_OK) {
+			fprintf(stderr, "kdump_attr_ref_get failed for %s: %s\n",
+				"uts.sysname", kdump_get_err(ctx));
+			rc = TEST_FAIL;
+		} if (attr.type != KDUMP_STRING) {
 			fprintf(stderr, "Wrong type for %s: %d\n",
 				ATTRPATH, (int) attr.type);
 			rc = TEST_FAIL;