NPE when accessing FileFormatSPI #11088
Assignees
Comments
and core dump: |
Closed
JaroslavTulach
added
-libs
|
Please note the last line in this exception is at $ git diff std-bits/base/src/main/java/org/enso/base/polyglot/EnsoMeta.java
diff --git std-bits/base/src/main/java/org/enso/base/polyglot/EnsoMeta.java std-bits/base/src/main/java/org/enso/base/polyglot/EnsoMeta.java
index 249dcd2ad5..b56ff57aef 100644
--- std-bits/base/src/main/java/org/enso/base/polyglot/EnsoMeta.java
+++ std-bits/base/src/main/java/org/enso/base/polyglot/EnsoMeta.java
@@ -12,7 +12,13 @@ public final class EnsoMeta {
/** Returns a type object from the Enso runtime. */
public static Value getType(String moduleName, String typeName) {
var module = getBindings().invokeMember("get_module", moduleName);
- return module.invokeMember("get_type", typeName);
+ try {
+ return module.invokeMember("get_type", typeName);
+ } catch (NullPointerException e) {
+ var ex = new NullPointerException("Cannot get type for " + moduleName + " type: " + typeName + " at " + module);
+ ex.initCause(ex);
+ throw ex;
+ }
}
/** Calls a static method defined directly on a module (not inside of a type). */and wait until it reproduces. |
hubertp
added a commit
that referenced
this issue
Sep 18, 2024
We are seeing this problem almost daily and need more info rather urgently. Related to #11088.
hubertp
added a commit
that referenced
this issue
Sep 18, 2024
We are seeing this problem almost daily and need more info rather urgently. Related to #11088.
The enhanced logging diff has been integrated by #11125. Now we need a new bug report with the additional info. |
kazcw
added a commit
that referenced
this issue
Sep 19, 2024
- Fix debug logging for #11088--attempt to create an exception that is its own cause fails. - In case the parser is used after closing, throw an `IllegalStateException` instead of UB.
mergify bot
pushed a commit
that referenced
this issue
Sep 20, 2024
- Fix debug logging for #11088 case--attempt to create an exception that is its own cause fails. - In case the parser is used after closing, throw an `IllegalStateException` instead of UB. (This case is not known to occur and doesn't seem to be behind the #11121, but we should handle it more safely if it does.)
jdunkerley
pushed a commit
that referenced
this issue
Sep 26, 2024
jdunkerley
pushed a commit
that referenced
this issue
Sep 26, 2024
- Fix debug logging for #11088 case--attempt to create an exception that is its own cause fails. - In case the parser is used after closing, throw an `IllegalStateException` instead of UB. (This case is not known to occur and doesn't seem to be behind the #11121, but we should handle it more safely if it does.) (cherry picked from commit e587d56)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
and another exception reported as #11104
The text was updated successfully, but these errors were encountered: