aMule Forum

Please login or register.

Login with username, password and session length
Advanced search  

News:

We're back! (IN POG FORM)

Pages: 1 2 [3]

Author Topic: another crash  (Read 22469 times)

Stu Redman

  • Administrator
  • Hero Member
  • *****
  • Karma: 214
  • Offline Offline
  • Posts: 3739
  • Engines screaming
Re: another crash
« Reply #30 on: December 19, 2008, 11:02:39 PM »
Quote from: wires on December 18, 2008, 09:30:00 AM
Hi, this may help you to detect the cause
Really interesting. Problem is, I have no 64bit Linux platform for testing at hand. And it's nothing that can be deduced from program flow and logic.
My workaround does nothing useful in theory, it just stops using the this-pointer as status variable. Now if the pointer keeps ketting corrupted, at least with tray icon off nothing bad happens. But - luquino has tested it with tray icon on and it still works. So the read access to the pointer appears to corrupt it ? Whenever a message comes in ?  ???

Quote from: luquino on December 19, 2008, 10:31:10 PM
Your workaround for the tray icon pointer will be inserted in SVN version? If  yes, when?
It's in trunk since Thursday (8268).

Quote from: luquino on December 19, 2008, 10:31:10 PM
Thank you so much for your help.
You're welcome! Thank you for helping to get to the root of the problem (even if I still haven't found it).
Logged
The image of mother goddess, lying dormant in the eyes of the dead, the sheaf of the corn is broken, end the harvest, throw the dead on the pyre -- Iron Maiden, Isle of Avalon

wires

  • Jr. Member
  • **
  • Karma: 6
  • Offline Offline
  • Posts: 83
Re: another crash
« Reply #31 on: December 22, 2008, 08:37:19 AM »
Quote from: Stu Redman on December 18, 2008, 12:53:03 AM
Quote from: luquino on December 15, 2008, 10:03:03 PM
Probably you have found the right solution even if is not the better one.
It's just a workaround. The real problem remains in the dark.  :(
Commited in 9268.

Ok Stu, amule r9278 (trunk) seems to be ok. I've tried to crash it but it didn't  :D
Quote from: wires on December 18, 2008, 09:30:00 AM
Hi, this may help you to detect the cause: I'm able to crash amule just starting a new client (running with another user), adding first amule as a friend and sending a message from the second one. Amule crashes inmediately (2_2_X rev 9262 no patch).

No asserts and no log after adding some debug messages to check it (debug & non debug builds). I also expected it to crash on close since RemoveSystray is not protected but it didn't (It happened to me with my own patched version). So is this patch included in prerelease version? I'll run some more tests but I think it won't appear again  :)

Thank you very much for your work! (and patience  ;D)
Logged

Stu Redman

  • Administrator
  • Hero Member
  • *****
  • Karma: 214
  • Offline Offline
  • Posts: 3739
  • Engines screaming
Re: another crash
« Reply #32 on: December 22, 2008, 09:46:42 AM »
Patch is now in 2.2.3 too (9285). Please test it if it holds.
Logged
The image of mother goddess, lying dormant in the eyes of the dead, the sheaf of the corn is broken, end the harvest, throw the dead on the pyre -- Iron Maiden, Isle of Avalon

wires

  • Jr. Member
  • **
  • Karma: 6
  • Offline Offline
  • Posts: 83
Re: another crash
« Reply #33 on: December 23, 2008, 10:30:24 PM »
Quote from: Stu Redman on December 22, 2008, 09:46:42 AM
Patch is now in 2.2.3 too (9285). Please test it if it holds.

no problem! I'll do it as soon as it gets published, at this moment last rev available is 9276 for release-2_2_X http://amule.uw.hu/.

Thanks!
Logged

wires

  • Jr. Member
  • **
  • Karma: 6
  • Offline Offline
  • Posts: 83
Re: another crash
« Reply #34 on: December 26, 2008, 02:08:50 PM »
aMule RELEASE_2_2_X rev 9288 running fine for +20h. It successfully handles incoming chat messages so this problem has been fixed (may be something collateral) or at least the workaround works great.

Thanks!
Logged
Pages: 1 2 [3]
« previous next »