This is an automated email from the ASF dual-hosted git repository.
nmalin pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git
The following commit(s) were added to refs/heads/trunk by this push:
new d4127a683e Fixed: Escape potential NPE after jobTracker start (#2094)
d4127a683e is described below
commit d4127a683eac9c707544b6e58ef7452e3fee7965
Author: Nicolas Malin <[email protected]>
AuthorDate: Mon Sep 28 09:30:39 2026 +0200
Fixed: Escape potential NPE after jobTracker start (#2094)
After a JobTracker start, if the linked runtime isn't present, all
operation to manage it failed due to NPE. To be sure that we keep han on
it, during information collect, if no runtime present we just return
empty.
---
.../java/org/apache/ofbiz/service/tracker/JobTracker.java | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git
a/framework/service/src/main/java/org/apache/ofbiz/service/tracker/JobTracker.java
b/framework/service/src/main/java/org/apache/ofbiz/service/tracker/JobTracker.java
index c5c372510f..8c7c710290 100644
---
a/framework/service/src/main/java/org/apache/ofbiz/service/tracker/JobTracker.java
+++
b/framework/service/src/main/java/org/apache/ofbiz/service/tracker/JobTracker.java
@@ -275,11 +275,14 @@ public class JobTracker {
private Map<String, Object> retrieveServiceParams() throws
GenericEntityException {
GenericValue runtimeData = delegator
.findOne("RuntimeData", true, "runtimeDataId", runtimeDataId);
- try {
- return UtilGenerics.checkMap(
-
XmlSerializer.deserialize(runtimeData.getString("runtimeInfo"), delegator),
String.class, Object.class);
- } catch (SerializeException | SAXException |
ParserConfigurationException | IOException e) {
- throw new RuntimeException(e);
+ if (runtimeData != null) {
+ try {
+ return UtilGenerics.checkMap(
+
XmlSerializer.deserialize(runtimeData.getString("runtimeInfo"), delegator),
String.class, Object.class);
+ } catch (SerializeException | SAXException |
ParserConfigurationException | IOException e) {
+ throw new RuntimeException(e);
+ }
}
+ return Map.of();
}
}