Regarding Deleteing pointer!!!!!

**subhashish1213** · 09-25-2009

Hi friends...

Can any one give me the solutions....

Code:

int *m_ptr_ptr = new com::att::ib::Core_V1_0::NVPair

                .......
                .......
                .......

void
com::att::ib::Core_V1_0::IT_Gen_NVPairStreamable::replace_nonshareable_arg(
    CORBA::IT_InStream_ptr is,
    unsigned long _direction
)
{
    if (*m_ptr_ptr != 0)
    {
        if (m_own_value|| _direction == CORBA::ARG_OUT)
        {
            delete *m_ptr_ptr;
        }
        if (_direction != CORBA::ARG_INOUT)
        {
            *m_ptr_ptr = 0;
        }
        if (_direction == CORBA::ARG_IN)
        {
            m_own_value = IT_TRUE;
        }
    }
    read(is);
}

Then I got the error like this......
==============================================
Passing '*this->m_ptr_ptr' as parameter after its value was freed.
==============================================

Also I want to know Can we delete a pointer like this?????

delete *m_ptr_ptr; // is it correct
delete m_ptr_ptr; // or is it correct

**Sebastiani** · 09-25-2009

Well, m_ptr_ptr is an int*, so *m_ptr_ptr is an int, right? Which do you think would be valid in a delete expression - the former or the latter?

Even so, a delete'd pointer should always be reset to NULL, and you should check if it is so before dereferencing it.

Also, your code violates the RAII principle - use a 'smart' pointer.

**subhashish1213** · 10-01-2009

Sorry the code is like this...

Code:

class cATT_BUS
{
 ....
 ...code//
 ...
};
cATT_BUS *m_ptr_ptr = new com::att::ib::Core_V1_0::NVPair

                .......
                .......
                .......

void
com::att::ib::Core_V1_0::IT_Gen_NVPairStreamable::replace_nonshareable_arg(
    CORBA::IT_InStream_ptr is,
    unsigned long _direction
)
{
    if (*m_ptr_ptr != 0)
    {
        if (m_own_value|| _direction == CORBA::ARG_OUT)
        {
            delete *m_ptr_ptr;
        }
        if (_direction != CORBA::ARG_INOUT)
        {
            *m_ptr_ptr = 0;
        }
        if (_direction == CORBA::ARG_IN)
        {
            m_own_value = IT_TRUE;
        }
    }
    read(is);
}

After Compiling I got the error like this....

==============================================
Passing '*this->m_ptr_ptr' as parameter after its value was freed.
==============================================

Also I want to know Can we delete a pointer like this?????

delete *m_ptr_ptr; // is it correct
delete m_ptr_ptr; // or is it correct

**cpjust** · 10-01-2009

I have no idea what com::att::ib::Core_V1_0::NVPair is, but I believe this:

Code:

    if (*m_ptr_ptr != 0)
    {
        if (m_own_value|| _direction == CORBA::ARG_OUT)
        {
            delete *m_ptr_ptr;
        }

should be changed to this:

Code:

    if (m_ptr_ptr != 0)
    {
        if (m_own_value|| _direction == CORBA::ARG_OUT)
        {
            delete m_ptr_ptr;
            m_ptr_ptr = 0;
        }

**Elysia** · 10-04-2009

Also I want to know Can we delete a pointer like this?????

delete *m_ptr_ptr; // is it correct
delete m_ptr_ptr; // or is it correct

We already told you.
Pass to delete the actual pointer you want to delete. So if you have

Code:

int* x = new int;

Then do delete x, not delete *x, because *x would give you the int it points to, and delete needs its address.

**subhashish1213** · 10-05-2009

Thanks a lot.....

Thread: Regarding Deleteing pointer!!!!!

Thread Tools

Search Thread

Display

Regarding Deleteing pointer!!!!!

Similar Threads

Following CTools

Quick Pointer Question

Parameter passing with pointer to pointer

Direct3D problem

Struct *** initialization