-
Notifications
You must be signed in to change notification settings - Fork 40
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
Segfault #212
Closed
cpitclaudel opened this issue
Jun 19, 2020
· 1 comment
· Fixed by #214 or ocaml/opam-repository#17070
Closed
Segfault #212
cpitclaudel opened this issue
Jun 19, 2020
· 1 comment
· Fixed by #214 or ocaml/opam-repository#17070
Labels
Milestone
Comments
Thanks, fix in #214 ; a test has been added. However, a further issue appears here, namely that the verbose error serialization is way too verbose as it will include a huge env in this case, this should be likely addressed separately. I guess we want to reify errors upstream so these carrying the context can be handled in a more modular way, for now we will have to filter those by hand. |
cpitclaudel
added a commit
to cpitclaudel/alectryon
that referenced
this issue
Jun 26, 2020
ejgallego
added a commit
to ejgallego/opam-repository
that referenced
this issue
Aug 26, 2020
CHANGES: * [general] Require dune >= 2.0 (@ejgallego, ??) * [serapi] New query `Comments` to return all comments in a document (@ejgallego, rocq-archive/coq-serapi#20? , (partially) fixes rocq-archive/coq-serapi#191 , ejgallego/coq-serapi#200 ) * [general] Coq's error recovery is now disabled by default (@ejgallego , fixes rocq-archive/coq-serapi#201) * [general] New option `--error-recovery` to enable error recovery (@ejgallego , rocq-archive/coq-serapi#203) * [general] Bump sexplib dependency to v0.13 (@ejgallego , rocq-archive/coq-serapi#204) Fixes incorrect change in rocq-archive/coq-serapi#194. * [sertop] Set default value of allow-sprop to be true in agreement with upstream coq v8.11 and added option '--disallow-sprop' to optionally switch it off (--disallow-sprop forbids using the proof irrelevant SProp sort) (rocq-archive/coq-serapi#199, @pestun) * [sertop] Set default value of allow-sprop to be true in agreement with upstream coq v8.11 and added option '--disallow-sprop' to optionally switch it off (--disallow-sprop forbids using the proof irrelevant SProp sort) (@pestun , rocq-archive/coq-serapi#199) * [sertop] Added option `--topfile` to `sertop` to set top name from a filename (rocq-archive/coq-serapi#197, @pestun) * [deps] Require sexplib >= 0.12 , fixed deprecation warnings (rocq-archive/coq-serapi#194, @ejgallego) * [general] SerAPI is now tested with OCaml 4.08 and 4.09 (rocq-archive/coq-serapi#195 , @ejgallego) * [sertop ] Forward port sername from 0.7.1 (@ejgallego) * [serlib ] Fix rocq-archive/coq-serapi#212 "Segfault on universes" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#214) * [serapi ] Fix rocq-archive/coq-serapi#221 "Support COQPATH" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#224) * [sertop ] Fix rocq-archive/coq-serapi#222 "Support --indices-matter" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#223) * [sertop ] Fix "Stack overflow in main loop" (@pestun , rocq-archive/coq-serapi#216)
ejgallego
added a commit
to ejgallego/opam-repository
that referenced
this issue
Aug 27, 2020
CHANGES: * [general] Require dune >= 2.0 (@ejgallego, ??) * [serapi] New query `Comments` to return all comments in a document (@ejgallego, rocq-archive/coq-serapi#20? , (partially) fixes rocq-archive/coq-serapi#191 , ejgallego/coq-serapi#200 ) * [general] Coq's error recovery is now disabled by default (@ejgallego , fixes rocq-archive/coq-serapi#201) * [general] New option `--error-recovery` to enable error recovery (@ejgallego , rocq-archive/coq-serapi#203) * [general] Bump sexplib dependency to v0.13 (@ejgallego , rocq-archive/coq-serapi#204) Fixes incorrect change in rocq-archive/coq-serapi#194. * [sertop] Set default value of allow-sprop to be true in agreement with upstream coq v8.11 and added option '--disallow-sprop' to optionally switch it off (--disallow-sprop forbids using the proof irrelevant SProp sort) (rocq-archive/coq-serapi#199, @pestun) * [sertop] Set default value of allow-sprop to be true in agreement with upstream coq v8.11 and added option '--disallow-sprop' to optionally switch it off (--disallow-sprop forbids using the proof irrelevant SProp sort) (@pestun , rocq-archive/coq-serapi#199) * [sertop] Added option `--topfile` to `sertop` to set top name from a filename (rocq-archive/coq-serapi#197, @pestun) * [deps] Require sexplib >= 0.12 , fixed deprecation warnings (rocq-archive/coq-serapi#194, @ejgallego) * [general] SerAPI is now tested with OCaml 4.08 and 4.09 (rocq-archive/coq-serapi#195 , @ejgallego) * [sertop ] Forward port sername from 0.7.1 (@ejgallego) * [serlib ] Fix rocq-archive/coq-serapi#212 "Segfault on universes" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#214) * [serapi ] Fix rocq-archive/coq-serapi#221 "Support COQPATH" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#224) * [sertop ] Fix rocq-archive/coq-serapi#222 "Support --indices-matter" (@ejgallego, reported by @cpitclaudel , rocq-archive/coq-serapi#223) * [sertop ] Fix "Stack overflow in main loop" (@pestun , rocq-archive/coq-serapi#216)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @ejgallego,
The following example reliably causes a segfault on my machine:
Here's the backtrace:
And here's the full session. The segfault happens a few seconds after sending the last command:
I'm on 8.11.0+0.11.0
The text was updated successfully, but these errors were encountered: