| Summary: | possible memory leak in mipmap.c | ||
|---|---|---|---|
| Product: | Mesa | Reporter: | amarnath.valluri |
| Component: | GLU | Assignee: | amarnath.valluri |
| Status: | RESOLVED FIXED | QA Contact: | mesa-dev |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | Fixes the menory leak | ||
Created attachment 124449 [details] [review] Fixes the menory leak This patch fixes the memory leak. Patches should be send to the mesa-dev@lists.freedesktop.org mailing list see: http://mesa3d.org/devinfo.html#submitting Also you should only set the bug to resolved/fix if the patch has been pushed to git otherwise none will ever see the bug. I have submitted the patch to mesa-dev list. Thanks for the pointers. I reopened the bug. Patch updated and committed. 0bf42e41c8b63fc2488dd8d41f696310b5a5a6a7 Where... Can't find it in master. Or how long does it take to sync 'your' master and 'master'...;-) Argh... 'GLU' was slipped through. Sorry, for the noise. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.
There is a possible memory leak in function gluBuild2DMipmapLevelsCore() in case of OUT_OF_MEMORY void *newMipmapImage= malloc((size_t) (newRowLength*newheight)); if (newMipmapImage == NULL) { glPixelStorei(GL_UNPACK_ALIGNMENT, psm.unpack_alignment); glPixelStorei(GL_UNPACK_SKIP_ROWS, psm.unpack_skip_rows); glPixelStorei(GL_UNPACK_SKIP_PIXELS, psm.unpack_skip_pixels); glPixelStorei(GL_UNPACK_ROW_LENGTH, psm.unpack_row_length); glPixelStorei(GL_UNPACK_SWAP_BYTES, psm.unpack_swap_bytes); return GLU_OUT_OF_MEMORY; } In this case memory allocated for both 'srcImage' and 'dstImage' is leaked.