Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#33 CAbortThread - memory leak

Released in 1.4x
closed-fixed
Drzewiecki T.
General (8)
5
2006-03-07
2006-03-04
Drzewiecki T.
No

Start SQLTools inside VC++.
Click cancel inside connection dialog.
Close SQLTools.

Detected memory leaks!
Dumping objects ->
f:\prog\sqltools\current\sqlt\sqltools\abortthread\abortthread.cpp(41)
: {13182} client block at 0x018E0748, subtype c0, 244
bytes long.
a CAbortThread object at $018E0748, 244 bytes long
Object dump complete.

Discussion

  • Drzewiecki T.
    Drzewiecki T.
    2006-03-04

    Logged In: YES
    user_id=747155

    There is a bug inside main 'exit' application code:
    CSQLToolsApp::ExitInstance()

    Added:
    m_pAbortThread->PostThreadMessage(WM_QUIT, 0, 0);
    and detailed checking of 'abortthread' state.

     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-04

    • summary: CAbortThread memory leak --> CAbortThread - memory leak
     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-04

    Logged In: YES
    user_id=747155

    For detailed testing!

     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-04

    • status: open --> open-fixed
     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-06

    Logged In: YES
    user_id=747155

    After short testing I think that current state is much
    better than earlier.

    But there are still problems with:
    B#1111222 - SQLTools closing abruptly.
    It will be a very hard bug to find.

     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-06

    • status: open-fixed --> closed-fixed
     
  • Drzewiecki T.
    Drzewiecki T.
    2006-03-07

    • milestone: 455736 --> Released in 1.4x