Correct statement to actually be the intended assert statement.

anarazel · anarazel · commit 819aceaa0e7f · 2015-12-14T11:25:02.000+01:00
e3f4cfc introduced a LWLockHeldByMe() call, without the corresponding Assert() surrounding it. Spotted by Coverity. Backpatch: 9.1+, like the previous commit
diff --git a/src/backend/storage/buffer/bufmgr.c b/src/backend/storage/buffer/bufmgr.c
@@ -2564,7 +2564,7 @@ FlushOneBuffer(Buffer buffer)
 
 	bufHdr = &BufferDescriptors[buffer - 1];
 
-	LWLockHeldByMe(bufHdr->content_lock);
+	Assert(LWLockHeldByMe(bufHdr->content_lock));
 
 	FlushBuffer(bufHdr, NULL);
 }

Original file line number	Diff line number	Diff line change
`@@ -2564,7 +2564,7 @@ FlushOneBuffer(Buffer buffer)`
`2564`	`2564`
`2565`	`2565`	`bufHdr = &BufferDescriptors[buffer - 1];`
`2566`	`2566`
`2567`		`- LWLockHeldByMe(bufHdr->content_lock);`
	`2567`	`+ Assert(LWLockHeldByMe(bufHdr->content_lock));`
`2568`	`2568`
`2569`	`2569`	`FlushBuffer(bufHdr, NULL);`
`2570`	`2570`	`}`