1. 21 Feb, 2019 10 commits
  2. 19 Feb, 2019 4 commits
  3. 18 Feb, 2019 4 commits
  4. 14 Feb, 2019 2 commits
  5. 13 Feb, 2019 6 commits
  6. 12 Feb, 2019 9 commits
  7. 11 Feb, 2019 5 commits
    • Salvatore Sanfilippo's avatar
      Merge pull request #5831 from lamby/x32-warnings · 85cadafa
      Salvatore Sanfilippo authored
      Don't assume the __x86_64__ pointer size to avoid warnings on x32
      85cadafa
    • Chris Lamb's avatar
      Don't assume the __x86_64__ pointer size to avoid warnings on x32. · d0089cf2
      Chris Lamb authored
      __x86_64__ is defined on the on the x32 architecture and the conditionals in
      debug.c therefore assume the size of (void*) etc:
      
        debug.c: In function 'getMcontextEip':
        debug.c:757:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
             return (void*) uc->uc_mcontext.gregs[16]; /* Linux 64 */
                    ^
        debug.c: In function 'logRegisters':
        debug.c:920:21: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
             logStackContent((void**)uc->uc_mcontext.gregs[15]);
      
      We can remedy this by checking for __ILP32__ first. See:
      
        https://wiki.debian.org/ArchitectureSpecificsMemo
      
      ... for more info.
      d0089cf2
    • antirez's avatar
      ACL: return error when removing a non existing password. · f0c7cfa4
      antirez authored
      Otherwise it's very simple for an human mistake to think a password is
      removed because of a typo in the ACL SETUSER myuser <somepass command
      line.
      f0c7cfa4
    • antirez's avatar
      ACL: refactor+fix AUTH check in processCommand(). · f8a6132f
      antirez authored
      The part that is fixed is that now if the default user is off whatever
      is its configuration the user is not considered authenticated.
      f8a6132f
    • antirez's avatar
      ACL: ACLFreeUserAndKillClients(): free user later. · df346bca
      antirez authored
      Soon or later we may have code in freeClient() that may have to deal
      with ACLs. Imagine for instance the command proposed multiple times (not
      sure if this will ever be accepted but still...):
      
          ONCLOSE DEL mykey
      
      Accumulating commands to run when a client is disconnected. Now the
      function is compatible with such use cases.
      
      Related to #5829.
      df346bca