SwapRandom(unsigned int, unsigned int): Assertion 'it_1 != mapInfo.end()' failed. #297

Closed
opened 1 year ago by duke · 11 comments
duke commented 1 year ago
Owner

Related to #266

addrman.cpp:153: void CAddrMan::SwapRandom(unsigned int, unsigned int): Assertion 'it_1 != mapInfo.end()' failed.
Related to #266 ``` addrman.cpp:153: void CAddrMan::SwapRandom(unsigned int, unsigned int): Assertion 'it_1 != mapInfo.end()' failed. ```
duke added the
bug
label 1 year ago
Poster
Owner

@fekt have you seen this bug happen again lately?

@fekt have you seen this bug happen again lately?
Collaborator

@duke I haven't seen it again since reported. Now that I said that I won't be surprised if it crashes with it soon 🙂

One DRGX user reported the original assert here: #266

@duke I haven't seen it again since reported. Now that I said that I won't be surprised if it crashes with it soon 🙂 One DRGX user reported the original assert here: https://git.hush.is/hush/hush3/issues/266
Poster
Owner

This assert is removed in commit 04c28e3eef on the duke branch. After some testing it will be sent to the dev branch.

This assert is removed in commit 04c28e3eef00427751d3cd5aecd05858910ba1c8 on the duke branch. After some testing it will be sent to the dev branch.
Collaborator

Sounds good. I just got this assert again so I might try updating to your branch. It seems to crash whenever I say it's good just to prove me wrong.

Sounds good. I just got this assert again so I might try updating to your branch. It seems to crash whenever I say it's good just to prove me wrong.
Collaborator

I just received this bug with Hush Daemon version v3.9.3-5184b4483-dirty on OS: Debian GNU/Linux 11 (bullseye) aarch64

I just received this bug with Hush Daemon version v3.9.3-5184b4483-dirty on OS: Debian GNU/Linux 11 (bullseye) aarch64
Poster
Owner

@jahway603 did you get this bug with a hush full node or dragonx full node?

@jahway603 did you get this bug with a hush full node or dragonx full node?
Collaborator

@duke I got this with a hush full node, specifically a lightwalletd server

@duke I got this with a hush full node, specifically a lightwalletd server
Collaborator

@duke I updated to latest duke branch yesterday on lite2.hushpool.is. I'll report back if I get any new crashes with asserts removed.

@duke I updated to latest `duke` branch yesterday on `lite2.hushpool.is`. I'll report back if I get any new crashes with asserts removed.
Poster
Owner

@fekt it might be time for you to say "it seems to be working" so that we can tempt the code into breaking again 😅

@fekt it might be time for you to say "it seems to be working" so that we can tempt the code into breaking again 😅
Collaborator

@duke it hasn't crashed again yet. maybe it will now lol

@duke it hasn't crashed again yet. maybe it will now lol
Poster
Owner

@fekt I am going to go ahead and close this. Since this assertion was removed, if there are still bugs, they will look different with different errors and we can make new issues for those

@fekt I am going to go ahead and close this. Since this assertion was removed, if there are still bugs, they will look different with different errors and we can make new issues for those
duke closed this issue 11 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.