Giter VIP home page Giter VIP logo

Comments (8)

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
Thanks! fixing this in SVN, I'm writing a regression test for the issue. I'm 
curious,
did you spot it because of a crash on unknown command or looking at the source 
code? :)

Original comment by [email protected] on 19 Mar 2009 at 10:20

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
It crashed when running the "Non existing command" test from the testsuite.

Original comment by [email protected] on 19 Mar 2009 at 11:22

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
ah ok, pretty strange that valgrind didn't reported this even running the full 
'make
test' under valgrind, so it didn't fixed before. Btw now it's fixed under SVN. 
Thanks
for the help. I'll probably release beta-9 tomorrow with just this fix since 
it's a
bit critical if the server can crash with the standard test.

Original comment by [email protected] on 19 Mar 2009 at 11:30

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
I mean beta-8 :) not 9

Original comment by [email protected] on 19 Mar 2009 at 11:31

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
... which is pretty much the same as the test you just added.

I bet the original test worked for you by fluke because whatever happened to be 
in
memory past the end of the array happened to be zeroes.

Original comment by [email protected] on 19 Mar 2009 at 11:48

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
Oh weird that valgrind didn't catch it.

Original comment by [email protected] on 19 Mar 2009 at 11:49

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024
Yes sorry I removed the duplicated test, I added it by instinct since it was 
out of
my mental scheme that there was already a test about it and still the bug was 
there.
Now I removed the new test since it's absolutely useless. I'll close the bug 
only
after beta-7 will be deprecated and beta-8 released. Thanks again.

Original comment by [email protected] on 19 Mar 2009 at 11:54

from redis.

GoogleCodeExporter avatar GoogleCodeExporter commented on August 20, 2024

Original comment by [email protected] on 22 Mar 2009 at 3:12

  • Changed state: Verified

from redis.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.