Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Correct statement to actually be the intended assert statement.
authorAndres Freund <andres@anarazel.de>
Mon, 14 Dec 2015 10:23:24 +0000 (11:23 +0100)
committerAndres Freund <andres@anarazel.de>
Mon, 14 Dec 2015 10:23:24 +0000 (11:23 +0100)
e3f4cfc7 introduced a LWLockHeldByMe() call, without the corresponding
Assert() surrounding it.

Spotted by Coverity.

Backpatch: 9.1+, like the previous commit

src/backend/storage/buffer/bufmgr.c

index b32543b1d56d79a3d92440e5a3156b07cca0586d..0d5fb0db88f61ed15e9460a444bfa4cad85fc453 100644 (file)
@@ -3004,7 +3004,7 @@ FlushOneBuffer(Buffer buffer)
 
    bufHdr = GetBufferDescriptor(buffer - 1);
 
-   LWLockHeldByMe(bufHdr->content_lock);
+   Assert(LWLockHeldByMe(bufHdr->content_lock));
 
    FlushBuffer(bufHdr, NULL);
 }