Re: [HACKERS] CurTransactionContext freed before transaction COMMIT ??? - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: [HACKERS] CurTransactionContext freed before transaction COMMIT ???
Date
Msg-id CAA4eK1KK+ZmQMrNoVoBiqoGyrncBhjsHijzs4CNsj=gk85PPeA@mail.gmail.com
Whole thread Raw
In response to [HACKERS] CurTransactionContext freed before transaction COMMIT ???  (Gaddam Sai Ram <gaddamsairam.n@zohocorp.com>)
Responses Re: [HACKERS] CurTransactionContext freed before transaction COMMIT???
List pgsql-hackers

On Tue, Oct 24, 2017 at 7:42 PM, Gaddam Sai Ram <gaddamsairam.n@zohocorp.com> wrote:

Hello people,

            We are implementing in-memory index. As a part of that, during index callbacks, under CurTransactionContext, we cache all the DMLs of a transaction in dlist(postgres's doubly linked list).
            We registered transaction callback, and in transaction pre-commit event(XACT_EVENT_PRE_COMMIT), we iterate through all cached DMLs(dlist) and populate in my in-memory data structure.

           --> For detailed explanation of our implementation, please look into below thread.

           --> It was working fine until I was greeted with a segmentation fault while accessing dlist!
           
           --> On further examining I found that dlist head is not NULL and it is pointing to some address, but the container I extracted is pointing to 0x7f7f7f7f7f7f7f5f, and I was not able to access any members in my container.

            From the above wiki, we came to a conclusion that the memory context in which that dlist was present was freed.
            And yes CLOBBER_FREED_MEMORY is enabled by passing --enable-cassert to enable asserts in my code.

           --> Normally the memory context inside XACT_EVENT_PRE_COMMIT is TopTransactionContext but in this particular case, the memory context was 'MessageContext'. Little unusual! Not sure why!

           --> So basically CurTransactionContext shouldn't get freed before transaction COMMIT.
           --> So what has actually happened here??? Kindly help us with your insights!


Can you check if CurTransactionContext is valid at that point?  To see, if this problem is related to CurTransactionContext, can you try to populate the list in TopMemoryContext and see if that works.


--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: [HACKERS] Pluggable storage
Next
From: Gaddam Sai Ram
Date:
Subject: Re: [HACKERS] CurTransactionContext freed before transaction COMMIT???