Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

version 1.3.4 #135

Closed
steom opened this issue Feb 5, 2024 · 4 comments
Closed

version 1.3.4 #135

steom opened this issue Feb 5, 2024 · 4 comments

Comments

@steom
Copy link

steom commented Feb 5, 2024

Listening for ICMP Packets
Unhandled exception. System.ArgumentException: Offset and length were out of bounds for the array or count is greater than the number of elements from index to the end of the source collection.
at System.Buffer.BlockCopy(Array , Int32 , Array , Int32 , Int32 )
at PowerPing.ICMP..ctor(Byte[] data, Int32 size, Int32 offset)
at PowerPing.Listen.ListenForICMPOnAddress(IPAddress address)
at PowerPing.Listen.<>c__DisplayClass1_1.b__0()
at System.Threading.Thread.StartCallback()

@Killeroo
Copy link
Owner

Killeroo commented Feb 5, 2024

I'm sorry I could not reproduce this, could you give some more details so I can debug this further, any info like:

  • How frequently this occurs
  • A screenshot of PowerPing before the exception
  • What platform and OS you are running on
  • What type of ping you were listening for when this happened (if known)

Thanks

Killeroo added a commit that referenced this issue Jun 2, 2024
@Killeroo
Copy link
Owner

Killeroo commented Jun 5, 2024

I've added a speculative fix that should stop this from happening in the latest release. Please let me know if it keeps occuring.

@steom
Copy link
Author

steom commented Jun 6, 2024

speculative fix works. version 1.3.5 don't crash.

@Killeroo
Copy link
Owner

Glad to hear that, closing this issue for now then 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants