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

assertion failed #172

Closed
cmumford opened this issue Sep 9, 2014 · 1 comment
Closed

assertion failed #172

cmumford opened this issue Sep 9, 2014 · 1 comment
Assignees

Comments

@cmumford
Copy link
Contributor

cmumford commented Sep 9, 2014

Original issue 166 created by pmarches on 2013-05-11T06:15:39.000Z:

*What steps will reproduce the problem?

  1. Create a leveldb database with this leveldb implementation (in my case, it was with the bitcoind client)
  2. Open the database with the java version of leveldb (https://github.com/dain/leveldb)
  3. Open the database with the C++ version of leveldb

*What is the expected output? What do you see instead?
The c++ leveldb will crash with an assert failure for table/block.cc, at line 19. IMHO, the code should be:
assert(size_ >= sizeof(uint32_t));

and table/block.cc, at line 256 should be
if (size_ < sizeof(uint32_t)) {

*What version of the product are you using? On what operating system?
1.6.0 on Ubuntu Linux kernel 3.5.0-28-generic, x86_64

I changed the code and tested a bit and everything seems fine. Please let me know how this goes, so I can propagate the changes to the leveldb version used by the bitcoind client.

Thanks!

@cmumford
Copy link
Contributor Author

cmumford commented Sep 9, 2014

Comment #1 originally posted by dgrogan@chromium.org on 2013-05-15T00:11:30.000Z:

<empty>

@cmumford cmumford self-assigned this Sep 9, 2014
@cmumford cmumford closed this as completed Sep 9, 2014
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

1 participant