summaryrefslogtreecommitdiff
path: root/net.h
AgeCommit message (Collapse)Author
2014-04-09Fix const correctnessPieter Noordhuis
2014-04-08Add ability to bind source address on connectMatt Stancliff
Some environments require binding to specific source addresses instead of letting the system determine which IP a connection should originate from. Closes #233
2014-04-08Remove possiblity of multiple close on same fdMatt Stancliff
With all the async connects and disconnects and error handling going on in hiredis, we need to centralize how we close our fd and set it so it doesn't get re-closed. Prior to this commit, sometimes we'd close(fd), run an async error handler, then call close(fd) again. To stop multiple closes, we now set fd to -1 after we free it, but that requires not passing fd as an independent argument to functions. This commit moves all fd usage to c->fd. Since the context has a fd field and all functions receive the context, it makes more sense to use the fd inside of c instead of passing along fd as an independent argument. Also, by only using c->fd, we can set c->fd after we close it to signify we shouldn't re-close the same fd again. This does change one semi-public interface function redisCheckSocketError() to only take (context) instead of (context, fd). A search on github returned zero occasions of people using redisCheckSocketError() outside of net.{c,h} in hiredis itself. Commit inspired by the bug report at: https://groups.google.com/forum/#!topic/redis-db/mQm46XkIPOY Thanks go out to Thijs for trying high-frequency reconnects on a host that isn't there. Closes #230
2013-07-10Mark the timeout parameter as const in various functionsNoah Williamsson
The struct timeval argument in redisConnectWithTimeout(), redisConnectUnixWithTimeout(), redisSetTimeout(), redisContextSetTimeout(), redisContextConnectTcp() and redisContextConnectUnix() is never modified and can therefore be marked as const. Signed-off-by: Noah Williamsson <noah.williamsson@gmail.com>
2013-04-30Make KeepAlive optionalAllen.Dou
Make Connection KeepAlive being optional instead of default.
2011-06-27Extract function to check a socket for errorsPieter Noordhuis
2011-04-21Update licensePieter Noordhuis
2011-02-04Use select(2) for enforce a timeout on blocking connect(2)Pieter Noordhuis
2011-01-07Return error on socket timeout for a blocking contextPieter Noordhuis
2010-12-29LicensePieter Noordhuis
2010-12-16Solaris doesn't know AF_LOCALPieter Noordhuis
2010-11-03Add functiont to net.c to connect to a unix socketPieter Noordhuis
2010-11-02Strip net.c down to the bare minimumPieter Noordhuis
2010-11-02Move anet.{c,h} to net.{c,h}Pieter Noordhuis