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

[alpha.3] libevent2.d(1140): Range violation #1364

Closed
9il opened this Issue Jan 5, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@9il
Contributor

9il commented Jan 5, 2016

core.exception.RangeError@../../.dub/packages/vibe-d-0.7.27-alpha.3/source/vibe/core/drivers/libevent2.d(1140): Range violation
violation
----------------
??:? _d_arraybounds [0x952363]
??:? vibe.core.drivers.libevent2.__array [0x918fbf]
../../.dub/packages/vibe-d-0.7.27-alpha.3/source/vibe/core/drivers/libevent2.d:1140 bool vibe.core.drivers.libevent2.InotifyDirectoryWatcher.readChanges(ref vibe.core.file.DirectoryChange[], core.time.Duration) [0x8bc02e]
source/app.d:178 void app.start(uint, uint) [0x78b2c7]
source/app.d:53 void app.main(immutable(char)[][]).__lambda2(uint, uint) [0x78abdd]
../../.dub/packages/vibe-d-0.7.27-alpha.3/source/vibe/core/core.d:486 void vibe.core.core.makeTaskFuncInfo!(void delegate(uint, uint), uint, uint).makeTaskFuncInfo(ref void delegate(uint, uint), ref uint, ref uint).callDelegate(vibe.core.core.TaskFuncInfo*) [0x79fc7d]
../../.dub/packages/vibe-d-0.7.27-alpha.3/source/vibe/core/core.d:1039 void vibe.core.core.CoreTask.run() [0x8b2ebf]
??:? void core.thread.Fiber.run() [0x9aa921]
??:? fiber_entryPoint [0x9aa69b]
??:? [0xffffffff]
Converted /home/ilya/data/grad_sharded/merger_20151223T075036Z_audio_kp2_26998510_27001509_v0_i2.json.gz: 10.5282M json, 1.2866M json.gz, 0.1501M bin, 0.045490M bin.gz
Converted /home/ilya/data/grad_sharded/merger_20151223T094808Z_audio_kp50_27052489_27055488_v0_i2.json.gz: 11.6206M json, 1.3948M json.gz, 0.1580M bin, 0.047737M bin.gz
Task terminated with unhandled exception: Range violation
@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Jan 5, 2016

Member

Looks strange. Looking at the code, I couldn't see an issue on vibe.d's side. I'll add an explicit check and a log message for this case to at least avoid the crash.

Member

s-ludwig commented Jan 5, 2016

Looks strange. Looking at the code, I couldn't see an issue on vibe.d's side. I'll add an explicit check and a log message for this case to at least avoid the crash.

@etcimon

This comment has been minimized.

Show comment
Hide comment
@etcimon

etcimon Jan 5, 2016

Contributor

It happens, all the time. Linux can pass some garbage file descriptors or watch descriptors for some reason. That was a big question mark for me in libasync, until I just decided there's probably a lot of undocumented stuff going on. Checking for validity on the descriptor is something that always needs to be done because of that.

Contributor

etcimon commented Jan 5, 2016

It happens, all the time. Linux can pass some garbage file descriptors or watch descriptors for some reason. That was a big question mark for me in libasync, until I just decided there's probably a lot of undocumented stuff going on. Checking for validity on the descriptor is something that always needs to be done because of that.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Jan 5, 2016

Member

Okay, good to know.

Member

s-ludwig commented Jan 5, 2016

Okay, good to know.

@9il

This comment has been minimized.

Show comment
Hide comment
@9il

9il Jan 5, 2016

Contributor

Looks strange. Looking at the code, I couldn't see an issue on vibe.d's side. I'll add an explicit check and a log message for this case to at least avoid the crash.

Thanks!

Contributor

9il commented Jan 5, 2016

Looks strange. Looking at the code, I couldn't see an issue on vibe.d's side. I'll add an explicit check and a log message for this case to at least avoid the crash.

Thanks!

@s-ludwig s-ludwig closed this in 90ed699 Jan 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment