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.

Sign up for GitHub

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

Jump to bottom

UDP crash #314

Closed
obscuren opened this issue Feb 15, 2015 · 3 comments
Closed

UDP crash #314

obscuren opened this issue Feb 15, 2015 · 3 comments

Comments

@obscuren
Copy link
Contributor 

2015/02/15 14:57:13 [TXP] Known transaction (28bd85bf)
2015/02/15 14:57:13 [TXP] Known transaction (de4ea2b6)
2015/02/15 14:57:13 [TXP] Known transaction (eac76bc8)
2015/02/15 14:57:13 [TXP] Known transaction (a94416db)
2015/02/15 14:58:14 [Peer ec38655e49a561b3 208.52.168.126:49424] whisper stopped
panic: runtime error: slice bounds out of range

goroutine 24 [running]:
github.com/ethereum/go-ethereum/p2p/discover.(*udp).loop(0xc2080d0b40)
    /Users/jeffrey/go/src/github.com/ethereum/go-ethereum/p2p/discover/udp.go:259 +0x852
created by github.com/ethereum/go-ethereum/p2p/discover.ListenUDP
    /Users/jeffrey/go/src/github.com/ethereum/go-ethereum/p2p/discover/udp.go:155 +0x5a6
@Gustav-Simonsson
Copy link

I think this should resolve it? should work even if the last element of the slice is deleted: #316

@Gustav-Simonsson
Copy link

nm, realized now the copy method should also work with the last element; could this be a concurrency issue?

@fjl
Copy link
Contributor

fjl commented Feb 16, 2015

It cannot be a concurrency issue because it updates a slice in a local variable. I know exactly how to test and fix it, just didn't do it yet.

@fjl fjl mentioned this issue Feb 17, 2015
Merged
@obscuren obscuren removed the type:bug label Feb 17, 2015
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Aug 25, 2021
@alexqrid
fix: tracing system transactions failed due to wrong state (ethereum#374
804c853 
)

* fixed wrong state creation which resulted in fail of tracing transaction

* sorted package imports

* added the check to `standardTraceBlockToFile` that resolves the ethereum#314

Co-authored-by: alexqrid <>
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Oct 31, 2023
@noel2004 @Thegaram @0xmountaintop
feat(trace): add storage proof about l1fee (baseFee, overhead, scalar…
910d4f1 
…) and withdraw root into trace (ethereum#314)

* add proof for predeployed storages

* reverse inneeded code

* update for mainbranch merging

* add coinbase storage as trace

* comment for clarify

* Update version.go

---------

Co-authored-by: Péter Garamvölgyi <th307q@gmail.com>
Co-authored-by: HAOYUatHZ <37070449+HAOYUatHZ@users.noreply.github.com>
marioevz pushed a commit to marioevz/go-ethereum that referenced this issue Feb 6, 2024
@gballet
Refactor transition post genesis (ethereum#311)
1d80ebd 
* rewrite per-block conversion pointer management

* remove unused method

* fix: a branch that can verge at genesis or post genesis (ethereum#314)

* fix: import cycle in conversion refactor (ethereum#315)

* fix shadowfork panic in OpenStorageTrie
maoueh pushed a commit to streamingfast/go-ethereum that referenced this issue Jun 13, 2024
@trianglesphere
ethapi: Set post Ecotone receipt fields (ethereum#314)
3653ceb
sduchesneau pushed a commit to streamingfast/go-ethereum that referenced this issue Jun 14, 2024
@magicxyyz
Merge pull request ethereum#314 from OffchainLabs/fix-pebble-metrics-…
f941743 
…registration

fix pebble metrics registration [NIT-2468]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fjl @Gustav-Simonsson @obscuren