hugemmap/hugemmap05.c: fix warnings in cleanup()

1) We only destroy the shared memory segment if it was still created.
2) We only close and delete the test file if it was still created and opened.

---
 testcases/kernel/mem/hugetlb/hugemmap/hugemmap05.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

index 90389dd..c73a975 100644
@@ -178,9 +178,10 @@ static void test_overcommit(void)
 
 static void cleanup(void)
 {
-	if (opt_shmid) {
+	if (opt_shmid && shmid != -1)
 		SAFE_SHMCTL(shmid, IPC_RMID, NULL);
-	} else {
+
+	if (!opt_shmid && fd != -1) {
 		SAFE_CLOSE(fd);
 		SAFE_UNLINK(TEST_FILE);
 	}

1.8.3.1

After applying this patch, the warning is gone. Thanks!

safe_macros.c:62: WARN: hugemmap05.c:184: close(-1) failed: EBADF safe_macros.c:360: WARN: hugemmap05.c:185: unlink(hugemmap05/file)
failed: ENOENT

Li Wang
[email protected]

Hi!
Pushed, thanks.

Cyril Hrubis
[email protected]