1. 18 Nov, 2015 1 commit
    • Tom Lee's avatar
      Fix strerror_r on some esoteric platforms · bb1747b1
      Tom Lee authored
      Defining _XOPEN_SOURCE=1 causes strange behavior on Debian kfreebsd
      archs (i.e. GNU userspace with FreeBSD kernel) when _GNU_SOURCE is not
      defined.
      
      Not sure I fully understand the bizarre semantics, but it seems to
      use the XSI-compliant interface
      (int strerror_r(int, char*, size_t)) but the GNU implementation
      (char *strerror_r(int, char*, size_t)) such that strerror_r returns
      32-bits of a 64-bit char * on x86_64 kfreebsd. We would expect
      strerror_r to return zero when using the XSI-compliant strerror_r
      implementation or a 64-bit char* when using the GNU version. Instead,
      we get something in between!
      
      Unless I'm missing something, being more explicit about what version
      of _XOPEN_SOURCE we want seems to be the prudent thing to do here --
      and if folks want the GNU implementation of strerror_r for some reason
      they can always -D_GNU_SOURCE explicitly.
      bb1747b1
  2. 27 Oct, 2015 2 commits
  3. 18 Oct, 2015 1 commit
  4. 05 Oct, 2015 1 commit
  5. 16 Sep, 2015 2 commits
  6. 14 Sep, 2015 2 commits
  7. 25 Aug, 2015 1 commit
  8. 24 Aug, 2015 2 commits
  9. 27 Jul, 2015 10 commits
  10. 25 Jun, 2015 1 commit
  11. 24 Jun, 2015 1 commit
  12. 22 Jun, 2015 1 commit
  13. 30 May, 2015 1 commit
    • w359405949's avatar
      Update Makefile · 485d0a14
      w359405949 authored
      fix link error while run "make hiredis-example-libuv":
      
      undefined reference to `clock_gettime'
      undefined reference to `clock_getres'
      485d0a14
  14. 03 May, 2015 3 commits
  15. 30 Apr, 2015 1 commit
    • Alex Balashov's avatar
      Renamed redisContext struct member 'unix' to 'unix_sock' to avoid encountering... · d132d676
      Alex Balashov authored
      Renamed redisContext struct member 'unix' to 'unix_sock' to avoid encountering defined constant 'unix' in GNU C environment (see commit d8145d79).
      
      Not all code using hiredis can compile using '-std=c99', and/or not all users are able to easily make that change to the build process of various open-source projects, so it is more pragmatic to choose a different identifier that does not impose this requirement.
      d132d676
  16. 29 Apr, 2015 1 commit
  17. 28 Apr, 2015 1 commit
    • antirez's avatar
      Fix memory leak in async spontaneous reply handling · 2fc39eb4
      antirez authored
      When an asynchronous hiredis connection subscribes to a Pub/Sub channel
      and gets an error, and in other related conditions, the function
      redisProcessCallbacks() enters a code path where the link is
      disconnected, however the function returns before freeing the allocated
      reply object. This causes a memory leak. The memory leak was trivial to
      trigger in Redis Sentinel, which uses hiredis, every time we tried to
      subscribe to an instance that required a password, in case the Sentinel
      was configured either with the wrong password or without password at
      all. In this case, the -AUTH error caused the leaking code path to be
      executed.
      2fc39eb4
  18. 16 Apr, 2015 7 commits
  19. 29 Mar, 2015 1 commit