|
From: David H. <da...@da...> - 2016-02-12 09:33:10
|
Hi Philippe, sorry to be poking about this again. Is there any news on the issue? Can I help to provide a fix for it? Would increasing the pool size really be a good fix? Or should the code be re-written to dynamically allocate the needed memory? I guess another solution could be to just truncate the string, if it is just debug information then that would at least be better that crashing? Let me know how I can help Thanks /David On Mon, Feb 8, 2016 at 2:19 PM, David Hallas <da...@da...> wrote: > Hi Philippe, > > thanks a lot for your help. I have created this ticket > https://bugs.kde.org/show_bug.cgi?id=359133 to track the issue. > > I was wondering, is there some kind out of out buffering when using printf > in valgrind? I guess that could explain why I do not see the debug print? > > Thanks > > /David > > On Fri, Feb 5, 2016 at 9:05 PM, Philippe Waroquiers < > phi...@sk...> wrote: > >> On Fri, 2016-02-05 at 11:20 +0100, David Hallas wrote: >> >> >> > thanks for the reply. I tried increasing the SEGINFO_STRPOOLSIZE as >> > you requested, and for me the magic number is 268*1024 then the assert >> > goes away :) I also tried to add the print to the ML_(addStr) function >> > in storage.c but for some reason I never see the print? >> Strange. VG_(printf) prints unconditionally. >> > >> > I tried to play with objdump and nm to find some large symbols, but I >> > think I do not have enough objdump/nm skills to do that. But I also >> > tried to play with strings and I found one string (a symbol name) that >> > is 274154 bytes long (this fits perfectly with that I see the problem >> > if the SEGINFO_STRPOOLSIZE is 267*1024 but not with 268*1024). The >> > code that I am debugging is using a lot of templates, so go figure ;) >> > So I think it sounds very plausible that it is this string that is >> > causing the problem? >> A symbol name of 268 Kb is quite impressive, >> I hope you never have to print this variable in gdb :). >> > >> > Please let me know if there is any other information that you need? Or >> > if you have some other things to test. >> The information is clear, I think the problem is identified now. >> The best is to file a bug entry in bugzilla, I hope to fix >> this problem in the coming weeks or so. >> >> Thanks >> >> Philippe >> >> >> >> > |