diff options
author | Ian Romanick <[email protected]> | 2013-04-12 16:47:52 -0700 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2013-04-16 10:13:48 -0700 |
commit | a27c6e1aea1d048e0c701ca59c3c5bcf12b69d7b (patch) | |
tree | 0826d4fc722feba6ed24b46ec25bed99edba16f1 /src/mesa/swrast/s_renderbuffer.h | |
parent | 6758498eb79287fe5d67cd5c869e9fd387f95def (diff) |
mesa/swrast: Move free calls outside the attachment loop
This was originally discovered by Klocwork analysis:
Possible memory leak. Dynamic memory stored in 'srcBuffer0'
allocated through function 'malloc' at line 566 can be lost at line
746
However, I think the problem is actually much worse. Since the memory
is freed after the first pass through the loop, the released buffer may
be used on the next iteration!
NOTE: This is a candidate for stable release branches.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Brian Paul <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/swrast/s_renderbuffer.h')
0 files changed, 0 insertions, 0 deletions