1. 27 Apr, 2012 7 commits
    • antirez's avatar
      Set LUA_MASKCOUNT hook more selectively. Fixes issue #480. · 77a75fde
      antirez authored
      An user reported a crash with Redis scripting (see issue #480 on
      github), inspection of the kindly provided strack trace showed that
      server.lua_caller was probably set to NULL. The stack trace also slowed
      that the call to the hook was originating from a point where we just
      used to set/get a few global variables in the Lua state.
      
      What was happening is that we did not set the timeout hook selectively
      only when the user script was called. Now we set it more selectively,
      specifically only in the context of the lua_pcall() call, and make sure
      to remove the hook when the call returns. Otherwise the hook can get
      called in random contexts every time we do something with the Lua
      state.
      77a75fde
    • antirez's avatar
      Re-introduce -g -rdynamic -ggdb when linking, fixing strack traces. · e69e76d4
      antirez authored
      A previous commit removed -g -rdynamic -ggdb as LDFLAGS, not allowing
      Redis to produce a stack trace wth symbol names on crash.
      This commit fixes the issue.
      e69e76d4
    • antirez's avatar
      Produce the stack trace in an async safe way. · 9db4cea2
      antirez authored
      9db4cea2
    • antirez's avatar
      Don't use an alternative stack for SIGSEGV & co. · a28ab2a9
      antirez authored
      This commit reverts most of c5757662, in
      order to use back main stack for signal handling.
      
      The main reason is that otherwise it is completely pointless that we do
      a lot of efforts to print the stack trace on crash, and the content of
      the stack and registers as well. Using an alternate stack broken this
      feature completely.
      a28ab2a9
    • antirez's avatar
      Redis test: More reliable BRPOPLPUSH replication test. · b1ee7da7
      antirez authored
      Now it uses the new wait_for_condition testing primitive.
      Also wait_for_condition implementation was fixed in this commit to properly
      escape the expr command and its argument.
      b1ee7da7
    • antirez's avatar
      Redis test: scripting EVALSHA replication test more reliable. · d7bad544
      antirez authored
      A new primitive wait_for_condition was introduced in the scripting
      engine that makes waiting for events simpler, so that it is simpler to
      write tests that are more resistant to timing issues.
      d7bad544
    • David Tran's avatar
      Spelling: s/synchrnonization/synchronization · 8111a803
      David Tran authored
      8111a803
  2. 24 Apr, 2012 13 commits
  3. 21 Apr, 2012 4 commits
  4. 19 Apr, 2012 2 commits
  5. 18 Apr, 2012 14 commits