Check module for memory leaks #122

Closed
Sannis opened this Issue Apr 5, 2012 · 3 comments

Comments

Projects
None yet
1 participant
Owner

Sannis commented Apr 5, 2012

Write a simple script that inserts and selects rows usyng sync and async methods and run it for a day.

Reported by @puzrin.

Sannis was assigned Apr 9, 2012

Owner

Sannis commented May 22, 2012

In file src/mysql_bindings/result.cc:

    // TODO: Is this a memory leak?
    // delete fetchAll_req->res;

@Sannis Sannis added a commit that referenced this issue Jun 1, 2012

@Sannis Sannis Improve tools/run-memory-usage.js
It leaks, refs #122
a4d97e0

@Sannis Sannis added a commit that referenced this issue Jun 3, 2012

@Sannis Sannis Improve tools/run-memory-usage.js
It leaks, refs #122
7d23f6d

@Sannis Sannis added a commit that referenced this issue Jun 3, 2012

@Sannis Sannis Update memory leaks finder REPL
New test cases
passTimesAsArgument support
Small fixes

Refs #122
9959777

@Sannis Sannis added a commit that referenced this issue Jun 3, 2012

@Sannis Sannis User must can manipulate result after fetchAll()
Refs #122, reverted changes from #60
e59d92b
Owner

Sannis commented Jun 5, 2012

Memory usage profiler with graphs: 622169b...bef2212

Sannis closed this Jul 24, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment