2.4 #154

merged 1 commit into from Oct 26, 2011


None yet
2 participants

florean commented Oct 21, 2011

I apologize, I screwed up in my commit by not setting errno to zero before checking it after the strtol call. The other uses of it in config.c check to see if there was an err before checking errno, so it is safe to assume errno is set correctly. In my case, it is not. On my test system, it worked because there weren't any errors before I checked. But on the two very different production Linux servers I use (Ubuntu 10.10 and CentOS 5.6), something is generating a "File/Directory not found" (errno == 2) before it gets to my code and thus it fails every time. Here is the very simple fix.

@antirez antirez added a commit that referenced this pull request Oct 26, 2011

@antirez antirez Merge pull request #154 from florean/2.4

@antirez antirez merged commit 00af23b into antirez:2.4 Oct 26, 2011


antirez commented Oct 26, 2011

Thanks, merged.

@JackieXie168 JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Aug 29, 2016

@antirez antirez Merge pull request #154 from florean/2.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment