Remove unnecessary cleanup code.

robertmhaas · robertmhaas · commit 11ad3b35c25d · 2014-05-22T10:41:48.000-04:00
This is all inside a block guarded by op == DSM_OP_ATTACH, so it can
never be the case that op == DSM_OP_CREATE.

Reported by Coverity.
diff --git a/src/backend/storage/ipc/dsm_impl.c b/src/backend/storage/ipc/dsm_impl.c
@@ -554,14 +554,6 @@ dsm_impl_sysv(dsm_op op, dsm_handle handle, Size request_size,
 
 		if (shmctl(ident, IPC_STAT, &shm) != 0)
 		{
-			int			save_errno;
-
-			/* Back out what's already been done. */
-			save_errno = errno;
-			if (op == DSM_OP_CREATE)
-				shmctl(ident, IPC_RMID, NULL);
-			errno = save_errno;
-
 			ereport(elevel,
 					(errcode_for_dynamic_shared_memory(),
 					 errmsg("could not stat shared memory segment \"%s\": %m",
