valgrind: Mismatched free in libnux
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Nux |
Fix Released
|
High
|
Daniel van Vugt | ||
Unity |
Fix Released
|
Undecided
|
Unassigned | ||
nux (Ubuntu) |
Fix Released
|
Undecided
|
Daniel van Vugt | ||
unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
This is a simple one. There are only two new/delete pairs in IOpenGLAsmShade
==1555== Mismatched free() / delete / delete []
==1555== at 0x4C27FFF: operator delete(void*) (vg_replace_
==1555== by 0x15800D32: nux::IOpenGLAsm
==1555== Mismatched free() / delete / delete []
==1555== at 0x4C27FFF: operator delete(void*) (vg_replace_
==1555== by 0x15800C42: nux::IOpenGLAsm
This is worth fixing at least to reduce the number of errors reported by valgrind for unity/compiz. It takes up a significant amount of space in valgrind logs and is trivial to fix.
To fix, replace both instances of:
delete ShaderSource;
with:
delete[] ShaderSource;
Related branches
- Jay Taoko (community): Approve
-
Diff: 12 lines (+1/-1)1 file modifiedNuxGraphics/IOpenGLGLSLShader.cpp (+1/-1)
description: | updated |
Changed in nux: | |
assignee: | nobody → Daniel van Vugt (vanvugt) |
status: | New → In Progress |
Changed in nux (Ubuntu): | |
assignee: | nobody → Daniel van Vugt (vanvugt) |
status: | New → In Progress |
Changed in unity: | |
status: | New → Invalid |
Changed in nux: | |
importance: | Undecided → High |
Changed in unity: | |
status: | Invalid → In Progress |
Changed in nux: | |
status: | Fix Committed → Fix Released |
Changed in unity: | |
status: | Fix Committed → Fix Released |
Changed in unity (Ubuntu): | |
status: | New → Fix Released |
Valgrind log attached. You can find the errors by search for the string "Mismatch".