Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixed memory leak #431

Merged
merged 1 commit into from Jun 23, 2014
Merged

Fixed memory leak #431

merged 1 commit into from Jun 23, 2014

Conversation

nkt
Copy link
Contributor

@nkt nkt commented Jun 21, 2014

I've found this error by cppcheck, also it's find this:

[parser/base.c:647]: (error) Common realloc mistake: 'program' nulled but not freed upon failure

@ovr
Copy link
Contributor

ovr commented Jun 23, 2014

👍
@phalcon ping
Can we delete these comments?

@nkt
Copy link
Contributor Author

nkt commented Jun 23, 2014

@ovr of course we can, we are inside CVS.

ovr added a commit that referenced this pull request Jun 23, 2014
@ovr
Merge pull request #431 from nkt/master
8bdeca1 
Fixed memory leak
@ovr ovr merged commit 8bdeca1 into zephir-lang:master Jun 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nkt @ovr