-
Notifications
You must be signed in to change notification settings - Fork 82
Pidgin crashes if there are too many messages to synchronize (only win32) #220
Comments
Can you post debug logs of the crash happening? I'm sure Eion would appreciate that. You may want to send them directly to him, in case it contains "sensitive information". Btw: We do test this case specifically, so this is news to us. |
Thats some of the last lines i see right before the crash, if i start Pidgin in debug mode: (21:37:17) prpl-telegram: sending all pending recipes Program: D:\Pidgin\pidgin.exe Expression: c It dont creates a entrys in the pidgin.RPT file. |
The pseudo-backtrace for this is
and it looks like its asserting because the query it's trying to put in the |
@TomTheDragon can you get a full debug log from connection start up until the assert() happens, what we should see is something like |
Ok, looking through the code, this looks like a 32-bit vs 64-bit problem.
|
@TomTheDragon can you try with this dll, which has BenWiederhake/tgl@87abdf0 added to try and fix your crash |
(repost from the telegram chat) (04:57:24) prpl-telegram: Resent query #6238002925376460184 as #6238002951143444744 of size 68 to DC 2 it also dont writes any changes to the "state" file |
@EionRobb: AFAIR I sent you a patch that uses only a single, global counter in |
libtelegram.zip Apologies for the hold up. This build has the patches from https://github.com/BenWiederhake/telegram-purple/tree/tmp/for-eion |
OK so i did two tests with the new test build and i did use a 10 days old state file. Test 1: Test 2: Here is the error message from the two crashes (only the some of the last lines): (20:09:41) proxy: Connected to 149.154.167.91:443. (20:09:41) prpl-telegram: Resent query #6241577808057671064 as #6241577821256865788 of size 32 to DC 4 Program: D:\Pidgin\pidgin.exe Expression: success |
Sorry, it seems this nearly petered out. It looks like it now fails to download a document, which seems to be another, unrelated bug. I fear that the bug is another incarnation of EionRobb/tgl@958aeec . |
New report from @radasbona / Markus M. via dev chat on the same "Expression: c" thingy we originally had. So the issue seems to be an issue for more than one person. @TomTheDragon -- poke! I'd really like to hear what that second issue is. |
Give me some time, i dont have an old state file atm so i just made one today. I will test it in some days again, when i got enough new massages. |
Take your time :) |
I did a test again today but i was not able to reproduce the crash, it did sync up the logs for all the 2,5 weeks and did download 97 files without any problems. ... But i will make a backup of the newest state file so that i can reproduce any crash if one is happening again. |
The bug came back since i updated from v1.2.4 to v1.2.6. It did work yesterday but it always crashes now, since there are much more messages to synchronise now. Here is the log: (14:46:25) prefs: /pidgin/conversations/toolbar/wide changed, scheduling save. Program: D:\Pidgin\pidgin.exe Expression: c No entry in the pidgin.RPT file was created. |
This problem seems to get more and more ridicules. It now crashes at least once when i try to launch Pidgin every day and when i restart (sometimes multiple times in a row), all the last messages are gone. So i always have to make a backup of the state file before starting Pidgin, so that i can copy it back, to get all the old messages. |
A little report after some time: |
I got this in bitlbee/linux, and the error matches the original bug. However, I'm using a very old git version (65105dd, 10 months old) and the assert that was failing here failing is gone from current versions.
@TomTheDragon You should get fresh debug logs, or maybe just consider this bug fixed and open a different one. |
It gives me a runtime error when there are too many messages to synchronize. This only happens in the Windows version of the plugin, the Linux one handles this without any problems.
The text was updated successfully, but these errors were encountered: