Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test exit with segmentation fault #7

Closed
lp opened this issue Nov 15, 2010 · 3 comments
Closed

test exit with segmentation fault #7

lp opened this issue Nov 15, 2010 · 3 comments

Comments

@lp
Copy link

lp commented Nov 15, 2010

On OSX 10.6:

$ hiredis/ make
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   net.c
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   hiredis.c
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   sds.c
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   async.c
libtool -dynamic -o libhiredis.dylib -lm -g -ggdb  - net.o hiredis.o sds.o async.o
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   example.c
cc -o hiredis-example -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -lm -pthread   -g -ggdb  -L. -lhiredis -Wl,-rpath,. example.o
cc -c -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -arch i386 -arch x86_64 -g -ggdb   test.c
cc -o hiredis-test -std=c99 -pedantic -O2 -fPIC -Wall -W -Wwrite-strings   -lm -pthread   -g -ggdb  -L. -lhiredis -Wl,-rpath,. test.o
$ hiredis/ make test
./hiredis-test
##01 Format command without interpolation: PASSED
##02 Format command with %s string interpolation: PASSED
##03 Format command with %b string interpolation: PASSED
##04 Format command by passing argc/argv without lengths: PASSED
##05 Format command by passing argc/argv with lengths: PASSED
make: *** [test] Segmentation fault
@pietern
Copy link
Contributor

pietern commented Nov 15, 2010

Thanks for the report, looking into it. I think it's failing on not being able to connect to Redis. In that case, it should quit a little more friendly than segfault :-)

@lp
Copy link
Author

lp commented Nov 15, 2010

Also:
I updated my redis to 2.0.4
And the server is running while I run the test suite.

Thanks!

@pietern
Copy link
Contributor

pietern commented Nov 15, 2010

Running 2.0 made me realize I created the test against master, which returns OK on QUIT. I've resolved this by making this test conditional with respect to the version of Redis that is being tested against. Thanks again!

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants