search

djveremix / redis

Automatically exported from code.google.com/p/redis
0 stars 0 forks source link

redis-server crashes on ZUNION #216

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
redis-server crashes when I'm executing a command like "ZUNION key 2 k1 k2".
But if I modify the command to "ZUNION key 2 'k1' 'k2'" everything works fine.

This is what redis-server returns on crash:

[1049] 30 Mar 14:13:54 # 1 redis-server 0x100006cff _redisAssert + 95
[1049] 30 Mar 14:13:54 # 2   ???                                 
0x0000000000000000 0x0 + 0
[1049] 30 Mar 14:13:54 # 3 redis-server 0x1000080b6 decrRefCount + 118
[1049] 30 Mar 14:13:54 # 4 redis-server 0x1000144b0 zunionInterGenericCommand + 
1344
[1049] 30 Mar 14:13:54 # 5 redis-server 0x10000bbb4 call + 36
[1049] 30 Mar 14:13:54 # 6 redis-server 0x100014c00 processCommand + 1248
[1049] 30 Mar 14:13:54 # 7 redis-server 0x100014dfb processInputBuffer + 459
[1049] 30 Mar 14:13:54 # 8 redis-server 0x100016e60 readQueryFromClient + 112
[1049] 30 Mar 14:13:54 # 9   redis-server                        
0x0000000100001936 aeProcessEvents 
+ 406
[1049] 30 Mar 14:13:54 # 10  redis-server                        
0x0000000100001b3e aeMain + 62
[1049] 30 Mar 14:13:54 # 11  redis-server                        
0x0000000100013a38 main + 232
[1049] 30 Mar 14:13:54 # 12  redis-server                        
0x0000000100000e74 start + 52
[1049] 30 Mar 14:13:54 # 13  ???                                 
0x0000000000000002 0x0 + 2

The host of the redis-server is Mac OS X.

Original issue reported on code.google.com by mett...@googlemail.com on 30 Mar 2010 at 12:21

GoogleCodeExporter commented 8 years ago 
Sorry, version is 1.3.8

Original comment by mett...@googlemail.com on 30 Mar 2010 at 12:26

GoogleCodeExporter commented 8 years ago 
Please can you provide the full error log? There should be more info than the 
stack trace.
Also if you have a script that can reproduce this, it will be *much* simpler to 
fix.

Thanks for reporting.

Original comment by anti...@gmail.com on 30 Mar 2010 at 12:29

GoogleCodeExporter commented 8 years ago 
Bug has been identified. The wrong variable was passed to decrRefCount when the 
resulting zset was empty. 
Added a test to the suite and patched the source: 
http://github.com/pietern/redis/tree/issue_216

Original comment by pcnoordh...@gmail.com on 1 Apr 2010 at 8:09

GoogleCodeExporter commented 8 years ago 
Merged in redis master, thanks.

Original comment by anti...@gmail.com on 2 Apr 2010 at 8:01