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

Missing CAMLreturn in asmrun/io.c/caml_ml_flush and caml_ml_flush_partial #4109

Closed
vicuna opened this Issue Sep 11, 2006 · 2 comments

Comments

Projects
None yet
1 participant
@vicuna
Copy link
Collaborator

vicuna commented Sep 11, 2006

Original bug ID: 4109
Reporter: necula
Status: closed (set by @damiendoligez on 2007-02-21T13:20:15Z)
Resolution: fixed
Priority: normal
Severity: crash
Version: 3.09.2
Fixed in version: 3.09.3
Category: ~DO NOT USE (was: OCaml general)
Duplicate of: #4039

Bug description

The functions caml_ml_flush and caml_ml_flush_partial in asmrun/io.c can return without CAMLreturn if they are called on a file that was already closed. This screws up the caml_local_roots data structure, which results in crashes.

Additional information

I am pretty sure that I have filed an identical report a few days ago, and I confirmed that it showed in View Issues. Now I do not see it anymore. If somebody is deleting these reports I expect that they would at least bother to send me email to let me know the reason.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Sep 11, 2006

Comment author: @mmottl

This bug has already been fixed in the CVS a long time ago. The upcoming release will work fine.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Sep 12, 2006

Comment author: @damiendoligez

It was not deleted. It was closed because the bug is fixed.

If you have the same bug in 3.09.3+rc2, I would like to hear about it.
If you try 3.09.3+rc2 and it solves the problem, I would like to
get confirmation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.