admin/libiscsi
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

test_writesame16_unmap: Do not leak memory if WRITESAME16 is not supported

Browse Source 
Detected by Valgrind.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
This commit is contained in:
Bart Van Assche
2013-09-29 21:06:29 +02:00
committed by Ronnie Sahlberg
parent 1337185fe9
commit e5ba667cea
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
test-tool/test_writesame16_unmap.c
View File

@@ -54,7 +54,7 @@ test_writesame16_unmap(void)
if (ret == -2) { if (ret == -2) {
logging(LOG_NORMAL, "[SKIPPED] WRITESAME16 is not implemented."); logging(LOG_NORMAL, "[SKIPPED] WRITESAME16 is not implemented.");
CU_PASS("[SKIPPED] Target does not support WRITESAME16. Skipping test"); CU_PASS("[SKIPPED] Target does not support WRITESAME16. Skipping test");
return; goto finished;
} }
CU_ASSERT_EQUAL(ret, 0); CU_ASSERT_EQUAL(ret, 0);
@@ -142,7 +142,7 @@ test_writesame16_unmap(void)
"BlockLimits VPD is missing."); "BlockLimits VPD is missing.");
CU_FAIL("[FAILED] WRITESAME16 works but " CU_FAIL("[FAILED] WRITESAME16 works but "
"BlockLimits VPD is missing."); "BlockLimits VPD is missing.");
return; goto finished;
} }
i = 256; i = 256;
@@ -247,5 +247,7 @@ test_writesame16_unmap(void)
0, 1, 0, 0, buf); 0, 1, 0, 0, buf);
CU_ASSERT_EQUAL(ret, 0); CU_ASSERT_EQUAL(ret, 0);
} }
finished:
free(buf); free(buf);
} }