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

bug(eval) SIGILL when execute EVAL command #121

Closed
yoshino-s opened this issue Jun 8, 2022 · 2 comments
Closed

bug(eval) SIGILL when execute EVAL command #121

yoshino-s opened this issue Jun 8, 2022 · 2 comments

Comments

@yoshino-s
Copy link

Version: dragonfly -128-NOTFOUND build time: 2022-06-06 13:36:06

Reproduce bug:

docker pull docker.dragonflydb.io/dragonflydb/dragonfly && \
docker tag docker.dragonflydb.io/dragonflydb/dragonfly dragonfly
docker run -p 6379:6379 --ulimit memlock=-1 -d --name dragonfly dragonfly
redis-cli

And run

EVAL "return {KEYS[1],KEYS[2],ARGV[1],ARGV[2]}" 2 key1 key2 first second [k

On Redis it will return

1) "key1"
2) "key2"
3) "first"
4) "second"

But on dragonfly db it will throw an error.

Log:

I20220608 01:52:41.115177     1 init.cc:56] dragonfly running in opt mode.
I20220608 01:52:41.115240     1 dfly_main.cc:179] maxmemory has not been specified. Deciding myself....
I20220608 01:52:41.115280     1 dfly_main.cc:184] Found 12.54GiB available memory. Setting maxmemory to 10.04GiB
I20220608 01:52:41.116029    10 proactor.cc:456] IORing with 1024 entries, allocated 102720 bytes, cq_entries is 2048
I20220608 01:52:41.117841     1 proactor_pool.cc:66] Running 12 io threads
I20220608 01:52:41.125202     1 server_family.cc:190] Data directory is "/data"
I20220608 01:52:41.125350     1 server_family.cc:114] Checking "/data/dump"
I20220608 01:52:41.125839    12 listener_interface.cc:79] sock[39] AcceptServer - listening on port 6379
*** SIGILL received at time=1654653180 on cpu 0 ***
PC: @     0x562988468a97  (unknown)  luaH_newkey
    @ ... and at least 1 more frames
@romange
Copy link
Collaborator

romange commented Jun 8, 2022

Thanks for reporting this. Yeah, it's related to #68 .

I will release a new version today that hopefully will fix it. ✌🏼

@romange
Copy link
Collaborator

romange commented Jun 8, 2022

fixed here: https://github.com/dragonflydb/dragonfly/releases/tag/v0.2.0
new docker image will be ready in 10min.

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