Regarding Deleteing pointer!!!!!

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

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.

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

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;
        }

---

Quote:

---

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.

Thanks a lot.....