Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / qt / 2011-04

Fwd: Change in qt45.pas

3 messages · 2 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Vincent Snijders

Den Jean,

This is the conversation I had with Martin about the change in qt45.pas.

Vincent

---------- Forwarded message ----------

Earlier message 66 lines
From: Martin Date: 2011/4/14 Subject: Re: Change in qt45.pas To: Vincent Snijders On 14/04/2011 12:21, Vincent Snijders wrote: > > 2011/4/14 Martin: >> >> On 14/04/2011 07:50, Vincent Snijders wrote: >>> >>> Hi Martin, >>> >>> I don't know if you are subscribed to the qt list, but can you give >>> more information about >>> >>> http://svn.freepascal.org/cgi-bin/viewvc.cgi?view=rev&root=lazarus&revision=25368 >>> >>> See >>> http://lists.lazarus.freepascal.org/pipermail/qt/2011-April/001670.html >>> and further in that thread. >> >> No I am not on the QT list. Thanks for forwarding. >> >> http://bugs.freepascal.org/view.php?id=16448 >> >> That's all I could find... Don't remember much unfortunately. > > Seems to be another range check, that you fixed there: > http://svn.freepascal.org/cgi-bin/viewvc.cgi?diff_format=l&view=rev&root=lazarus&revision=25385 > > Vincent Hm, it's the only think I could find. And it had the revision number in the "fixed in rev". All I remember (and that only after I found the bur report), is that I had problems with various range check errors. So apparently, I must have eventually traced a call to this function, that had a nil pointer. Setting breakpoints shows the code is still reached, but gdb cannot backtrace through the qt libs... #0 GETPTRINTARRAYADDR(0xa00fd24) at qt\qt45.pas:13870 #1 ?? at :0 #2 ?? at :0 #3 ?? at :0 stepping out all the way, eventually got me here: #0 GETACTIONBYINDEX(0xeb68c60, 0) at qt\qtwidgets.pas:3520 #1 INSERTMENU(0xeb68c60, 0, 0xc638438) at qt\qtwidgets.pas:11416 #2 ATTACHMENU(0xf0d94a0, 0xf1292a8) at qt\qtwsmenus.pp:101 #3 CREATEHANDLE(0xf1292a8) at include\menuitem.inc:134 #4 HANDLENEEDED(0xf1292a8) at include\menuitem.inc:628 #5 INSERT(0xf128ac8, 0, 0xf1292a8) at include\menuitem.inc:785 #6 UPDATEMENUSTRUCTURE(0xe967a78) at menuintf.pas:1152 #7 SETMENUITEM(0xe967a78, 0xf128ac8) at menuintf.pas:1595 #8 SETUPMAINMENU(0x2e6df8) at main.pp:2350 #9 CREATE(0x2e6df8, 0xb57ae8, 0x2a6e90) at main.pp:1377 #10 :107 This is during IDE start up Martin

Reply by email

Den Jean
to Vincent Snijders
Vincent Snijders wrote
This is the conversation I had with Martin about the change in qt45.pas.

thx both. the concerned procedures are only used internally and the uses cases are not foreseen for nil pointers. I will see if I can reproduce it in normal cases, if so it is a bug else it was probably something like a stack alignment error (more frequent on arm)
that causes code execution with random parameter.

Reply by email

Den Jean
to Den Jean
Den Jean wrote
thx both. the concerned procedures are only used internally and the uses cases are not foreseen for nil pointers.

finally had some time to look into it,
will be fixed in next release.
current fix (change to qt45.pas) is good enough

Reply by email