It wouldn't change the semantics, but perhaps better to set fd to -1 here instead of 0, so that it matches with redisContextCloseFd() if nothing else.