--------------030208010703070807090606
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Hi,


Inside Query there is a memory leak.
Please find the patch included that fixes this issue.

The reason is that you pass NOT the created memory, but the address of
the reference.


Greetings,
Steven

--------------030208010703070807090606
Content-Type: text/x-patch;
name="query_memleak.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
filename="query_memleak.patch"

Index: query.cpp
================================================== =================
--- query.cpp (revision 64)
+++ query.cpp (working copy)
@@ -262,7 +262,7 @@
if (replace) {
S = *ss;
S.processed = true;
- return &S;
+ return ss;
}
else {
return ss;


--------------030208010703070807090606
Content-Type: text/plain; charset=us-ascii


--
MySQL++ Mailing List
For list archives: http://lists.mysql.com/plusplus
To unsubscribe: http://lists.mysql.com/plusplus?unsu...ie.nctu.edu.tw
--------------030208010703070807090606--

Steven Van Ingelgem wrote:
Agreed.

Sorry, that's not the right fix. That passes responsibility for freeing
the memory to code that doesn't do it, so your code still has a memory
leak. I've applied the correct one.

Thanks in any case for the hint.

--
MySQL++ Mailing List
For list archives: http://lists.mysql.com/plusplus
To unsubscribe: http://lists.mysql.com/plusplus?unsu...ie.nctu.edu.tw

Worked for me ... But happy it has been resolved anyway

Warren Young wrote:

--
MySQL++ Mailing List
For list archives: http://lists.mysql.com/plusplus
To unsubscribe: http://lists.mysql.com/plusplus?unsu...ie.nctu.edu.tw